[jboss-cvs] JBossAS SVN: r86811 - projects/jboss-cl/trunk/classloading/src/main/java/org/jboss/classloading/spi/dependency.

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

Author: alesj
Date: 2009-04-05 07:30:00 -0400 (Sun, 05 Apr 2009)
New Revision: 86811

Modified:
   projects/jboss-cl/trunk/classloading/src/main/java/org/jboss/classloading/spi/dependency/Domain.java
Log:
[JBCL-97]; make remove Module asymetric to add.

Modified: projects/jboss-cl/trunk/classloading/src/main/java/org/jboss/classloading/spi/dependency/Domain.java
===================================================================
--- projects/jboss-cl/trunk/classloading/src/main/java/org/jboss/classloading/spi/dependency/Domain.java	2009-04-05 10:24:38 UTC (rev 86810)
+++ projects/jboss-cl/trunk/classloading/src/main/java/org/jboss/classloading/spi/dependency/Domain.java	2009-04-05 11:30:00 UTC (rev 86811)
@@ -34,6 +34,7 @@
  * Domain.
  * 
  * @author <a href="adrian at jboss.org">Adrian Brock</a>
+ * @author <a href="ales.justin at jboss.org">Ales Justin</a>
  * @version $Revision: 1.1 $
  */
 public class Domain
@@ -175,15 +176,15 @@
          throw new IllegalArgumentException("Null module");
 
       log.debug(this + " remove module " + module);
-      
-      modulesByName.remove(module.getContextName());
-      modules.remove(module);
-      module.setDomain(null);
-      module.removeDependencies();
-      
+
       ClassLoadingSpace space = module.getClassLoadingSpace();
       if (space != null)
          space.split(module);
+
+      module.removeDependencies();
+      modules.remove(module);
+      modulesByName.remove(module.getContextName());
+      module.setDomain(null);
    }
 
    /**