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 2012/12/17 18:10:52 UTC
svn commit: r1423022 - in /logging/log4j/log4j2/trunk:
core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
src/changes/changes.xml
Author: ggregory
Date: Mon Dec 17 17:10:51 2012
New Revision: 1423022
URL: http://svn.apache.org/viewvc?rev=1423022&view=rev
Log:
[LOG4J2-135] BaseConfiguration does not close the first appender.
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=1423022&r1=1423021&r2=1423022&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 Mon Dec 17 17:10:51 2012
@@ -132,7 +132,7 @@ public class BaseConfiguration extends A
}
// Stop the appenders in reverse order in case they still have activity.
final Appender[] array = appenders.values().toArray(new Appender[appenders.size()]);
- for (int i = array.length - 1; i > 0; --i) {
+ for (int i = array.length - 1; i >= 0; --i) {
array[i].stop();
}
stopFilter();
Modified: logging/log4j/log4j2/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/src/changes/changes.xml?rev=1423022&r1=1423021&r2=1423022&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/src/changes/changes.xml (original)
+++ logging/log4j/log4j2/trunk/src/changes/changes.xml Mon Dec 17 17:10:51 2012
@@ -23,6 +23,9 @@
<body>
<release version="2.0-beta4" date="TBD" description="Bug fixes and enhancements">
+ <action issue="LOG4J2-135" dev="ggregory" type="add">
+ BaseConfiguration does not close the first appender.
+ </action>
<action dev="rgoers" type="add">
Add MessageFormatMessage and FormattedMessage.
</action>