[jboss-cvs] JBossAS SVN: r100260 - projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base.

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

Author: alesj
Date: 2010-02-02 10:15:26 -0500 (Tue, 02 Feb 2010)
New Revision: 100260

Modified:
   projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoader.java
   projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoaderPolicy.java
Log:
[JBCL-141]; check debug before creating long string.

Modified: projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoader.java
===================================================================
--- projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoader.java	2010-02-02 15:06:00 UTC (rev 100259)
+++ projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoader.java	2010-02-02 15:15:26 UTC (rev 100260)
@@ -111,7 +111,9 @@
       if (basePolicy.isBlackListable())
          blackList = new ConcurrentHashMap<String, String>();
 
-      log.debug("Created " + this + " with policy " + policy.toLongString());
+      if (log.isDebugEnabled())
+         log.debug("Created " + this + " with policy " + policy.toLongString());
+      //log.debugf("Created %1s with policy %2s", this, policy);
    }
 
    public ObjectName getObjectName()

Modified: projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoaderPolicy.java
===================================================================
--- projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoaderPolicy.java	2010-02-02 15:06:00 UTC (rev 100259)
+++ projects/jboss-cl/branches/Branch_2_0/classloader/src/main/java/org/jboss/classloader/spi/base/BaseClassLoaderPolicy.java	2010-02-02 15:15:26 UTC (rev 100260)
@@ -24,9 +24,7 @@
 import java.security.AccessControlContext;
 import java.security.AccessController;
 import java.security.ProtectionDomain;
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.List;
+import java.util.*;
 import javax.management.ObjectName;
 
 import org.jboss.classloader.spi.ClassLoaderDomain;
@@ -45,7 +43,7 @@
  * @author <a href="ales.justin at jboss.com">Ales Justin</a>
  * @version $Revision: 1.1 $
  */
-public abstract class BaseClassLoaderPolicy
+public abstract class BaseClassLoaderPolicy implements Formattable
 {
    /** The log */
    private static final Logger log = Logger.getLogger(BaseClassLoaderPolicy.class);
@@ -239,6 +237,12 @@
       return builder.toString();
    }
    
+   public void formatTo(Formatter formatter, int flags, int width, int precision)
+   {
+      // TODO perhaps decide which toXString to use depending on args?
+      formatter.format(toLongString());
+   }
+
    /**
     * Get the classloader domain name
     *