[jboss-cvs] JBossAS SVN: r60519 - branches/Branch_AOP_1_5/aop/src/main/org/jboss/aop.

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

Author: flavia.rainone at jboss.com
Date: 2007-02-13 17:00:39 -0500 (Tue, 13 Feb 2007)
New Revision: 60519

Modified:
   branches/Branch_AOP_1_5/aop/src/main/org/jboss/aop/AspectManager.java
Log:
[JBAOP-360] Synchronization of configuration methods (add, remove binding) with class advisor initialization public methods (getAdvisor, initializeAdvisor)

Modified: branches/Branch_AOP_1_5/aop/src/main/org/jboss/aop/AspectManager.java
===================================================================
--- branches/Branch_AOP_1_5/aop/src/main/org/jboss/aop/AspectManager.java	2007-02-13 19:54:27 UTC (rev 60518)
+++ branches/Branch_AOP_1_5/aop/src/main/org/jboss/aop/AspectManager.java	2007-02-13 22:00:39 UTC (rev 60519)
@@ -566,7 +566,7 @@
     * @param clazz
     * @return
     */
-   public ClassAdvisor getAdvisor(Class clazz)
+   public synchronized ClassAdvisor getAdvisor(Class clazz)
    {
       ClassAdvisor advisor = null;
       // See if one already exists
@@ -580,7 +580,7 @@
       return advisor;
    }
 
-   public void initialiseClassAdvisor(Class clazz, ClassAdvisor advisor)
+   public synchronized void initialiseClassAdvisor(Class clazz, ClassAdvisor advisor)
    {
       synchronized (advisors)
       {
@@ -1295,7 +1295,7 @@
    /**
     * Remove an interceptor pointcut with a given name
     */
-   public void removeBinding(String name)
+   public synchronized void removeBinding(String name)
    {
       AdviceBinding binding = internalRemoveBinding(name);
       if (binding != null)
@@ -1305,7 +1305,7 @@
       }
    }
 
-   public void removeBindings(ArrayList binds)
+   public synchronized void removeBindings(ArrayList binds)
    {
       clearUnregisteredClassLoaders();