[jboss-cvs] JBossAS SVN: r96659 - projects/jboss-classpool/trunk/src/main/java/org/jboss/classpool/plugins/as5/jbosscl.

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

Author: flavia.rainone at jboss.com
Date: 2009-11-20 15:25:33 -0500 (Fri, 20 Nov 2009)
New Revision: 96659

Modified:
   projects/jboss-classpool/trunk/src/main/java/org/jboss/classpool/plugins/as5/jbosscl/JBossClDelegatingClassPool.java
Log:
[JBREFLECT-69] Removed callback invocation from inside JBossClDelegatingCP.close(). This is not the correct place to perform this clean up, which makes that call useless.

Modified: projects/jboss-classpool/trunk/src/main/java/org/jboss/classpool/plugins/as5/jbosscl/JBossClDelegatingClassPool.java
===================================================================
--- projects/jboss-classpool/trunk/src/main/java/org/jboss/classpool/plugins/as5/jbosscl/JBossClDelegatingClassPool.java	2009-11-20 20:24:37 UTC (rev 96658)
+++ projects/jboss-classpool/trunk/src/main/java/org/jboss/classpool/plugins/as5/jbosscl/JBossClDelegatingClassPool.java	2009-11-20 20:25:33 UTC (rev 96659)
@@ -52,6 +52,10 @@
          ScopedClassPoolRepository repository, Module module, RegisterModuleCallback callback)
    {
       super(domain, cl, parent, repository);
+      if (logger.isTraceEnabled())
+      {
+         logger.trace("ClassPool being created: " + cl + " - " + this);
+      }
       if (module == null)
       {
          throw new IllegalStateException("Null Module for loader " + cl);
@@ -90,6 +94,5 @@
    public void close()
    {
       super.close();
-      callback.removeModule(getModule());
    }
 }