[jboss-cvs] JBossAS SVN: r76096 - projects/microcontainer/trunk/aop-mc-int/src/main/org/jboss/aop/microcontainer/beans.

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

Author: kabir.khan at jboss.com
Date: 2008-07-22 07:18:20 -0400 (Tue, 22 Jul 2008)
New Revision: 76096

Modified:
   projects/microcontainer/trunk/aop-mc-int/src/main/org/jboss/aop/microcontainer/beans/Aspect.java
Log:
Kill unused code for 'plain' aspects

Modified: projects/microcontainer/trunk/aop-mc-int/src/main/org/jboss/aop/microcontainer/beans/Aspect.java
===================================================================
--- projects/microcontainer/trunk/aop-mc-int/src/main/org/jboss/aop/microcontainer/beans/Aspect.java	2008-07-22 11:05:34 UTC (rev 76095)
+++ projects/microcontainer/trunk/aop-mc-int/src/main/org/jboss/aop/microcontainer/beans/Aspect.java	2008-07-22 11:18:20 UTC (rev 76096)
@@ -263,8 +263,7 @@
          }
          else
          {
-            //Not sure when this would get called???
-            definition = getAspectDefinitionPlainAspectFactory();
+            throw new IllegalStateException("Unknown type of managed aspects");
          }
          addDefinitionToManager();
       }
@@ -305,13 +304,6 @@
       return new ManagedAspectDefinition(aspectDefName, scope, factory, false);
    }
 
-   protected ManagedAspectDefinition getAspectDefinitionPlainAspectFactory()
-   {
-      AspectFactory factory = this.factory ?  
-            new AspectFactoryDelegator(myname, null) : new GenericAspectFactory(myname, element);
-      return new ManagedAspectDefinition(aspectDefName, scope, factory);
-   }
-
    protected void addDefinitionToManager()
    {
       manager.addAspectDefinition(definition);