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

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

Author: wolfc
Date: 2008-05-12 02:38:47 -0400 (Mon, 12 May 2008)
New Revision: 73274

Modified:
   projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/aop/AbstractInterceptorFactory.java
   projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/aop/InjectInterceptorsFactory.java
Log:
AbstractInterceptorFactory.getName should never be overriden. It has no functional purpose in AOP.

Modified: projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/aop/AbstractInterceptorFactory.java
===================================================================
--- projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/aop/AbstractInterceptorFactory.java	2008-05-12 06:37:06 UTC (rev 73273)
+++ projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/aop/AbstractInterceptorFactory.java	2008-05-12 06:38:47 UTC (rev 73274)
@@ -30,7 +30,7 @@
  * An aspect factory that can do nothing.
  *
  * @author <a href="mailto:carlo.dewolf at jboss.com">Carlo de Wolf</a>
- * @version $Revision: $
+ * @version $Revision$
  */
 public abstract class AbstractInterceptorFactory implements AspectFactory
 {
@@ -59,7 +59,7 @@
       throw new IllegalStateException("Per vm scope not supported");
    }
    
-   public String getName()
+   public final String getName()
    {
       return getClass().getName();
    }

Modified: projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/aop/InjectInterceptorsFactory.java
===================================================================
--- projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/aop/InjectInterceptorsFactory.java	2008-05-12 06:37:06 UTC (rev 73273)
+++ projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/aop/InjectInterceptorsFactory.java	2008-05-12 06:38:47 UTC (rev 73274)
@@ -183,9 +183,4 @@
    {
       return advisor.hasAnnotation(method, ExcludeDefaultInterceptors.class) || advisor.resolveAnnotation(ExcludeDefaultInterceptors.class) != null;
    }
-   
-   public String getName()
-   {
-      return "InjectInterceptorsFactory";
-   }
 }