You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openwebbeans.apache.org by st...@apache.org on 2013/02/13 22:53:16 UTC

svn commit: r1445950 - in /openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans: component/InterceptorBean.java event/NotificationManager.java

Author: struberg
Date: Wed Feb 13 21:53:15 2013
New Revision: 1445950

URL: http://svn.apache.org/r1445950
Log:
OWB-344 remove more unused code

Modified:
    openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/component/InterceptorBean.java
    openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java

Modified: openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/component/InterceptorBean.java
URL: http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/component/InterceptorBean.java?rev=1445950&r1=1445949&r2=1445950&view=diff
==============================================================================
--- openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/component/InterceptorBean.java (original)
+++ openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/component/InterceptorBean.java Wed Feb 13 21:53:15 2013
@@ -24,7 +24,6 @@ import javax.enterprise.inject.spi.Inter
 import javax.enterprise.inject.spi.Interceptor;
 import javax.interceptor.InvocationContext;
 import java.lang.annotation.Annotation;
-import java.lang.reflect.Constructor;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
 import java.util.Collections;
@@ -41,10 +40,6 @@ import org.apache.webbeans.util.Exceptio
  */
 public abstract class InterceptorBean<T> extends InjectionTargetBean<T> implements Interceptor<T>
 {
-   /**
-     * Constructor of the Interceptor component
-     */
-    private Constructor<T> constructor;
 
     /**
      * The Methods to be called per interception type.
@@ -89,26 +84,6 @@ public abstract class InterceptorBean<T>
     }
 
     /**
-     * Get constructor.
-     *
-     * @return constructor
-     */
-    public Constructor<T> getConstructor()
-    {
-        return constructor;
-    }
-
-    /**
-     * Set constructor.
-     *
-     * @param constructor constructor instance
-     */
-    public void setConstructor(Constructor<T> constructor)
-    {
-        this.constructor = constructor;
-    }
-
-    /**
      * Interceptors are by default &#064;Dependent scoped.
      */
     @Override

Modified: openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java
URL: http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java?rev=1445950&r1=1445949&r2=1445950&view=diff
==============================================================================
--- openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java (original)
+++ openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java Wed Feb 13 21:53:15 2013
@@ -30,7 +30,6 @@ import java.util.Iterator;
 import java.util.Map;
 import java.util.Set;
 import java.util.concurrent.ConcurrentHashMap;
-import java.util.logging.Logger;
 
 import javax.enterprise.event.ObserverException;
 import javax.enterprise.event.Observes;
@@ -59,8 +58,6 @@ import org.apache.webbeans.util.WebBeans
 @SuppressWarnings("unchecked")
 public final class NotificationManager
 {
-    private final static Logger logger = WebBeansLoggerFacade.getLogger(NotificationManager.class);
-
     private final Map<Type, Set<ObserverMethod<?>>> observers = new ConcurrentHashMap<Type, Set<ObserverMethod<?>>>();
     private final WebBeansContext webBeansContext;
 
@@ -117,7 +114,7 @@ public final class NotificationManager
                         catch(NoSuchMethodException nsme)
                         {
                             // that's perfectly fine.
-                            // it means that we don't need to remove anything becasue the
+                            // it means that we don't need to remove anything because the
                             // observer method didn't get overridden.
                         }
                     }