svn commit: r1089228 - /commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/DiscoveryLogFactory.java

[si...@apache.org]

Author: simonetripodi
Date: Tue Apr  5 21:17:46 2011
New Revision: 1089228

URL: http://svn.apache.org/viewvc?rev=1089228&view=rev
Log:
fixed checkstyle violations

Modified:
    commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/DiscoveryLogFactory.java

Modified: commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/DiscoveryLogFactory.java
URL: http://svn.apache.org/viewvc/commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/DiscoveryLogFactory.java?rev=1089228&r1=1089227&r2=1089228&view=diff
==============================================================================
--- commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/DiscoveryLogFactory.java (original)
+++ commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/log/DiscoveryLogFactory.java Tue Apr  5 21:17:46 2011
@@ -57,6 +57,10 @@ public class DiscoveryLogFactory {
     private static Log log = DiscoveryLogFactory._newLog(DiscoveryLogFactory.class);
 
     /**
+     * Creates a new {@code Log} instance for the input class.
+     *
+     * @param clazz The class the log has to be created for
+     * @return The input class logger
      */
     public static Log newLog(Class<?> clazz) {
         /**
@@ -69,7 +73,9 @@ public class DiscoveryLogFactory {
                                                               setLogParamClasses);
 
             if (setLog == null) {
-                String msg = "Internal Error: " + clazz.getName() + " required to implement 'public static void setLog(Log)'";
+                String msg = "Internal Error: "
+                    + clazz.getName()
+                    + " required to implement 'public static void setLog(Log)'";
                 log.fatal(msg);
                 throw new DiscoveryException(msg);
             }
@@ -79,14 +85,18 @@ public class DiscoveryLogFactory {
             throw new DiscoveryException(msg, se);
         }
 
-        if (log.isDebugEnabled())
+        if (log.isDebugEnabled()) {
             log.debug("Class meets requirements: " + clazz.getName());
+        }
 
         return _newLog(clazz);
     }
 
     /**
      * This method MUST not invoke any logging..
+     *
+     * @param clazz The class the log has to be created for
+     * @return The input class logger
      */
     public static Log _newLog(Class<?> clazz) {
         classRegistry.put(clazz, clazz);
@@ -96,12 +106,19 @@ public class DiscoveryLogFactory {
                : logFactory.getInstance(clazz.getName());
     }
 
+    /**
+     * Sets the {@code Log} for this class.
+     *
+     * @param _log This class {@code Log}
+     */
     public static void setLog(Log _log) {
         log = _log;
     }
 
     /**
      * Set logFactory, works ONLY on first call.
+     *
+     * @param factory The log factory
      */
     public static void setFactory(LogFactory factory) {
         if (logFactory == null) {