[jboss-cvs] JBossAS SVN: r99965 - projects/aop/trunk/asintegration-core/src/main/java/org/jboss/aop/classpool.

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

Author: kabir.khan at jboss.com
Date: 2010-01-26 18:54:39 -0500 (Tue, 26 Jan 2010)
New Revision: 99965

Modified:
   projects/aop/trunk/asintegration-core/src/main/java/org/jboss/aop/classpool/DelegatingClassPool.java
Log:
[JBAOP-766] Better looking up in cache for classes higher up in the pool/domain hierarchy

Modified: projects/aop/trunk/asintegration-core/src/main/java/org/jboss/aop/classpool/DelegatingClassPool.java
===================================================================
--- projects/aop/trunk/asintegration-core/src/main/java/org/jboss/aop/classpool/DelegatingClassPool.java	2010-01-26 23:53:34 UTC (rev 99964)
+++ projects/aop/trunk/asintegration-core/src/main/java/org/jboss/aop/classpool/DelegatingClassPool.java	2010-01-26 23:54:39 UTC (rev 99965)
@@ -116,6 +116,11 @@
    @Override
    public CtClass getCached(String classname)
    {
+      //TODO Not 100 sure this is correct, but it seems to do the job where repeated calls to get() on a pool in a hierarchy returns a different instance of the class
+      CtClass clazz = super.getCachedLocally(classname);
+      if (clazz != null)
+         return clazz;  
+      
       if (isGeneratedClass(classname))
       {
          return null;