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

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

Author: flavia.rainone at jboss.com
Date: 2007-02-13 17:05:59 -0500 (Tue, 13 Feb 2007)
New Revision: 60522

Modified:
   projects/aop/trunk/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: projects/aop/trunk/aop/src/main/org/jboss/aop/AspectManager.java
===================================================================
--- projects/aop/trunk/aop/src/main/org/jboss/aop/AspectManager.java	2007-02-13 22:03:02 UTC (rev 60521)
+++ projects/aop/trunk/aop/src/main/org/jboss/aop/AspectManager.java	2007-02-13 22:05:59 UTC (rev 60522)
@@ -575,7 +575,7 @@
     * @param clazz
     * @return
     */
-   public ClassAdvisor getAdvisor(Class clazz)
+   public synchronized ClassAdvisor getAdvisor(Class clazz)
    {
       ClassAdvisor advisor = null;
       // See if one already exists
@@ -589,7 +589,7 @@
       return advisor;
    }
 
-   public void initialiseClassAdvisor(Class clazz, ClassAdvisor advisor)
+   public synchronized void initialiseClassAdvisor(Class clazz, ClassAdvisor advisor)
    {
       synchronized (advisors)
       {
@@ -1260,7 +1260,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)
@@ -1270,7 +1270,7 @@
       }
    }
 
-   public void removeBindings(ArrayList binds)
+   public synchronized void removeBindings(ArrayList binds)
    {
       clearUnregisteredClassLoaders();