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 @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.
}
}