Made DaemonThreadFactory THREAD_NUMBER a static field to ensure threads
created by different factories still have a unique name


Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/14f0d804
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/14f0d804
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/14f0d804

Branch: refs/heads/LOG4J2-89
Commit: 14f0d804c4703c243a809331745586f171aa343d
Parents: d3a748e
Author: rpopma <[email protected]>
Authored: Wed Nov 11 21:54:11 2015 +0900
Committer: Ralph Goers <[email protected]>
Committed: Fri Nov 20 17:39:22 2015 -0700

----------------------------------------------------------------------
 .../org/apache/logging/log4j/core/async/DaemonThreadFactory.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/14f0d804/log4j-core/src/main/java/org/apache/logging/log4j/core/async/DaemonThreadFactory.java
----------------------------------------------------------------------
diff --git 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/async/DaemonThreadFactory.java
 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/async/DaemonThreadFactory.java
index 1345d97..f853b32 100644
--- 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/async/DaemonThreadFactory.java
+++ 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/async/DaemonThreadFactory.java
@@ -26,8 +26,8 @@ import org.apache.logging.log4j.core.util.Log4jThread;
  */
 public class DaemonThreadFactory implements ThreadFactory {
 
+    private static final AtomicInteger THREAD_NUMBER = new AtomicInteger(1);
     private final ThreadGroup group;
-    private final AtomicInteger threadNumber = new AtomicInteger(1);
     private final String threadNamePrefix;
 
     public DaemonThreadFactory(final String threadNamePrefix) {
@@ -40,7 +40,7 @@ public class DaemonThreadFactory implements ThreadFactory {
     @Override
     public Thread newThread(final Runnable runnable) {
         final Thread thread = new Log4jThread(group, runnable, threadNamePrefix
-                + threadNumber.getAndIncrement(), 0);
+                + THREAD_NUMBER.getAndIncrement(), 0);
         if (!thread.isDaemon()) {
             thread.setDaemon(true);
         }

Reply via email to