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

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

Author: kabir.khan at jboss.com
Date: 2007-03-21 08:33:40 -0400 (Wed, 21 Mar 2007)
New Revision: 61540

Modified:
   projects/aop/trunk/aop/src/main/org/jboss/aop/advice/AspectDefinition.java
Log:
advisors should not be public

Modified: projects/aop/trunk/aop/src/main/org/jboss/aop/advice/AspectDefinition.java
===================================================================
--- projects/aop/trunk/aop/src/main/org/jboss/aop/advice/AspectDefinition.java	2007-03-21 12:18:05 UTC (rev 61539)
+++ projects/aop/trunk/aop/src/main/org/jboss/aop/advice/AspectDefinition.java	2007-03-21 12:33:40 UTC (rev 61540)
@@ -41,10 +41,9 @@
    protected Scope scope = Scope.PER_VM;
    protected AspectFactory factory;
    protected boolean deployed = true;
-   
-   /** @deprecated Should not access directly */
-   public Map advisors = new ConcurrentReaderHashMap();
 
+   private Map advisors = new ConcurrentReaderHashMap();
+
    /**
     * @param name
     * @param scope   defaults to PER_VM if null
@@ -135,12 +134,12 @@
       if (!(obj instanceof AspectDefinition)) return false;
       return name.equals(((AspectDefinition) obj).name);
    }
-   
+
    public synchronized void registerAdvisor(Advisor advisor)
    {
       advisors.put(advisor, Boolean.TRUE);
    }
-   
+
    public synchronized void unregisterAdvisor(Advisor advisor)
    {
       advisors.remove(advisor);