[logback-dev] [GIT] Logback: the generic, reliable, fast and flexible logging framework. branch master updated. v_1.0.1-29-ge1750a1

[Gitbot]

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  e1750a116b652ae912d9da35c22de07947762f05 (commit)
      from  608ed4c58717296b8182006e35aff52ca2ecb598 (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=e1750a116b652ae912d9da35c22de07947762f05
http://github.com/ceki/logback/commit/e1750a116b652ae912d9da35c22de07947762f05

commit e1750a116b652ae912d9da35c22de07947762f05
Author: Ceki Gulcu <ceki at qos.ch>
Date:   Thu Apr 26 00:14:12 2012 +0200

    slightly better logic in updateMaskIfNecessary

diff --git a/logback-classic/src/main/java/ch/qos/logback/classic/turbo/ReconfigureOnChangeFilter.java b/logback-classic/src/main/java/ch/qos/logback/classic/turbo/ReconfigureOnChangeFilter.java
index c812c2b..741bf88 100644
--- a/logback-classic/src/main/java/ch/qos/logback/classic/turbo/ReconfigureOnChangeFilter.java
+++ b/logback-classic/src/main/java/ch/qos/logback/classic/turbo/ReconfigureOnChangeFilter.java
@@ -84,7 +84,7 @@ public class ReconfigureOnChangeFilter extends TurboFilter {
   // IMPORTANT: This field can be updated by multiple threads. It follows that
   // its values may *not* be incremented sequentially. However, we don't care
   // about the actual value of the field except that from time to time the
-  // expression (invocationCounter++ & 0xF) == 0xF) should be true.
+  // expression (invocationCounter++ & mask) == mask) should be true.
   private long invocationCounter = 0;
 
   private volatile long mask = 0xF;
@@ -98,8 +98,9 @@ public class ReconfigureOnChangeFilter extends TurboFilter {
       return FilterReply.NEUTRAL;
     }
 
-    // for performance reasons, skip any type of computation (MASK-1) times out of MASK.
-    // Only once every MASK calls is the check for elapsed time is performed
+    // for performance reasons, skip change detection (MASK-1) times out of MASK.
+    // Only once every MASK calls is change detection code executed
+    // Note that MASK is a variable itself.
     if (((invocationCounter++) & mask) != mask) {
       return FilterReply.NEUTRAL;
     }
@@ -124,15 +125,16 @@ public class ReconfigureOnChangeFilter extends TurboFilter {
   // for CPU intensive applications with 200 or more threads MASK values in the order of 0xFFFF is appropriate
   private static final int MAX_MASK = 0xFFFF;
 
+  // update the mask so as to execute change detection code about once every 1s.
   private void updateMaskIfNecessary(long now) {
-    if (now - lastMaskCheck < MILLIS_IN_ONE_SECOND) {
-      if (mask < MAX_MASK) {
+    // if less than 1s elapsed since last maskUpdate double the mask size and add 1
+    final long timeElapsedSinceLastCheck = now - lastMaskCheck;
+    lastMaskCheck = now;
+    if (timeElapsedSinceLastCheck < MILLIS_IN_ONE_SECOND && (mask < MAX_MASK)) {
         mask = (mask << 1) | 1;
-      }
-    } else {
+    } else if (timeElapsedSinceLastCheck > 8*MILLIS_IN_ONE_SECOND) {
       mask = mask >>> 2;
     }
-    lastMaskCheck = now;
   }
 
   // by detaching reconfiguration to a new thread, we release the various

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

Summary of changes:
 .../classic/turbo/ReconfigureOnChangeFilter.java   |   18 ++++++++++--------
 1 files changed, 10 insertions(+), 8 deletions(-)


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