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 2012/11/27 10:09:28 UTC

svn commit: r1414027 - /directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java

Author: elecharny
Date: Tue Nov 27 09:09:27 2012
New Revision: 1414027

URL: http://svn.apache.org/viewvc?rev=1414027&view=rev
Log:
Made the LOG a protected member, to use it in inherited classes

Modified:
    directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java

Modified: directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java?rev=1414027&r1=1414026&r2=1414027&view=diff
==============================================================================
--- directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java (original)
+++ directory/apacheds/trunk/protocol-ldap/src/main/java/org/apache/directory/server/ldap/handlers/LdapRequestHandler.java Tue Nov 27 09:09:27 2012
@@ -56,7 +56,7 @@ import org.slf4j.LoggerFactory;
 public abstract class LdapRequestHandler<T extends Request> implements MessageHandler<T>
 {
     /** The logger for this class */
-    private static final Logger LOG = LoggerFactory.getLogger( LdapRequestHandler.class );
+    protected static final Logger LOG = LoggerFactory.getLogger( LdapRequestHandler.class );
 
     /** The reference on the Ldap server instance */
     protected LdapServer ldapServer;
@@ -120,8 +120,6 @@ public abstract class LdapRequestHandler
     /**
      *{@inheritDoc} 
      */
-    // False positive, we want to keep the comment
-    @SuppressWarnings("PMD.CollapsibleIfStatements")
     public final void handleMessage( IoSession session, T message ) throws Exception
     {
         LdapSession ldapSession = ldapServer.getLdapSessionManager().getLdapSession( session );