more javadoc fixes #2056

Signed-off-by: olivier lamy <olamy@webtide.com>

jetty-jaas/src/main/java/org/eclipse/jetty/jaas/spi/LdapLoginModule.java

@@ -184,8 +184,9 @@ public class LdapLoginModule extends AbstractLoginModule
         Attributes attributes;
 
         /**
-         * @param userName
-         * @param credential
+         * @param userName the user name
+         * @param credential the credential
+         * @param attributes the user {@link Attributes}
          */
         public LDAPUserInfo(String userName, Credential credential, Attributes attributes)
         {
@@ -266,25 +267,21 @@ public class LdapLoginModule extends AbstractLoginModule
      * NOTE: this is not an user authenticated operation
      *
      * @param username
-     * @return
+     * @return the {@link Attributes} from the user
      * @throws LoginException
      */
     private Attributes getUserAttributes(String username) throws LoginException
     {
-        Attributes attributes = null;
-
-        SearchResult result;
         try
         {
-            result = findUser(username);
-            attributes = result.getAttributes();
+            SearchResult result = findUser(username);
+            Attributes attributes = result.getAttributes();
+            return attributes;
         }
         catch (NamingException e)
         {
             throw new LoginException("Root context binding failure.");
         }
-
-        return attributes;
     }
 
     private String getUserCredentials(Attributes attributes) throws LoginException

jetty-jaas/src/main/java/org/eclipse/jetty/jaas/spi/UserInfo.java

@@ -42,9 +42,9 @@ public class UserInfo
     
     
     /**
-     * @param userName
-     * @param credential
-     * @param roleNames
+     * @param userName the user name
+     * @param credential the credential
+     * @param roleNames a {@link List} of role name
      */
     public UserInfo (String userName, Credential credential, List<String> roleNames)
     {
@@ -76,16 +76,16 @@ public class UserInfo
      * Should be overridden by subclasses to obtain
      * role info
      * 
-     * @return
-     * @throws Exception
+     * @return List of role associated to the user
+     * @throws Exception if the roles cannot be retrieved
      */
-    public List<String> doFetchRoles ()
+    public List<String> doFetchRoles()
     throws Exception
     {
         return Collections.emptyList();
     }
     
-    public void fetchRoles () throws Exception
+    public void fetchRoles() throws Exception
     {
         synchronized (_roleNames)
         {

jetty-quickstart/src/main/java/org/eclipse/jetty/quickstart/QuickStartDescriptorProcessor.java

@@ -123,7 +123,7 @@ public class QuickStartDescriptorProcessor extends IterativeDescriptorProcessor
      * @param context  the webapp 
      * @param descriptor the xml file to process
      * @param node the context-param node in the xml file
-     * @throws Exception
+     * @throws Exception if some resources cannot be read
      */
     public void visitContextParam (WebAppContext context, Descriptor descriptor, XmlParser.Node node)
             throws Exception