[jboss-cvs] JBossAS SVN: r99963 - projects/aop/branches/Branch_2_1/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:52:03 -0500 (Tue, 26 Jan 2010)
New Revision: 99963

Modified:
   projects/aop/branches/Branch_2_1/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/branches/Branch_2_1/asintegration-core/src/main/java/org/jboss/aop/classpool/DelegatingClassPool.java
===================================================================
--- projects/aop/branches/Branch_2_1/asintegration-core/src/main/java/org/jboss/aop/classpool/DelegatingClassPool.java	2010-01-26 23:07:56 UTC (rev 99962)
+++ projects/aop/branches/Branch_2_1/asintegration-core/src/main/java/org/jboss/aop/classpool/DelegatingClassPool.java	2010-01-26 23:52:03 UTC (rev 99963)
@@ -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;