svn commit: r1431163 - in /openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util: Logger.java URLs.java

[rmannibucau]

Author: rmannibucau
Date: Thu Jan 10 00:38:11 2013
New Revision: 1431163

URL: http://svn.apache.org/viewvc?rev=1431163&view=rev
Log:
no need to create a logger in URLs since that's only debug stuff and it loose 
time (depend on the logger impl) + avoiding to create multiple times a 
logstreamfactory

Modified:
    
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/Logger.java
    
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/URLs.java

Modified: 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/Logger.java
URL: 
http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/Logger.java?rev=1431163&r1=1431162&r2=1431163&view=diff
==============================================================================
--- 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/Logger.java
 (original)
+++ 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/Logger.java
 Thu Jan 10 00:38:11 2013
@@ -39,6 +39,9 @@ public class Logger {
     }
 
     public static synchronized void configure() {
+        if (logStreamFactory != null) {
+            return;
+        }
 
         //See if user factory has been specified
         String factoryName = 
SystemInstance.get().getOptions().get("openejb.log.factory", 
JuliLogStreamFactory.class.getName());

Modified: 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/URLs.java
URL: 
http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/URLs.java?rev=1431163&r1=1431162&r2=1431163&view=diff
==============================================================================
--- 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/URLs.java
 (original)
+++ 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/URLs.java
 Thu Jan 10 00:38:11 2013
@@ -32,8 +32,6 @@ import static org.apache.openejb.loader.
  * @version $Rev$ $Date$
  */
 public class URLs {
-    public static final Logger LOGGER = 
Logger.getInstance(LogCategory.OPENEJB, URLs.class.getPackage().getName());
-
     public static File toFile(final URL url) {
         if ("jar".equals(url.getProtocol())) {
             try {
@@ -94,14 +92,12 @@ public class URLs {
         urls = urls.excludeJavaHome();
         urls = urls.excludePaths(System.getProperty("sun.boot.class.path", 
""));
         urls = urls.exclude(".*/JavaVM.framework/.*");
-        if (LOGGER.isDebugEnabled()) LOGGER.debug("Culled {0} system urls from 
set", original.size() - urls.size());
         return urls;
     }
 
     public static UrlSet cullOpenEJBJars(final UrlSet original) throws 
IOException {
         UrlSet urls = new UrlSet(original.getUrls());
         urls = urls.exclude(".*openejb.*");
-        if (LOGGER.isDebugEnabled()) LOGGER.debug("Culled {0} OpenEJB urls 
from set", original.size() - urls.size());
         return urls;
     }