You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2018/08/03 04:27:05 UTC

[directory-ldap-api] branch master updated: o Fixed all the remaining Javadoc errors, and many warnings o Changed the pom.xml so that javadoc lint is on, with a higher limit for warnings, in order to be able to run mvn site fully.

This is an automated email from the ASF dual-hosted git repository.

elecharny pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/directory-ldap-api.git


The following commit(s) were added to refs/heads/master by this push:
     new 5212950  o Fixed all the remaining Javadoc errors, and many warnings o Changed the pom.xml so that javadoc lint is on, with a higher limit for warnings, in order to be able to run mvn site fully.
5212950 is described below

commit 52129503d7f2247bad4a5f73b234023b8158ff07
Author: Emmanuel Lécharny <el...@symas.com>
AuthorDate: Fri Aug 3 06:27:02 2018 +0200

    o Fixed all the remaining Javadoc errors, and many warnings
    o Changed the pom.xml so that javadoc lint is on, with a higher limit
    for warnings, in order to be able to run mvn site fully.
---
 .../directory/api/dsmlv2/engine/Dsmlv2Engine.java  |  6 +-
 .../api/AbstractPoolableLdapConnectionFactory.java |  6 +-
 .../ldap/client/api/DefaultSchemaLoader.java       |  5 +-
 .../ldap/client/api/LdapNetworkConnection.java     | 36 ++++++++--
 .../directory/ldap/client/api/LdifAnonymizer.java  | 36 ++++++++++
 .../ldap/client/api/SaslExternalRequest.java       |  2 +
 .../ldap/client/api/future/HandshakeFuture.java    |  5 --
 .../api/search/AttributeDescriptionFilter.java     |  2 +
 .../api/search/AttributeValueAssertionFilter.java  | 14 ++--
 .../ldap/client/api/search/FilterBuilder.java      |  2 +
 .../ldap/client/api/search/FilterOperator.java     |  8 ++-
 .../ldap/client/api/search/SetOfFiltersFilter.java |  2 +
 .../ldap/client/api/search/SubstringFilter.java    |  5 ++
 .../directory/api/ldap/codec/api/LdapEncoder.java  |  1 +
 .../trigger/src/main/antlr/TriggerSpecification.g  |  4 +-
 .../directory/api/ldap/trigger/LdapOperation.java  |  2 +-
 .../ReusableAntlrTriggerSpecificationParser.java   |  2 +
 .../directory/api/ldap/util/tree/DnNode.java       | 24 ++++++-
 .../ldap/schema/loader/AttributeClassLoader.java   | 11 ++++
 .../ldap/schema/loader/JarLdifSchemaLoader.java    |  3 +-
 .../api/ldap/schema/loader/LdifSchemaLoader.java   |  3 +-
 .../ldap/schema/loader/SchemaEntityFactory.java    | 76 ++++++++++++++++++++--
 .../ldap/schema/loader/SingleLdifSchemaLoader.java |  7 ++
 .../schema/manager/impl/DefaultSchemaManager.java  | 68 ++++++++++++++++++-
 pom.xml                                            |  6 +-
 25 files changed, 292 insertions(+), 44 deletions(-)

diff --git a/dsml/engine/src/main/java/org/apache/directory/api/dsmlv2/engine/Dsmlv2Engine.java b/dsml/engine/src/main/java/org/apache/directory/api/dsmlv2/engine/Dsmlv2Engine.java
index c9e09d4..5fd6103 100644
--- a/dsml/engine/src/main/java/org/apache/directory/api/dsmlv2/engine/Dsmlv2Engine.java
+++ b/dsml/engine/src/main/java/org/apache/directory/api/dsmlv2/engine/Dsmlv2Engine.java
@@ -31,8 +31,6 @@ import java.io.OutputStreamWriter;
 import java.nio.charset.StandardCharsets;
 
 import org.apache.commons.codec.Charsets;
-import org.apache.directory.api.asn1.DecoderException;
-import org.apache.directory.api.asn1.EncoderException;
 import org.apache.directory.api.dsmlv2.DsmlDecorator;
 import org.apache.directory.api.dsmlv2.Dsmlv2Parser;
 import org.apache.directory.api.dsmlv2.ParserUtils;
@@ -190,7 +188,7 @@ public class Dsmlv2Engine
      * @param fileName the path to the file
      * @return the XML response in DSMLv2 Format
      * @throws XmlPullParserException if an error occurs in the parser
-     * @throws FileNotFoundException if the file does not exist
+     * @throws IOException if the file does not exist
      */
     public String processDSMLFile( String fileName ) throws XmlPullParserException, IOException
     {
@@ -795,8 +793,6 @@ public class Dsmlv2Engine
      * 
      * @param messageId the message Id
      * @throws LdapException If we had an issue while binding
-     * @throws EncoderException If we had an issue while encoding the request
-     * @throws DecoderException If we had an issue while decoding the request
      * @throws IOException If we had an issue while transmitting the request or re ceiving the response
      */
     protected void bind( int messageId ) throws LdapException, IOException
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/AbstractPoolableLdapConnectionFactory.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/AbstractPoolableLdapConnectionFactory.java
index 6d58853..0178979 100644
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/AbstractPoolableLdapConnectionFactory.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/AbstractPoolableLdapConnectionFactory.java
@@ -176,9 +176,9 @@ public abstract class AbstractPoolableLdapConnectionFactory implements PooledObj
      * will only occur if the connection pool was configured to validate.  This
      * means one of:
      * <ul>
-     * <li>{@link org.apache.commons.pool.impl.GenericObjectPool#setTestOnBorrow setTestOnBorrow}</li>
-     * <li>{@link org.apache.commons.pool.impl.GenericObjectPool#setTestWhileIdle setTestWhileIdle}</li>
-     * <li>{@link org.apache.commons.pool.impl.GenericObjectPool#setTestOnReturn setTestOnReturn}</li>
+     * <li>{@link org.apache.commons.pool2.impl.GenericObjectPool#setTestOnBorrow setTestOnBorrow}</li>
+     * <li>{@link org.apache.commons.pool2.impl.GenericObjectPool#setTestWhileIdle setTestWhileIdle}</li>
+     * <li>{@link org.apache.commons.pool2.impl.GenericObjectPool#setTestOnReturn setTestOnReturn}</li>
      * </ul>
      * must have been set to true on the pool.  The default validator is 
      * {@link LookupLdapConnectionValidator}.
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/DefaultSchemaLoader.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/DefaultSchemaLoader.java
index 2c30cf9..af8802b 100644
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/DefaultSchemaLoader.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/DefaultSchemaLoader.java
@@ -239,7 +239,7 @@ public class DefaultSchemaLoader extends AbstractSchemaLoader
      * @param rootDse the Root DSE
      * @return <code>true</code> if this is an ApacheDS server,
      *         <code>false</code> if not.
-     * @throws LdapInvalidAttributeValueException
+     * @throws LdapInvalidAttributeValueException If the vendorName attribute contains an invalid value 
      */
     private boolean isApacheDs( Entry rootDse ) throws LdapInvalidAttributeValueException
     {
@@ -260,8 +260,7 @@ public class DefaultSchemaLoader extends AbstractSchemaLoader
     /**
      * Load all the schemas.
      * 
-     * @param subschemaSubentryDn
-     * @throws LdapException
+     * @throws LdapException If one schema can't be loaded
      */
     private void loadSchemas() throws LdapException
     {
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java
index 16f1456..72ad1e6 100644
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java
@@ -492,6 +492,8 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
 
     /**
      * Create the connector
+     * 
+     * @throws LdapException If the connector can't be created
      */
     private void createConnector() throws LdapException
     {
@@ -557,7 +559,7 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
      * Check that a session is valid, ie we can send requests to the
      * server
      *
-     * @throws Exception If the session is not valid
+     * @throws InvalidConnectionException If the session is not valid
      */
     private void checkSession() throws InvalidConnectionException
     {
@@ -1190,6 +1192,8 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
 
     /**
      * Internal AbandonRequest handling
+     * 
+     * @param abandonRequest The request to abandon
      */
     private void abandonInternal( AbandonRequest abandonRequest )
     {
@@ -1477,6 +1481,10 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
 
     /**
      * Create a Simple BindRequest ready to be sent.
+     * 
+     * @param name The Bind name
+     * @param credentials The Bind credentials
+     * @return The created BindRequest instance
      */
     private BindRequest createBindRequest( String name, byte[] credentials )
     {
@@ -1486,6 +1494,10 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
 
     /**
      * Create a Simple BindRequest ready to be sent.
+     * 
+     * @param name The Bind name
+     * @param credentials The Bind credentials
+     * @return The created BindRequest instance
      */
     private BindRequest createBindRequest( Dn name, byte[] credentials )
     {
@@ -2386,7 +2398,7 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
      *
      * @param session The session we got the exception on
      * @param cause The exception cause
-     * @throws Exception The t
+     * @throws Exception If we have had another exception
      */
     @Override
     public void exceptionCaught( IoSession session, Throwable cause ) throws Exception
@@ -2418,6 +2430,9 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
 
     /**
      * Check if the message is a NoticeOfDisconnect message
+     * 
+     * @param message The message to check
+     * @return <tt>true</tt> if the message is a Notice of Disconnect
      */
     private boolean isNoticeOfDisconnect( Message message )
     {
@@ -4356,7 +4371,9 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
 
     /**
      * fetches the rootDSE from the server
-     * @throws LdapException
+     * 
+     * @param explicitAttributes The list of requested attributes
+     * @throws LdapException If we weren't bale to fetch the RootDSE
      */
     private void fetchRootDSE( String... explicitAttributes ) throws LdapException
     {
@@ -4630,7 +4647,9 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
 
 
     /**
-     * adds {@link SslFilter} to the IOConnector or IOSession's filter chain
+     * Adds {@link SslFilter} to the IOConnector or IOSession's filter chain
+     * 
+     * @throws LdapException If the SSL filter addition failed
      */
     private void addSslFilter() throws LdapException
     {
@@ -4897,7 +4916,10 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
 
 
     /**
-     * a reusable code block to be used in various bind methods
+     * A reusable code block to be used in various bind methods
+     * 
+     * @param request The request to send
+     * @throws LdapException If the request was ot properly sent
      */
     private void writeRequest( Request request ) throws LdapException
     {
@@ -4973,7 +4995,11 @@ public class LdapNetworkConnection extends AbstractLdapConnection implements Lda
      *     }
      * </pre>
      *
+     * @param realmName The realm name
+     * @param kdcHost The Kerberos server host
+     * @param kdcPort The Kerberos server port
      * @return the full path of the config file
+     * @throws IOException If the config file cannot be created
      */
     private String createKrb5ConfFile( String realmName, String kdcHost, int kdcPort ) throws IOException
     {
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdifAnonymizer.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdifAnonymizer.java
index c37cd14..db22d7e 100644
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdifAnonymizer.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdifAnonymizer.java
@@ -187,6 +187,8 @@ public class LdifAnonymizer
     
     /**
      * Print the string into the PrintStream, with a NL at the end
+     * 
+     * @param str The string to print
      */
     private void println( String str )
     {
@@ -211,6 +213,11 @@ public class LdifAnonymizer
 
     /**
      * Initialize the anonymizer, filling the maps we use.
+     * 
+     * @param stringLatestValueMap The map of already seen Strings
+     * @param binaryLatestValueMap The map of already seen byte[]
+     * @param integerLatestValueMap  The map of already seen Integers
+     * @param telephoneNumberLatestValueMap   The map of already seen telephone numbers
      */
     private void init( Map<Integer, String> stringLatestValueMap, Map<Integer, byte[]> binaryLatestValueMap, 
         Map<Integer, String> integerLatestValueMap, Map<Integer, String> telephoneNumberLatestValueMap )
@@ -388,6 +395,11 @@ public class LdifAnonymizer
     
     /**
      * Anonymize an AVA
+     * 
+     * @param ava The AVA to anonymize
+     * @return The anonymized AVA
+     * @throws LdapInvalidDnException If the Ava is invalid
+     * @throws LdapInvalidAttributeValueException If teh Ava content is invalid
      */
     private Ava anonymizeAva( Ava ava ) throws LdapInvalidDnException, LdapInvalidAttributeValueException
     {
@@ -446,6 +458,10 @@ public class LdifAnonymizer
     
     /**
      * Anonymize the entry's DN
+     * 
+     * @param entryDn The DN to anonymize
+     * @return The anonymized DN
+     * @throws LdapException If the anonymization failed
      */
     private Dn anonymizeDn( Dn entryDn ) throws LdapException
     {
@@ -637,6 +653,10 @@ public class LdifAnonymizer
     
     /**
      * Anonymize a Modify change
+     * 
+     * @param ldifEntry The entry to anonymize
+     * @return The anonymized entry
+     * @throws LdapException If the anonymization failed
      */
     private LdifEntry anonymizeChangeModify( LdifEntry ldifEntry ) throws LdapException
     {
@@ -704,6 +724,10 @@ public class LdifAnonymizer
     
     /**
      * Anonymize a Add change
+     * 
+     * @param ldifEntry The entry to anonymize
+     * @return The anonymized entry
+     * @throws LdapException If the anonymization failed
      */
     private LdifEntry anonymizeChangeAdd( LdifEntry ldifEntry ) throws LdapException
     {
@@ -761,6 +785,10 @@ public class LdifAnonymizer
     
     /**
      * Anonymize a Delete change
+     * 
+     * @param ldifEntry The entry to anonymize
+     * @return The anonymized entry
+     * @throws LdapException If the anonymization failed
      */
     private LdifEntry anonymizeChangeDelete( LdifEntry ldifEntry ) throws LdapException
     {
@@ -777,6 +805,10 @@ public class LdifAnonymizer
     
     /**
      * Anonymize a Delete change
+     * 
+     * @param ldifEntry The entry to anonymize
+     * @return The anonymized entry
+     * @throws LdapException If the anonymization failed
      */
     private LdifEntry anonymizeChangeModDn( LdifEntry ldifEntry ) throws LdapException
     {
@@ -816,6 +848,10 @@ public class LdifAnonymizer
     
     /**
      * Anonymize the full entry
+     * 
+     * @param ldifEntry The entry to anonymize
+     * @return The anonymized entry
+     * @throws LdapException If the anonymization failed
      */
     private Entry anonymizeEntry( LdifEntry ldifEntry ) throws LdapException
     {
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/SaslExternalRequest.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/SaslExternalRequest.java
index c438d47..014d6cc 100644
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/SaslExternalRequest.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/SaslExternalRequest.java
@@ -42,6 +42,8 @@ public class SaslExternalRequest extends AbstractSaslRequest
     
     /**
      * Creates a new instance of SaslExternalRequest.
+     * 
+     * @param authzId The authorization ID
      */
     public SaslExternalRequest( String authzId )
     {
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/future/HandshakeFuture.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/future/HandshakeFuture.java
index d32a403..cf8e869 100644
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/future/HandshakeFuture.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/future/HandshakeFuture.java
@@ -41,9 +41,6 @@ public class HandshakeFuture implements Future<Boolean>
 
     /**
      * Creates a new instance of HandshakeFuture.
-     *
-     * @param connection the LDAP connection
-     * @param messageId The associated messageId
      */
     public HandshakeFuture()
     {
@@ -67,8 +64,6 @@ public class HandshakeFuture implements Future<Boolean>
 
     /**
      * Set the Future to done when the TLS handshake has completed
-     * 
-     * @throws InterruptedException if the operation has been cancelled by client
      */
     public synchronized void secured()
     {
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/AttributeDescriptionFilter.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/AttributeDescriptionFilter.java
index c45052c..aaf5b49 100755
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/AttributeDescriptionFilter.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/AttributeDescriptionFilter.java
@@ -33,6 +33,8 @@ package org.apache.directory.ldap.client.api.search;
 
     /**
      * Creates a new instance of AttributeDescription filter.
+     * 
+     * @param attribute The attribute to set
      */
     private AttributeDescriptionFilter( String attribute )
     {
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/AttributeValueAssertionFilter.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/AttributeValueAssertionFilter.java
index c9985eb..c241fa2 100755
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/AttributeValueAssertionFilter.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/AttributeValueAssertionFilter.java
@@ -24,7 +24,7 @@ import org.apache.directory.api.ldap.model.filter.FilterEncoder;
 
 
 /**
- * A class to represent the various filters that take a value, like =, <=, >= or ~=.
+ * A class to represent the various filters that take a value, like =, &lt;=, &gt;= or ~=.
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
@@ -42,6 +42,10 @@ import org.apache.directory.api.ldap.model.filter.FilterEncoder;
 
     /**
      * Creates a new instance of AttributeValueAssertionFilter.
+     * 
+     * @param attribute The Attribute
+     * @param value The value
+     * @param operator The operator
      */
     private AttributeValueAssertionFilter( String attribute, String value, FilterOperator operator )
     {
@@ -52,7 +56,7 @@ import org.apache.directory.api.ldap.model.filter.FilterEncoder;
 
 
     /**
-     * Creates an Approximate Filter : ( <attribute> ~= <value> )
+     * Creates an Approximate Filter : ( &lt;attribute&gt; ~= &lt;value&gt; )
      *
      * @param attribute The AttributeType
      * @param value The Value
@@ -65,7 +69,7 @@ import org.apache.directory.api.ldap.model.filter.FilterEncoder;
 
 
     /**
-     * Creates an equal Filter : ( <attribute> = <value> )
+     * Creates an equal Filter : ( &lt;attribute&gt; = &lt;value&gt; )
      *
      * @param attribute The AttributeType
      * @param value The Value
@@ -78,7 +82,7 @@ import org.apache.directory.api.ldap.model.filter.FilterEncoder;
 
 
     /**
-     * Creates a Greater Than Or Equal Filter : ( <attribute> >= <value> )
+     * Creates a Greater Than Or Equal Filter : ( &lt;attribute&gt; &gt;= &lt;value&gt; )
      *
      * @param attribute The AttributeType
      * @param value The Value
@@ -91,7 +95,7 @@ import org.apache.directory.api.ldap.model.filter.FilterEncoder;
 
 
     /**
-     * Creates a Less Than Or Equal Filter : ( <attribute> <= <value> )
+     * Creates a Less Than Or Equal Filter : ( &lt;attribute&gt; &lt;= &lt;value&gt; )
      *
      * @param attribute The AttributeType
      * @param value The Value
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/FilterBuilder.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/FilterBuilder.java
index c73eb29..2a7398c 100755
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/FilterBuilder.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/FilterBuilder.java
@@ -54,6 +54,8 @@ public class FilterBuilder
     /**
      * A private constructor that creates a new instance of a FilterBuilder
      * containing a given filter.
+     * 
+     * @param filter The filter
      */
     /* No qualifier*/ FilterBuilder( Filter filter )
     {
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/FilterOperator.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/FilterOperator.java
index b8ae01c..e22a549 100644
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/FilterOperator.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/FilterOperator.java
@@ -24,12 +24,12 @@ package org.apache.directory.ldap.client.api.search;
 /**
  * The operators that can be used in a Filter :
  * <ul>
- * <li>AND: the '&' operator</li>
+ * <li>AND: the '&amp;' operator</li>
  * <li>OR: the '|' operator</li>
  * <li>NOT: the '!' operator</li>
  * <li>EQUAL: the '=' operator</li>
- * <li>LESS_THAN_OR_EQUAL: the '<=' operator</li>
- * <li>GREATER_THAN_OR_EQUAL: the '>=' operator</li>
+ * <li>LESS_THAN_OR_EQUAL: the '&lt;=' operator</li>
+ * <li>GREATER_THAN_OR_EQUAL: the '&gt;=' operator</li>
  * <li>PRESENT: the '=*' operator</li>
  * <li>APPROXIMATELY_EQUAL: the '~=' operator</li>
  * </ul>
@@ -54,6 +54,8 @@ package org.apache.directory.ldap.client.api.search;
 
     /**
      * Creates a new instance of FilterOperator.
+     * 
+     * @param operator The operator
      */
     FilterOperator( String operator )
     {
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/SetOfFiltersFilter.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/SetOfFiltersFilter.java
index 8a3b8c2..b869784 100755
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/SetOfFiltersFilter.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/SetOfFiltersFilter.java
@@ -42,6 +42,8 @@ import org.apache.directory.api.i18n.I18n;
 
     /**
      * Creates a new instance of SetOfFiltersFilter.
+     * 
+     * @param operator The operator
      */
     private SetOfFiltersFilter( FilterOperator operator )
     {
diff --git a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/SubstringFilter.java b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/SubstringFilter.java
index 8008238..4ad1df0 100644
--- a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/SubstringFilter.java
+++ b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/search/SubstringFilter.java
@@ -49,6 +49,11 @@ final class SubstringFilter extends AbstractFilter
 
     /**
      * A private constructor that builds a SubstringFilter 
+     * 
+     * @param attribute The Attribute
+     * @param initial The 'initial' part
+     * @param any The 'any' part
+     * @param end The 'end' part
      */
     private SubstringFilter( String attribute, String initial, String[] any, String end )
     {
diff --git a/ldap/codec/core/src/main/java/org/apache/directory/api/ldap/codec/api/LdapEncoder.java b/ldap/codec/core/src/main/java/org/apache/directory/api/ldap/codec/api/LdapEncoder.java
index d67c26b..5cf17e8 100644
--- a/ldap/codec/core/src/main/java/org/apache/directory/api/ldap/codec/api/LdapEncoder.java
+++ b/ldap/codec/core/src/main/java/org/apache/directory/api/ldap/codec/api/LdapEncoder.java
@@ -102,6 +102,7 @@ public class LdapEncoder
      * @param buffer The buffer that will contain the encoded control
      * @param control The control to encode
      * @return The control encoded in a byte[]
+     * @throws EncoderException If the encoding failed
      */
     public static ByteBuffer encodeControl( ByteBuffer buffer, Control control ) throws EncoderException
     {
diff --git a/ldap/extras/trigger/src/main/antlr/TriggerSpecification.g b/ldap/extras/trigger/src/main/antlr/TriggerSpecification.g
index d44aeee..09511f1 100644
--- a/ldap/extras/trigger/src/main/antlr/TriggerSpecification.g
+++ b/ldap/extras/trigger/src/main/antlr/TriggerSpecification.g
@@ -45,8 +45,6 @@ import org.slf4j.LoggerFactory;
 
 /**
  * The ANTLR generated TriggerSpecification parser.
- * 
- * @see http://docs.safehaus.org/display/APACHEDS/Grammar+for+Triggers
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
@@ -92,6 +90,8 @@ options
 
     /**
      * Sets the NameComponentNormalizer for this parser's dnParser.
+     *
+     * @param resolver The resolver to use
      */
     public void setNormalizerMappingResolver( NormalizerMappingResolver resolver )
     {
diff --git a/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/LdapOperation.java b/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/LdapOperation.java
index ec23c65..67e3ff9 100644
--- a/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/LdapOperation.java
+++ b/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/LdapOperation.java
@@ -56,7 +56,7 @@ public enum LdapOperation
      * 
      * Creates a new instance of LdapOperation.
      *
-     * @param name
+     * @param name The operation name
      */
     LdapOperation( String name )
     {
diff --git a/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/ReusableAntlrTriggerSpecificationParser.java b/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/ReusableAntlrTriggerSpecificationParser.java
index 72113ae..b50fee9 100644
--- a/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/ReusableAntlrTriggerSpecificationParser.java
+++ b/ldap/extras/trigger/src/main/java/org/apache/directory/api/ldap/trigger/ReusableAntlrTriggerSpecificationParser.java
@@ -41,6 +41,8 @@ class ReusableAntlrTriggerSpecificationParser extends AntlrTriggerSpecificationP
 {
     /**
      * Creates a ReusableAntlrTriggerSpecificationParser instance.
+     * 
+     * @param lexer The token stream to use
      */
     ReusableAntlrTriggerSpecificationParser( TokenStream lexer )
     {
diff --git a/ldap/extras/util/src/main/java/org/apache/directory/api/ldap/util/tree/DnNode.java b/ldap/extras/util/src/main/java/org/apache/directory/api/ldap/util/tree/DnNode.java
index 7bd6ee0..129cc72 100644
--- a/ldap/extras/util/src/main/java/org/apache/directory/api/ldap/util/tree/DnNode.java
+++ b/ldap/extras/util/src/main/java/org/apache/directory/api/ldap/util/tree/DnNode.java
@@ -139,6 +139,9 @@ public class DnNode<N>
     //-------------------------------------------------------------------------
     /**
      * Check that the Dn is not null
+     * 
+     * @param dn The Dn to check
+     * @throws LdapException If teh Dn is null or empty
      */
     private void checkDn( Dn dn ) throws LdapException
     {
@@ -153,6 +156,12 @@ public class DnNode<N>
 
     /**
      * Create a new DnNode, recursively creating all the intermediate nodes.
+     *
+     * @param dn The nod's Dn
+     * @param element The element to set
+     * @param nbRdns The number of RDNs in the Dn
+     * @param nbRdns The number of level to create
+     * @throws LdapException If the Dn is null or empty
      */
     private DnNode<N> createNode( Dn dn, N element, int nbRdns ) throws LdapException
     {
@@ -198,6 +207,8 @@ public class DnNode<N>
 
     /**
      * Store the given element into the node
+     * 
+     * @param element The element to set
      */
     private synchronized void setElement( N element )
     {
@@ -316,7 +327,10 @@ public class DnNode<N>
 
 
     /**
-     * recursively check if the node has a descendant having an element
+     * Recursively check if the node has a descendant having an element
+     * 
+     * @param node The node to start from
+     * @return <tt>true</tt> if the node has some descendant
      */
     private synchronized boolean hasDescendantElement( DnNode<N> node )
     {
@@ -379,7 +393,10 @@ public class DnNode<N>
 
 
     /**
-     * recursively get all the elements from nodes having an element
+     * Recursively get all the elements from nodes having an element
+     * 
+     * @param node The node to start with
+     * @param descendants The list of descendant to fulfill
      */
     private synchronized void getDescendantElements( DnNode<N> node, List<N> descendants )
     {
@@ -954,7 +971,8 @@ public class DnNode<N>
      * update the children's Dn based on the new parent Dn created
      * after a rename or move operation
      * 
-     * @param newParentDn
+     * @param newParentDn The new parent's Dn
+     * @throws LdapInvalidDnException The parent DN is invalid
      */
     private synchronized void updateAfterModDn( Dn newParentDn ) throws LdapInvalidDnException
     {
diff --git a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/AttributeClassLoader.java b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/AttributeClassLoader.java
index 6c89191..1c44abb 100644
--- a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/AttributeClassLoader.java
+++ b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/AttributeClassLoader.java
@@ -77,6 +77,10 @@ public class AttributeClassLoader extends ClassLoader
     
     /**
      * Read data from a jar, and write them into a byte[]
+     * 
+     * @param input The Stream we will load the Attribute from
+     * @return A byte[] containing the Attribute
+     * @throws IOException If the stream can't be read
      */
     private static byte[] getBytes( InputStream input ) throws IOException 
     {
@@ -98,6 +102,13 @@ public class AttributeClassLoader extends ClassLoader
     }
 
     
+    /**
+     * Load classes from a jar
+     *  
+     * @param jarBytes The jar bytes
+     * @return A map containing the read classes
+     * @throws IOException If the stream can't be read
+     */
     private Map<String, Class<?>> loadClasses( byte[] jarBytes ) throws IOException 
     {
         Map<String, Class<?>> map = new HashMap<>();
diff --git a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/JarLdifSchemaLoader.java b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/JarLdifSchemaLoader.java
index 47c7626..9a626b6 100644
--- a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/JarLdifSchemaLoader.java
+++ b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/JarLdifSchemaLoader.java
@@ -103,7 +103,8 @@ public class JarLdifSchemaLoader extends AbstractSchemaLoader
      * Scans for LDIF files just describing the various schema contained in
      * the schema repository.
      *
-     * @throws LdapException
+     * @throws LdapException If the schema can't be initialized
+     * @throws IOException If the file cannot be read
      */
     private void initializeSchemas() throws IOException, LdapException
     {
diff --git a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/LdifSchemaLoader.java b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/LdifSchemaLoader.java
index 9fe0b61..1501be3 100644
--- a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/LdifSchemaLoader.java
+++ b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/LdifSchemaLoader.java
@@ -116,7 +116,8 @@ public class LdifSchemaLoader extends AbstractSchemaLoader
      * Scans for LDIF files just describing the various schema contained in
      * the schema repository.
      *
-     * @throws LdapException
+     * @throws LdapException if the schemas can't be initialized
+     * @throws IOException If teh schema can't be read
      */
     private void initializeSchemas() throws LdapException, IOException
     {
diff --git a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/SchemaEntityFactory.java b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/SchemaEntityFactory.java
index 3f2e0ca..792c23f 100644
--- a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/SchemaEntityFactory.java
+++ b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/SchemaEntityFactory.java
@@ -112,6 +112,12 @@ public class SchemaEntityFactory implements EntityFactory
     /**
      * Get an OID from an entry. Handles the bad cases (null OID,
      * not a valid OID, ...)
+     * 
+     * @param entry The entry to process
+     * @param objectType The type of processed SchemaObject
+     * @param strict If we want a strict control of the OID
+     * @return The found OID
+     * @throws LdapInvalidAttributeValueException If the OID is not valid 
      */
     private String getOid( Entry entry, String objectType, boolean strict ) throws LdapInvalidAttributeValueException
     {
@@ -151,6 +157,11 @@ public class SchemaEntityFactory implements EntityFactory
     /**
      * Get an OID from an entry. Handles the bad cases (null OID,
      * not a valid OID, ...)
+     * 
+     * @param description The schemaObject description
+     * @param objectType The type of SchemaObject being processed
+     * @return The found OID
+     * @throws LdapInvalidAttributeValueException If the OID is invalid
      */
     private String getOid( SchemaObject description, String objectType ) throws LdapInvalidAttributeValueException
     {
@@ -187,6 +198,9 @@ public class SchemaEntityFactory implements EntityFactory
 
     /**
      * Check that the Entry is not null
+     * 
+     * @param entry The entry to check
+     * @param schemaEntity The message to log if the entry is invalid
      */
     private void checkEntry( Entry entry, String schemaEntity )
     {
@@ -206,6 +220,9 @@ public class SchemaEntityFactory implements EntityFactory
 
     /**
      * Check that the Description is not null
+     * 
+     * @param description description entry to check
+     * @param schemaEntity The message to log if the description is invalid
      */
     private void checkDescription( SchemaObject description, String schemaEntity )
     {
@@ -226,6 +243,10 @@ public class SchemaEntityFactory implements EntityFactory
     /**
      * Get the schema from its name. Return the Other reference if there
      * is no schema name. Throws a NPE if the schema is not loaded.
+     * 
+     * @param schemaName The schema name to fetch
+     * @param registries The registries where we get the schema from
+     * @return the found Schema
      */
     private Schema getSchema( String schemaName, Registries registries )
     {
@@ -301,7 +322,14 @@ public class SchemaEntityFactory implements EntityFactory
 
 
     /**
-     * Class load a syntaxChecker instance
+     * Class load a SyntaxChecker instance
+     * 
+     * @param schemaManager The SchemaManager
+     * @param oid The SyntaxChecker OID
+     * @param className The class name associated with the SyntaxChecker
+     * @param byteCode The SyntaxChecker class bytecode
+     * @return The loaded SyntaxChecker
+     * @throws LdapException If the SyntaxChecker cannot be loaded
      */
     private SyntaxChecker classLoadSyntaxChecker( SchemaManager schemaManager, String oid, String className,
         Attribute byteCode ) throws LdapException
@@ -478,6 +506,13 @@ public class SchemaEntityFactory implements EntityFactory
 
     /**
      * Class load a comparator instances
+     * 
+     * @param schemaManager The SchemaManager
+     * @param oid The comparator OID
+     * @param className The class name associated with the comparator
+     * @param byteCode The comparator class bytecode
+     * @return The loaded comparator
+     * @throws LdapException If the comparator cannot be loaded
      */
     private LdapComparator<?> classLoadComparator( SchemaManager schemaManager, String oid, String className,
         Attribute byteCode ) throws LdapException
@@ -701,6 +736,13 @@ public class SchemaEntityFactory implements EntityFactory
 
     /**
      * Class load a normalizer instances
+     * 
+     * @param schemaManager The SchemaManager
+     * @param oid The normalizer OID
+     * @param className The class name associated with the normalizer
+     * @param byteCode The normalizer class bytecode
+     * @return The loaded normalizer
+     * @throws LdapException If the normalizer cannot be loaded
      */
     private Normalizer classLoadNormalizer( SchemaManager schemaManager, String oid, String className,
         Attribute byteCode ) throws LdapException
@@ -990,6 +1032,9 @@ public class SchemaEntityFactory implements EntityFactory
 
     /**
      * Create a list of string from a multivalued attribute's values
+     * 
+     * @param attr The Attribute to read
+     * @return The list of values as Strings
      */
     private List<String> getStrings( Attribute attr )
     {
@@ -1232,7 +1277,11 @@ public class SchemaEntityFactory implements EntityFactory
 
     /**
      * Process the FQCN attribute
-     * @throws LdapInvalidAttributeValueException
+     * 
+     * @param entry The entry to read
+     * @param objectType The type of schema object
+     * @return The schema object FQCN
+     * @throws LdapInvalidAttributeValueException If the attribute does not contain a valid value
      */
     private String getFqcn( Entry entry, String objectType ) throws LdapInvalidAttributeValueException
     {
@@ -1257,6 +1306,10 @@ public class SchemaEntityFactory implements EntityFactory
 
     /**
      * Process the FQCN attribute
+     * 
+     * @param description The Schema Object description
+     * @param objectType The SchemaObject type
+     * @return The SchemaObject FQCN
      */
     private String getFqcn( LoadableSchemaObject description, String objectType )
     {
@@ -1281,6 +1334,10 @@ public class SchemaEntityFactory implements EntityFactory
 
     /**
      * Process the ByteCode attribute
+     * 
+     * @param description The SchemaObject description
+     * @param objectType The SchemaObject type
+     * @return The Attribute containing the byteCode
      */
     private Attribute getByteCode( LoadableSchemaObject description, String objectType )
     {
@@ -1305,7 +1362,10 @@ public class SchemaEntityFactory implements EntityFactory
 
 
     /**
-     * Return a String value, from teh given Valu, even if it's a binary value
+     * Return a String value, from the given Value, even if it's a binary value
+     * 
+     * @param attribute The Attribute to process
+     * @return The Attribute value as a String
      */
     private String getStringValue( Attribute attribute )
     {
@@ -1325,7 +1385,11 @@ public class SchemaEntityFactory implements EntityFactory
      *  - extensions
      *  - isReadOnly
      *  - isEnabled
-     * @throws org.apache.directory.api.ldap.model.exception.LdapInvalidAttributeValueException
+     *  
+     *  @param schemaObject The SchemaObject to set
+     *  @param entry The entry containing the SchemaObject properties
+     *  @param schema  the updated Schema 
+     * @throws LdapInvalidAttributeValueException If some of the properties are invalid
      */
     private void setSchemaObjectProperties( SchemaObject schemaObject, Entry entry, Schema schema )
         throws LdapInvalidAttributeValueException
@@ -1442,6 +1506,10 @@ public class SchemaEntityFactory implements EntityFactory
      *  - extensions
      *  - isReadOnly
      *  - isEnabled
+     *  
+     *  @param schemaObject The SchemaObject to set
+     *  @param description The SchemaObjetc description
+     *  @param schema  the updated Schema 
      */
     private void setSchemaObjectProperties( SchemaObject schemaObject, SchemaObject description, Schema schema )
     {
diff --git a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/SingleLdifSchemaLoader.java b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/SingleLdifSchemaLoader.java
index c5fda70..ed3adc6 100644
--- a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/SingleLdifSchemaLoader.java
+++ b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/loader/SingleLdifSchemaLoader.java
@@ -153,6 +153,10 @@ public class SingleLdifSchemaLoader extends AbstractSchemaLoader
 
     /**
      * Initialize the Schema object from a Single LDIF file
+     * 
+     * @param in The input stream to process
+     * @throws LdapException If the schemas can't be initialized
+     * @throws IOException If we had an issue processing the InputStream
      */
     private void initializeSchemas( InputStream in ) throws LdapException, IOException
     {
@@ -187,6 +191,9 @@ public class SingleLdifSchemaLoader extends AbstractSchemaLoader
 
     /**
      * Load all the schemaObjects
+     * 
+     * @param schemaName The schema name
+     * @param ldifEntry The entry to load
      */
     private void loadSchemaObject( String schemaName, LdifEntry ldifEntry )
     {
diff --git a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/manager/impl/DefaultSchemaManager.java b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/manager/impl/DefaultSchemaManager.java
index 679d910..9107ac5 100644
--- a/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/manager/impl/DefaultSchemaManager.java
+++ b/ldap/schema/data/src/main/java/org/apache/directory/api/ldap/schema/manager/impl/DefaultSchemaManager.java
@@ -232,6 +232,9 @@ public class DefaultSchemaManager implements SchemaManager
     /**
      * Clone the registries before doing any modification on it. Relax it
      * too so that we can update it.
+     * 
+     * @return The cloned Registries
+     * @throws LdapException If the Registries cannot be cloned
      */
     private Registries cloneRegistries() throws LdapException
     {
@@ -260,6 +263,10 @@ public class DefaultSchemaManager implements SchemaManager
 
     /**
      * Transform a String[] array of schema to a Schema[]
+     * 
+     * @param schemas The Schema names to process
+     * @return an array of Schema instance
+     * @throws LdapException If one of the Schema cannot be found
      */
     private Schema[] toArray( String... schemas ) throws LdapException
     {
@@ -315,6 +322,10 @@ public class DefaultSchemaManager implements SchemaManager
 
     /**
      * Delete all the schemaObjects for a given schema from the registries
+     * 
+     * @param schema The schema from which we want teh SchemaObjects to be deleted
+     * @param registries The Registries to process
+     * @throws LdapException If the SchemaObjects cannot be deleted
      */
     private void deleteSchemaObjects( Schema schema, Registries registries ) throws LdapException
     {
@@ -747,6 +758,11 @@ public class DefaultSchemaManager implements SchemaManager
      * Load the schema in the registries. We will load everything accordingly to the two flags :
      * - isRelaxed
      * - disabledAccepted
+     * 
+     * @param registries The Registries to process
+     * @param schema The schema to load in the Registries
+     * @return <tt>true</tt> if the schema has been loaded
+     * @throws LdapException If the schema cannot be loaded
      */
     private boolean load( Registries registries, Schema schema ) throws LdapException
     {
@@ -840,6 +856,11 @@ public class DefaultSchemaManager implements SchemaManager
      * Unload the schema from the registries. We will unload everything accordingly to the two flags :
      * - isRelaxed
      * - disabledAccepted
+     * 
+     * @param registries The Registries to process
+     * @param schema The schema to unload from the Registries
+     * @return <tt>true</tt> if the schema has been unloaded
+     * @throws LdapException If the schema cannot be unloaded
      */
     private boolean unload( Registries registries, Schema schema ) throws LdapException
     {
@@ -876,6 +897,11 @@ public class DefaultSchemaManager implements SchemaManager
 
     /**
      * Add all the Schema's AttributeTypes
+     * 
+     * @param schema The schema in which the AttributeTypes will be added
+     * @param registries The Registries to process
+     * @throws LdapException If the AttributeTypes cannot be added
+     * @throws IOException If the AttributeTypes cannot be loaded
      */
     private void addAttributeTypes( Schema schema, Registries registries ) throws LdapException, IOException
     {
@@ -895,6 +921,11 @@ public class DefaultSchemaManager implements SchemaManager
 
     /**
      * Add all the Schema's comparators
+     * 
+     * @param schema The schema in which the Comparators will be added
+     * @param registries The Registries to process
+     * @throws LdapException If the Comparators cannot be added
+     * @throws IOException If the Comparators cannot be loaded
      */
     private void addComparators( Schema schema, Registries registries ) throws LdapException, IOException
     {
@@ -940,6 +971,11 @@ public class DefaultSchemaManager implements SchemaManager
 
     /**
      * Add all the Schema's MatchingRules
+     * 
+     * @param schema The schema in which the MatchingRules will be added
+     * @param registries The Registries to process
+     * @throws LdapException If the MatchingRules cannot be added
+     * @throws IOException If the MatchingRules cannot be loaded
      */
     private void addMatchingRules( Schema schema, Registries registries ) throws LdapException, IOException
     {
@@ -989,6 +1025,11 @@ public class DefaultSchemaManager implements SchemaManager
 
     /**
      * Add all the Schema's Normalizers
+     * 
+     * @param schema The schema in which the Normalizers will be added
+     * @param registries The Registries to process
+     * @throws LdapException If the Normalizers cannot be added
+     * @throws IOException If the Normalizers cannot be loaded
      */
     private void addNormalizers( Schema schema, Registries registries ) throws LdapException, IOException
     {
@@ -1008,6 +1049,11 @@ public class DefaultSchemaManager implements SchemaManager
 
     /**
      * Add all the Schema's ObjectClasses
+     * 
+     * @param schema The schema in which the ObjectClasses will be added
+     * @param registries The Registries to process
+     * @throws LdapException If the ObjectClasses cannot be added
+     * @throws IOException If the ObjectClasses cannot be loaded
      */
     private void addObjectClasses( Schema schema, Registries registries ) throws LdapException, IOException
     {
@@ -1027,6 +1073,11 @@ public class DefaultSchemaManager implements SchemaManager
 
     /**
      * Add all the Schema's Syntaxes
+     * 
+     * @param schema The schema in which the Syntaxes will be added
+     * @param registries The Registries to process
+     * @throws LdapException If the Syntaxes cannot be added
+     * @throws IOException If the Syntaxes cannot be loaded
      */
     private void addSyntaxes( Schema schema, Registries registries ) throws LdapException, IOException
     {
@@ -1044,8 +1095,13 @@ public class DefaultSchemaManager implements SchemaManager
     }
 
 
-    /**Add
+    /**
      * Register all the Schema's SyntaxCheckers
+     * 
+     * @param schema The schema in which the SyntaxChecker will be added
+     * @param registries The Registries to process
+     * @throws LdapException If the SyntaxChecker cannot be added
+     * @throws IOException If the SyntaxChecker cannot be loaded
      */
     private void addSyntaxCheckers( Schema schema, Registries registries ) throws LdapException, IOException
     {
@@ -1869,6 +1925,9 @@ public class DefaultSchemaManager implements SchemaManager
 
     /**
      * Check that the given OID exists in the globalOidRegistry.
+     * 
+     * @param schemaObject The SchemaObject to check
+     * @return <tt>true</tt> if the OID exists
      */
     private boolean checkOidExist( SchemaObject schemaObject )
     {
@@ -1898,6 +1957,10 @@ public class DefaultSchemaManager implements SchemaManager
 
     /**
      * Get the inner SchemaObject if it's not a C/N/SC
+     * 
+     * @param schemaObject The SchemaObject to retreive
+     * @return The found SchemaObject
+     * @throws LdapException If the SchemaObject can't be found
      */
     private SchemaObject getSchemaObject( SchemaObject schemaObject ) throws LdapException
     {
@@ -1914,6 +1977,9 @@ public class DefaultSchemaManager implements SchemaManager
 
     /**
      * Retrieve the schema name for a specific SchemaObject, or return "other" if none is found.
+     * 
+     * @param schemaObject The schemaObject to read
+     * @return The schema name
      */
     private String getSchemaName( SchemaObject schemaObject )
     {
diff --git a/pom.xml b/pom.xml
index a07d7fc..68189da 100644
--- a/pom.xml
+++ b/pom.xml
@@ -33,7 +33,6 @@
   <version>2.0.0.AM2-SNAPSHOT</version>
 
   <properties>
-    <additionalparam>-Xdoclint:none</additionalparam>
     <projectName>ApacheDS-LDAP-API</projectName>
     <distMgmtSiteUrl>scpexe://people.apache.org/www/directory.apache.org/api/gen-docs/${project.version}/</distMgmtSiteUrl>
     <maven.compiler.source>1.8</maven.compiler.source>
@@ -303,7 +302,10 @@
         <configuration>
           <show>private</show>
           <nohelp>true</nohelp>
-          <doclint>none</doclint>
+          <additionalOptions>
+            <additionalOptions>-Xmaxwarns 1000</additionalOptions>
+            <additionalOptions>-Xmaxerrs 1000</additionalOptions>
+          </additionalOptions>
         </configuration>
       </plugin>