[jboss-cvs] JBossAS SVN: r87092 - projects/aop/trunk/aop/src/main/java/org/jboss/aop.

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

Author: flavia.rainone at jboss.com
Date: 2009-04-09 12:56:45 -0400 (Thu, 09 Apr 2009)
New Revision: 87092

Modified:
   projects/aop/trunk/aop/src/main/java/org/jboss/aop/AspectManager.java
Log:
[JBAOP-717] The lock.lockRead() call has been included inside the synchronized block.

Modified: projects/aop/trunk/aop/src/main/java/org/jboss/aop/AspectManager.java
===================================================================
--- projects/aop/trunk/aop/src/main/java/org/jboss/aop/AspectManager.java	2009-04-09 16:10:45 UTC (rev 87091)
+++ projects/aop/trunk/aop/src/main/java/org/jboss/aop/AspectManager.java	2009-04-09 16:56:45 UTC (rev 87092)
@@ -1044,10 +1044,10 @@
          {
             return null;
          }
-         lock.lockRead();
-         try
-         {
-            synchronized(this){
+         synchronized(this){
+            lock.lockRead();
+            try
+            {
                if (weavingStrategy == null)
                {
                   if (TransformerCommon.isCompileTime())
@@ -1067,11 +1067,11 @@
       
                return weavingStrategy.translate(this, className, loader, classfileBuffer);
             }
+            finally
+            {
+               lock.unlockRead();
+            }
          }
-         finally
-         {
-            lock.unlockRead();
-         }
       }
       catch (Exception e)
       {