[jboss-cvs] JBoss Messaging SVN: r4024 - branches/Branch_Stable/tests/src/org/jboss/test/messaging/tools/container.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Wed Apr 9 12:05:05 EDT 2008


Author: clebert.suconic at jboss.com
Date: 2008-04-09 12:05:05 -0400 (Wed, 09 Apr 2008)
New Revision: 4024

Modified:
   branches/Branch_Stable/tests/src/org/jboss/test/messaging/tools/container/ClassLoaderJMXWrapper.java
Log:
Changing the ClassLoaderJMXWrapper to use the DelegateClassLoader as suggested by Adrian at the forum:
http://www.jboss.com/index.html?module=bb&op=viewtopic&t=133377&start=0

Modified: branches/Branch_Stable/tests/src/org/jboss/test/messaging/tools/container/ClassLoaderJMXWrapper.java
===================================================================
--- branches/Branch_Stable/tests/src/org/jboss/test/messaging/tools/container/ClassLoaderJMXWrapper.java	2008-04-09 12:53:31 UTC (rev 4023)
+++ branches/Branch_Stable/tests/src/org/jboss/test/messaging/tools/container/ClassLoaderJMXWrapper.java	2008-04-09 16:05:05 UTC (rev 4024)
@@ -21,9 +21,7 @@
 */
 package org.jboss.test.messaging.tools.container;
 
-import java.lang.reflect.Array;
-import java.net.URL;
-import java.net.URLClassLoader;
+import org.jboss.util.loading.DelegatingClassLoader;
 
 /**
  * We extend URLClassLoader just to prevent UnifiedLoaderRepository3 to generate spurious warning
@@ -35,7 +33,7 @@
  *
  * $Id$
  */
-public class ClassLoaderJMXWrapper extends URLClassLoader implements ClassLoaderJMXWrapperMBean
+public class ClassLoaderJMXWrapper extends DelegatingClassLoader implements ClassLoaderJMXWrapperMBean
 {
    // Constants -----------------------------------------------------
 
@@ -43,45 +41,13 @@
 
    // Attributes ----------------------------------------------------
 
-   private ClassLoader delegate;
-
    // Constructors --------------------------------------------------
 
    public ClassLoaderJMXWrapper(ClassLoader delegate)
    {
-      super(new URL[0]);
-      this.delegate = delegate;
+      super(delegate);
    }
 
-   // ClassLoaderJMXWrapperMBean implementation ---------------------
-
-   public Class loadClass(String name) throws ClassNotFoundException
-   {
-      if (name.endsWith("[]"))
-      {
-         name = name.substring(0, name.length() - 2);
-         
-         //The classloader of an array type is the classloader of it's element (if non primitive)
-         
-         Class cl = delegate.loadClass(name);
-         
-         Object arr = Array.newInstance(cl, 0);
-         
-         return arr.getClass();
-      }
-      else
-      {      
-         return delegate.loadClass(name);
-      }
-   }
-
-   // ClassLoader overrides -----------------------------------------
-
-   public URL getResource(String name)
-   {
-      return delegate.getResource(name);
-   }
-
    // Public --------------------------------------------------------
 
    // Package protected ---------------------------------------------




More information about the jboss-cvs-commits mailing list