You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2013/08/22 16:56:57 UTC

svn commit: r1516474 - /logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java

Author: ggregory
Date: Thu Aug 22 14:56:57 2013
New Revision: 1516474

URL: http://svn.apache.org/r1516474
Log:
Consistently use "LOGGER" for the StatusLogger when used as a static final.

Modified:
    logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java

Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java?rev=1516474&r1=1516473&r2=1516474&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java (original)
+++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java Thu Aug 22 14:56:57 2013
@@ -73,7 +73,7 @@ import org.osgi.framework.wiring.BundleW
  */
 public class ResolverUtil {
     /** An instance of Log to use for logging in this class. */
-    private static final Logger LOG = StatusLogger.getLogger();
+    private static final Logger LOGGER = StatusLogger.getLogger();
 
     private static final String VFSZIP = "vfszip";
 
@@ -230,7 +230,7 @@ public class ResolverUtil {
         try {
             urls = loader.getResources(packageName);
         } catch (final IOException ioe) {
-            LOG.warn("Could not read package: " + packageName, ioe);
+            LOGGER.warn("Could not read package: " + packageName, ioe);
             return;
         }
 
@@ -250,7 +250,7 @@ public class ResolverUtil {
                     urlPath = urlPath.substring(0, urlPath.indexOf('!'));
                 }
 
-                LOG.info("Scanning for classes in [" + urlPath + "] matching criteria: " + test);
+                LOGGER.info("Scanning for classes in [" + urlPath + "] matching criteria: " + test);
                 // Check for a jar in a war in JBoss
                 if (VFSZIP.equals(url.getProtocol())) {
                     final String path = urlPath.substring(0, urlPath.length() - packageName.length() - 2);
@@ -273,7 +273,7 @@ public class ResolverUtil {
                     }
                 }
             } catch (final IOException ioe) {
-                LOG.warn("could not read entries", ioe);
+                LOGGER.warn("could not read entries", ioe);
             }
         }
     }
@@ -342,10 +342,10 @@ public class ResolverUtil {
             jarStream = new JarInputStream(new FileInputStream(jarFile));
             loadImplementationsInJar(test, parent, jarFile.getPath(), jarStream);
         } catch (final FileNotFoundException ex) {
-            LOG.error("Could not search jar file '" + jarFile + "' for classes matching criteria: " + test
+            LOGGER.error("Could not search jar file '" + jarFile + "' for classes matching criteria: " + test
                     + " file not found");
         } catch (final IOException ioe) {
-            LOG.error("Could not search jar file '" + jarFile + "' for classes matching criteria: " + test
+            LOGGER.error("Could not search jar file '" + jarFile + "' for classes matching criteria: " + test
                     + " due to an IOException", ioe);
         } finally {
             close(jarStream, jarFile);
@@ -361,7 +361,7 @@ public class ResolverUtil {
             try {
                 jarStream.close();
             } catch (IOException e) {
-                LOG.error("Error closing JAR file stream for {}", source, e);
+                LOGGER.error("Error closing JAR file stream for {}", source, e);
             }
         }
     }
@@ -388,7 +388,7 @@ public class ResolverUtil {
                 }
             }
         } catch (final IOException ioe) {
-            LOG.error("Could not search jar file '" + path + "' for classes matching criteria: " +
+            LOGGER.error("Could not search jar file '" + path + "' for classes matching criteria: " +
                 test + " due to an IOException", ioe);
         }
     }
@@ -405,8 +405,8 @@ public class ResolverUtil {
             final ClassLoader loader = getClassLoader();
             if (test.doesMatchClass()) {
                 final String externalName = fqn.substring(0, fqn.indexOf('.')).replace('/', '.');
-                if (LOG.isDebugEnabled()) {
-                    LOG.debug("Checking to see if class " + externalName + " matches criteria [" + test + "]");
+                if (LOGGER.isDebugEnabled()) {
+                    LOGGER.debug("Checking to see if class " + externalName + " matches criteria [" + test + "]");
                 }
 
                 final Class<?> type = loader.loadClass(externalName);
@@ -424,7 +424,7 @@ public class ResolverUtil {
                 }
             }
         } catch (final Throwable t) {
-            LOG.warn("Could not examine class '" + fqn + "' due to a " +
+            LOGGER.warn("Could not examine class '" + fqn + "' due to a " +
                 t.getClass().getName() + " with message: " + t.getMessage());
         }
     }