Fixing malformed Javadoc warnings in jetty-annotations

jetty-annotations/src/main/java/org/eclipse/jetty/annotations/AnnotationConfiguration.java

@@ -257,9 +257,9 @@ public class AnnotationConfiguration extends AbstractConfiguration
     /**
      * Check to see if the ServletContainerIntializer loaded via the ServiceLoader came
      * from a jar that is excluded by the fragment ordering. See ServletSpec 3.0 p.85.
-     * @param orderedJars
+     * @param context
      * @param service
-     * @return
+     * @return true if excluded
      */
     public boolean isFromExcludedJar (WebAppContext context, ServletContainerInitializer service)
     throws Exception
@@ -298,7 +298,7 @@ public class AnnotationConfiguration extends AbstractConfiguration
 
     /**
      * @param context
-     * @return
+     * @return list of non-excluded {@link ServletContainerInitializer}s
      * @throws Exception
      */
     public List<ServletContainerInitializer> getNonExcludedInitializers (WebAppContext context)
@@ -508,7 +508,7 @@ public class AnnotationConfiguration extends AbstractConfiguration
      * 
      * @param jar
      * @param frags
-     * @return
+     * @return the fragment if found, or null of not found
      * @throws Exception
      */
     public FragmentDescriptor getFragmentFromJar (Resource jar,  List<FragmentDescriptor> frags)

jetty-annotations/src/main/java/org/eclipse/jetty/annotations/AnnotationParser.java

@@ -237,8 +237,6 @@ public class AnnotationParser
         
         /**
          * Get the name of the annotation processed by this handler. Can be null
-         * 
-         * @return
          */
         public String getAnnotationName();
     }
@@ -513,7 +511,7 @@ public class AnnotationParser
      * Register a handler that will be called back when the named annotation is
      * encountered on a class.
      *
-     * @deprecated see registerHandler(Handler)
+     * @deprecated see {@link #registerHandler(Handler)}
      * @param annotationName
      * @param handler
      */
@@ -525,9 +523,8 @@ public class AnnotationParser
 
     
     /**
-     * @deprecated
+     * @deprecated no replacement provided
      * @param annotationName
-     * @return
      */
     @Deprecated
     public List<DiscoverableAnnotationHandler> getAnnotationHandlers(String annotationName)
@@ -547,8 +544,7 @@ public class AnnotationParser
     }
 
     /**
-     * @deprecated
+     * @deprecated no replacement available
-     * @return
      */
     @Deprecated
     public List<DiscoverableAnnotationHandler> getAnnotationHandlers()
@@ -563,7 +559,7 @@ public class AnnotationParser
     }
 
     /**
-     * @deprecated see registerHandler(Handler)
+     * @deprecated see {@link #registerHandler(Handler)}
      * @param handler
      */
     @Deprecated
@@ -605,7 +601,6 @@ public class AnnotationParser
      * Remove a particular handler
      * 
      * @param h
-     * @return
      */
     public boolean deregisterHandler(Handler h)
     {
@@ -625,7 +620,6 @@ public class AnnotationParser
     /**
      * True if the class has already been processed, false otherwise
      * @param className
-     * @return
      */
     public boolean isParsed (String className)
     {

jetty-annotations/src/main/java/org/eclipse/jetty/annotations/ServletSecurityAnnotationHandler.java

@@ -126,7 +126,6 @@ public class ServletSecurityAnnotationHandler extends AbstractIntrospectableAnno
      * @param rolesAllowed
      * @param permitOrDeny
      * @param transport
-     * @return
      */
     protected Constraint makeConstraint (Class servlet, String[] rolesAllowed, EmptyRoleSemantic permitOrDeny, TransportGuarantee transport)
     {
@@ -138,7 +137,6 @@ public class ServletSecurityAnnotationHandler extends AbstractIntrospectableAnno
     /**
      * Get the ServletMappings for the servlet's class.
      * @param className
-     * @return
      */
     protected List<ServletMapping> getServletMappings(String className)
     {
@@ -160,7 +158,6 @@ public class ServletSecurityAnnotationHandler extends AbstractIntrospectableAnno
      * Check if there are already <security-constraint> elements defined that match the url-patterns for
      * the servlet.
      * @param servletMappings
-     * @return
      */
     protected boolean constraintsExist (List<ServletMapping> servletMappings, List<ConstraintMapping> constraintMappings)
     {

jetty-annotations/src/main/java/org/eclipse/jetty/annotations/WebFilterAnnotation.java

@@ -61,7 +61,7 @@ public class WebFilterAnnotation extends DiscoveredAnnotation
     }
 
     /**
-     * @see org.eclipse.jetty.annotations.ClassAnnotation#apply()
+     * @see DiscoveredAnnotation#apply()
      */
     public void apply()
     {

jetty-annotations/src/main/java/org/eclipse/jetty/annotations/WebListenerAnnotation.java

@@ -58,7 +58,7 @@ public class WebListenerAnnotation extends DiscoveredAnnotation
     }
 
     /**
-     * @see org.eclipse.jetty.annotations.ClassAnnotation#apply()
+     * @see DiscoveredAnnotation#apply()
      */
     public void apply()
     {

jetty-annotations/src/main/java/org/eclipse/jetty/annotations/WebServletAnnotation.java

@@ -57,7 +57,7 @@ public class WebServletAnnotation extends DiscoveredAnnotation
     }
     
     /**
-     * @see org.eclipse.jetty.annotations.ClassAnnotation#apply()
+     * @see DiscoveredAnnotation#apply()
      */
     public void apply()
     {