You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by an...@apache.org on 2014/07/15 20:47:22 UTC

svn commit: r1610805 - /tomee/tomee/trunk/tomee/tomee-juli/src/main/java/org/apache/juli/logging/LogFactory.java

Author: andygumbrecht
Date: Tue Jul 15 18:47:22 2014
New Revision: 1610805

URL: http://svn.apache.org/r1610805
Log:
Sync call

Modified:
    tomee/tomee/trunk/tomee/tomee-juli/src/main/java/org/apache/juli/logging/LogFactory.java

Modified: tomee/tomee/trunk/tomee/tomee-juli/src/main/java/org/apache/juli/logging/LogFactory.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/tomee/tomee-juli/src/main/java/org/apache/juli/logging/LogFactory.java?rev=1610805&r1=1610804&r2=1610805&view=diff
==============================================================================
--- tomee/tomee/trunk/tomee/tomee-juli/src/main/java/org/apache/juli/logging/LogFactory.java (original)
+++ tomee/tomee/trunk/tomee/tomee-juli/src/main/java/org/apache/juli/logging/LogFactory.java Tue Jul 15 18:47:22 2014
@@ -58,15 +58,12 @@ public /* abstract */ class LogFactory {
         this.logConfig=p;
     }
 
-    public Collection<String> getNames() {
+    public synchronized Collection<String> getNames() {
         return names;
     }
 
-    public Log getInstance(final String name)
-            throws LogConfigurationException {
-        synchronized (names) {
-            names.add(name);
-        }
+    public synchronized Log getInstance(final String name) throws LogConfigurationException {
+        names.add(name);
         return DirectJDKLog.getInstance(name);
     }