logging-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rgo...@apache.org
Subject svn commit: r1463400 - in /logging/log4j/log4j2/trunk: core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java src/changes/changes.xml
Date Tue, 02 Apr 2013 06:40:46 GMT
Author: rgoers
Date: Tue Apr  2 06:40:46 2013
New Revision: 1463400

URL: http://svn.apache.org/r1463400
Log:
LOG4J2-190 - BaseConfiguration addLoggerAppender saved appender using the Logger name.

Modified:
    logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
    logging/log4j/log4j2/trunk/src/changes/changes.xml

Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java?rev=1463400&r1=1463399&r2=1463400&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
(original)
+++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
Tue Apr  2 06:40:46 2013
@@ -236,7 +236,7 @@ public class BaseConfiguration extends A
     }
 
     protected PluginManager getPluginManager() {
-        //don't cache a pluginmanager instance - packages may be updated, requiring 
+        //don't cache a pluginmanager instance - packages may be updated, requiring
         // re-discovery of plugins
         PluginManager mgr = new PluginManager("Core");
         mgr.collectPlugins();
@@ -307,7 +307,7 @@ public class BaseConfiguration extends A
     public void setConfigurationMonitor(ConfigurationMonitor monitor) {
         this.monitor = monitor;
     }
-    
+
     public ConfigurationMonitor getConfigurationMonitor() {
         return monitor;
     }
@@ -315,7 +315,7 @@ public class BaseConfiguration extends A
     public void setAdvertiser(Advertiser advertiser) {
         this.advertiser = advertiser;
     }
-    
+
     public Advertiser getAdvertiser() {
         return advertiser;
     }
@@ -332,7 +332,7 @@ public class BaseConfiguration extends A
     public synchronized void addLoggerAppender(final org.apache.logging.log4j.core.Logger
logger,
                                                final Appender appender) {
         final String name = logger.getName();
-        appenders.putIfAbsent(name, appender);
+        appenders.putIfAbsent(appender.getName(), appender);
         final LoggerConfig lc = getLoggerConfig(name);
         if (lc.getName().equals(name)) {
             lc.addAppender(appender, null, null);

Modified: logging/log4j/log4j2/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/src/changes/changes.xml?rev=1463400&r1=1463399&r2=1463400&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/src/changes/changes.xml (original)
+++ logging/log4j/log4j2/trunk/src/changes/changes.xml Tue Apr  2 06:40:46 2013
@@ -23,6 +23,9 @@
 
   <body>
     <release version="2.0-beta5" date="@TBD@" description="Bug fixes and enhancements">
+      <action issue="LOG4J2-190" dev="rgoers" type="fix" due-to="Werner">
+        BaseConfiguration addLoggerAppender saved appender using the Logger name.
+      </action>
       <action issue="LOG4J2-160" dev="rgoers" type="update" due-to="Joanne Polsky">
         Move Throwable pattern converter options processing to ThrowableFormatOptions class.
       </action>



Mime
View raw message