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 2015/09/24 09:06:30 UTC

logging-log4j2 git commit: Use the logger config's toString() because the root logger's name is "" which is not helpful when debugging a config, showing "root" makes sense.

Repository: logging-log4j2
Updated Branches:
  refs/heads/master d84eef032 -> 3231ec660


Use the logger config's toString() because the root logger's name is ""
which is not helpful when debugging a config, showing "root" makes
sense.

Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/3231ec66
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/3231ec66
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/3231ec66

Branch: refs/heads/master
Commit: 3231ec6601f1ee6f235a10ce009cc05ad47b1c3b
Parents: d84eef0
Author: ggregory <gg...@apache.org>
Authored: Thu Sep 24 00:06:27 2015 -0700
Committer: ggregory <gg...@apache.org>
Committed: Thu Sep 24 00:06:27 2015 -0700

----------------------------------------------------------------------
 .../logging/log4j/core/config/AbstractConfiguration.java    | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/3231ec66/log4j-core/src/main/java/org/apache/logging/log4j/core/config/AbstractConfiguration.java
----------------------------------------------------------------------
diff --git a/log4j-core/src/main/java/org/apache/logging/log4j/core/config/AbstractConfiguration.java b/log4j-core/src/main/java/org/apache/logging/log4j/core/config/AbstractConfiguration.java
index ff3032a..516e776 100644
--- a/log4j-core/src/main/java/org/apache/logging/log4j/core/config/AbstractConfiguration.java
+++ b/log4j-core/src/main/java/org/apache/logging/log4j/core/config/AbstractConfiguration.java
@@ -419,13 +419,14 @@ public abstract class AbstractConfiguration extends AbstractFilterable implement
         }
 
         for (final Map.Entry<String, LoggerConfig> entry : loggers.entrySet()) {
-            final LoggerConfig l = entry.getValue();
-            for (final AppenderRef ref : l.getAppenderRefs()) {
+            final LoggerConfig loggerConfig = entry.getValue();
+            for (final AppenderRef ref : loggerConfig.getAppenderRefs()) {
                 final Appender app = appenders.get(ref.getRef());
                 if (app != null) {
-                    l.addAppender(app, ref.getLevel(), ref.getFilter());
+                    loggerConfig.addAppender(app, ref.getLevel(), ref.getFilter());
                 } else {
-                    LOGGER.error("Unable to locate appender \"{}\" for logger \"{}\"", ref.getRef(), l.getName());
+                    LOGGER.error("Unable to locate appender \"{}\" for logger config \"{}\"", ref.getRef(),
+                            loggerConfig);
                 }
             }