From commits-return-1816-apmail-logging-commits-archive=logging.apache.org@logging.apache.org Mon Dec 17 17:11:18 2012 Return-Path: X-Original-To: apmail-logging-commits-archive@minotaur.apache.org Delivered-To: apmail-logging-commits-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 7DD21D41B for ; Mon, 17 Dec 2012 17:11:18 +0000 (UTC) Received: (qmail 25984 invoked by uid 500); 17 Dec 2012 17:11:18 -0000 Delivered-To: apmail-logging-commits-archive@logging.apache.org Received: (qmail 25958 invoked by uid 500); 17 Dec 2012 17:11:18 -0000 Mailing-List: contact commits-help@logging.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@logging.apache.org Delivered-To: mailing list commits@logging.apache.org Received: (qmail 25908 invoked by uid 99); 17 Dec 2012 17:11:15 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 17 Dec 2012 17:11:15 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=5.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 17 Dec 2012 17:11:13 +0000 Received: from eris.apache.org (localhost [127.0.0.1]) by eris.apache.org (Postfix) with ESMTP id BA44F2388AA9; Mon, 17 Dec 2012 17:10:52 +0000 (UTC) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r1423022 - in /logging/log4j/log4j2/trunk: core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java src/changes/changes.xml Date: Mon, 17 Dec 2012 17:10:52 -0000 To: commits@logging.apache.org From: ggregory@apache.org X-Mailer: svnmailer-1.0.8-patched Message-Id: <20121217171052.BA44F2388AA9@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org 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 @@ + + BaseConfiguration does not close the first appender. + Add MessageFormatMessage and FormattedMessage.