[logback-dev] [GIT] Logback: the generic, reliable, fast and flexible logging framework. branch, master, updated. v_0.9.25-7-g9399594

added by portage for gitosis-gentoo git-noreply at pixie.qos.ch
Mon Oct 18 20:32:44 CEST 2010


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Logback: the generic, reliable, fast and flexible logging framework.".

The branch, master has been updated
       via  93995946b59b3ed1622d92c06b39c21942d98c58 (commit)
      from  c97834304048203d7d66de6ee174481f6991c293 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://git.qos.ch/gitweb/?p=logback.git;a=commit;h=93995946b59b3ed1622d92c06b39c21942d98c58
http://github.com/ceki/logback/commit/93995946b59b3ed1622d92c06b39c21942d98c58

commit 93995946b59b3ed1622d92c06b39c21942d98c58
Author: Ceki Gulcu <ceki at qos.ch>
Date:   Mon Oct 18 20:30:15 2010 +0200

    patial fix for LBCLASSIC-183. It helps if the ThreadLoacal class is
    not anonymous.

diff --git a/logback-core/src/main/java/ch/qos/logback/core/UnsynchronizedAppenderBase.java b/logback-core/src/main/java/ch/qos/logback/core/UnsynchronizedAppenderBase.java
index 64a0d20..1455f5e 100644
--- a/logback-core/src/main/java/ch/qos/logback/core/UnsynchronizedAppenderBase.java
+++ b/logback-core/src/main/java/ch/qos/logback/core/UnsynchronizedAppenderBase.java
@@ -40,11 +40,8 @@ abstract public class UnsynchronizedAppenderBase<E> extends ContextAwareBase imp
    * The guard prevents an appender from repeatedly calling its own doAppend
    * method.
    */
-  private ThreadLocal<Boolean> guard = new ThreadLocal<Boolean>() {
-    protected Boolean initialValue() {
-      return false;
-    }
-  };
+  private ThreadLocal<Boolean> guard = new ThreadLocal<Boolean>();
+
 
   /**
    * Appenders are named.
@@ -65,14 +62,14 @@ abstract public class UnsynchronizedAppenderBase<E> extends ContextAwareBase imp
   public void doAppend(E eventObject) {
     // WARNING: The guard check MUST be the first statement in the
     // doAppend() method.
-
+      
     // prevent re-entry.
-    if (guard.get()) {
+    if (Boolean.TRUE.equals(guard.get())) {
       return;
     }
 
     try {
-      guard.set(true);
+      guard.set(Boolean.TRUE);
 
       if (!this.started) {
         if (statusRepeatCount++ < ALLOWED_REPEATS) {
@@ -95,7 +92,7 @@ abstract public class UnsynchronizedAppenderBase<E> extends ContextAwareBase imp
         addError("Appender [" + name + "] failed to append.", e);
       }
     } finally {
-      guard.set(false);
+      guard.set(Boolean.FALSE);
     }
   }
 
@@ -136,7 +133,6 @@ abstract public class UnsynchronizedAppenderBase<E> extends ContextAwareBase imp
     return fai.getCopyOfAttachedFiltersList();
   }
 
-  
   public FilterReply getFilterChainDecision(E event) {
     return fai.getFilterChainDecision(event);
   }

-----------------------------------------------------------------------

Summary of changes:
 .../logback/core/UnsynchronizedAppenderBase.java   |   16 ++++++----------
 1 files changed, 6 insertions(+), 10 deletions(-)


hooks/post-receive
-- 
Logback: the generic, reliable, fast and flexible logging framework.


More information about the logback-dev mailing list