[jboss-cvs] JBoss Messaging SVN: r3599 - trunk/src/main/org/jboss/messaging/util.

[jboss-cvs-commits at lists.jboss.org]

Author: timfox
Date: 2008-01-21 06:28:38 -0500 (Mon, 21 Jan 2008)
New Revision: 3599

Modified:
   trunk/src/main/org/jboss/messaging/util/Logger.java
Log:
Fixed race


Modified: trunk/src/main/org/jboss/messaging/util/Logger.java
===================================================================
--- trunk/src/main/org/jboss/messaging/util/Logger.java	2008-01-21 07:41:52 UTC (rev 3598)
+++ trunk/src/main/org/jboss/messaging/util/Logger.java	2008-01-21 11:28:38 UTC (rev 3599)
@@ -7,9 +7,10 @@
  * 
  * A Logger
  * 
- * For now just delegates to log4j.
- *
+ * For now just delegates to org.jboss.util.Logger
  * 
+ * This class allows us to isolate all our logging dependencies in one place
+ * 
  * @author <a href="mailto:tim.fox at jboss.com">Tim Fox</a>
  *
  */
@@ -25,7 +26,12 @@
       {
          logger = new Logger(clazz);
          
-         loggers.putIfAbsent(clazz, logger);
+         Logger oldLogger = loggers.putIfAbsent(clazz, logger);
+         
+         if (oldLogger != null)
+         {
+            logger = oldLogger;
+         }
       }      
       
       return logger;