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 2022/01/13 16:06:07 UTC

[logging-log4j2] 02/06: Sort members and use final.

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch release-2.x
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git

commit 0ca8f054ffda7db6c44975428242502669f3cdf8
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Thu Jan 13 10:38:50 2022 -0500

    Sort members and use final.
---
 .../log4j/helpers/AppenderAttachableImpl.java      | 44 +++++++++++-----------
 1 file changed, 22 insertions(+), 22 deletions(-)

diff --git a/log4j-1.2-api/src/main/java/org/apache/log4j/helpers/AppenderAttachableImpl.java b/log4j-1.2-api/src/main/java/org/apache/log4j/helpers/AppenderAttachableImpl.java
index b9e9cdc..455c0ce 100644
--- a/log4j-1.2-api/src/main/java/org/apache/log4j/helpers/AppenderAttachableImpl.java
+++ b/log4j-1.2-api/src/main/java/org/apache/log4j/helpers/AppenderAttachableImpl.java
@@ -41,6 +41,28 @@ public class AppenderAttachableImpl implements AppenderAttachable {
         }
     }
 
+    /**
+     * Calls the <code>doAppend</code> method on all attached appenders.
+     * 
+     * @param event The event to log.
+     * @return The number of appenders.
+     */
+    public int appendLoopOnAppenders(LoggingEvent event) {
+        for (Appender appender : appenders.values()) {
+            appender.doAppend(event);
+        }
+        return appenders.size();
+    }
+
+    /**
+     * Closes all appenders.
+     */
+    public void close() {
+        for (Appender appender : appenders.values()) {
+            appender.close();
+        }
+    }
+
     @Override
     public Enumeration<Appender> getAllAppenders() {
         return Collections.enumeration(appenders.values());
@@ -71,26 +93,4 @@ public class AppenderAttachableImpl implements AppenderAttachable {
     public void removeAppender(String name) {
         appenders.remove(name);
     }
-
-    /**
-     * Calls the <code>doAppend</code> method on all attached appenders.
-     * 
-     * @param event The event to log.
-     * @return The number of appenders.
-     */
-    public int appendLoopOnAppenders(LoggingEvent event) {
-        for (Appender appender : appenders.values()) {
-            appender.doAppend(event);
-        }
-        return appenders.size();
-    }
-
-    /**
-     * Closes all appenders.
-     */
-    public void close() {
-        for (Appender appender : appenders.values()) {
-            appender.close();
-        }
-    }
 }