[jboss-cvs] JBossAS SVN: r69028 - projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/annotation/impl.

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

Author: wolfc
Date: 2008-01-16 08:03:32 -0500 (Wed, 16 Jan 2008)
New Revision: 69028

Modified:
   projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/annotation/impl/InterceptorsImpl.java
Log:
Removed assertion

Modified: projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/annotation/impl/InterceptorsImpl.java
===================================================================
--- projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/annotation/impl/InterceptorsImpl.java	2008-01-16 12:34:13 UTC (rev 69027)
+++ projects/ejb3/trunk/interceptors/src/main/java/org/jboss/ejb3/interceptors/annotation/impl/InterceptorsImpl.java	2008-01-16 13:03:32 UTC (rev 69028)
@@ -30,7 +30,11 @@
 import org.jboss.logging.Logger;
 
 /**
- * Comment
+ * A new implementation of Interceptors which allows for
+ * a bit more functionality.
+ * 
+ * Note: this one has an unsafe life-cycle. Modifications can
+ * be made after it is in use.
  *
  * @author <a href="mailto:carlo.dewolf at jboss.com">Carlo de Wolf</a>
  * @version $Revision: $
@@ -45,12 +49,12 @@
    {
       if(annotation == null)
          return false;
+      boolean result = false;
       for(Class<?> cls : annotation.value())
       {
-         boolean result = values.add(cls);
-         assert result;
+         result |= values.add(cls);
       }
-      return true;
+      return result;
    }
    
    public Class<? extends Annotation> annotationType()