svn commit: r1088740 - /commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/resource/classes/DiscoverClasses.java

[si...@apache.org]

Author: simonetripodi
Date: Mon Apr  4 19:18:16 2011
New Revision: 1088740

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

Modified:
    commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/resource/classes/DiscoverClasses.java

Modified: commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/resource/classes/DiscoverClasses.java
URL: http://svn.apache.org/viewvc/commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/resource/classes/DiscoverClasses.java?rev=1088740&r1=1088739&r2=1088740&view=diff
==============================================================================
--- commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/resource/classes/DiscoverClasses.java (original)
+++ commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/resource/classes/DiscoverClasses.java Mon Apr  4 19:18:16 2011
@@ -31,6 +31,8 @@ import org.apache.commons.logging.Log;
 /**
  * The findResources() method will check every loader.
  *
+ * @param <T> The SPI type
+ *
  * @author Richard A. Sitze
  * @author Craig R. McClanahan
  * @author Costin Manolache
@@ -41,28 +43,42 @@ public class DiscoverClasses<T>
     implements ResourceClassDiscover<T>
 {
     private static Log log = DiscoveryLogFactory.newLog(DiscoverClasses.class);
+
+    /**
+     * Sets the {@code Log} for this class.
+     *
+     * @param _log This class {@code Log}
+     */
     public static void setLog(Log _log) {
         log = _log;
     }
 
-    /** Construct a new resource discoverer
+    /**
+     * Construct a new resource discoverer
      */
     public DiscoverClasses() {
         super();
     }
 
-    /** Construct a new resource discoverer
+    /**
+     * Construct a new resource discoverer.
+     *
+     * @param classLoaders The class loaders holder
      */
     public DiscoverClasses(ClassLoaders classLoaders) {
         super(classLoaders);
     }
 
+    /**
+     * {@inheritDoc}
+     */
     @Override
     public ResourceClassIterator<T> findResourceClasses(final String className) {
         final String resourceName = className.replace('.','/') + ".class";
 
-        if (log.isDebugEnabled())
+        if (log.isDebugEnabled()) {
             log.debug("find: className='" + className + "'");
+        }
 
         return new ResourceClassIterator<T>() {
             private List<URL> history = new ArrayList<URL>();
@@ -96,11 +112,13 @@ public class DiscoverClasses<T>
 
                             return new ResourceClass<T>(className, url, loader);
                         }
-                        if (log.isDebugEnabled())
+                        if (log.isDebugEnabled()) {
                             log.debug("getNextClass: duplicate URL='" + url + "'");
+                        }
                     } else {
-                        if (log.isDebugEnabled())
+                        if (log.isDebugEnabled()) {
                             log.debug("getNextClass: loader " + loader + ": '" + resourceName + "' not found");
+                        }
                     }
                 }
                 return null;