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

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

Author: alesj
Date: 2010-05-13 18:54:53 -0400 (Thu, 13 May 2010)
New Revision: 104783

Modified:
   projects/jboss-cl/trunk/classloading/src/main/java/org/jboss/classloading/spi/dependency/policy/WildcardClassLoaderPolicy.java
Log:
Conditional parent first check.

Modified: projects/jboss-cl/trunk/classloading/src/main/java/org/jboss/classloading/spi/dependency/policy/WildcardClassLoaderPolicy.java
===================================================================
--- projects/jboss-cl/trunk/classloading/src/main/java/org/jboss/classloading/spi/dependency/policy/WildcardClassLoaderPolicy.java	2010-05-13 22:51:24 UTC (rev 104782)
+++ projects/jboss-cl/trunk/classloading/src/main/java/org/jboss/classloading/spi/dependency/policy/WildcardClassLoaderPolicy.java	2010-05-13 22:54:53 UTC (rev 104783)
@@ -179,12 +179,11 @@
       if (md != null && module.canResolve(requirement))
       {
          boolean isAncestor = (domain != md); // not the same domain, so it must be ancestor
-         boolean isParentFirst = domain.isParentFirst();
          synchronized (this)
          {
             if (isAncestor)
             {
-               if (isParentFirst)
+               if (domain.isParentFirst())
                {
                   modules.add(0, module);
                   parentsBefore++;
@@ -212,8 +211,7 @@
             {
                Domain md = getDomain(module);
                boolean isAncestor = (domain != md);
-               boolean isParentFirst = domain.isParentFirst();
-               if (isAncestor && isParentFirst)
+               if (isAncestor && domain.isParentFirst())
                   parentsBefore--;
 
                reset();