diff src/org/eclipse/jetty/server/Request.java @ 989:900e5b8ccd19

remove useless listeners
author Franklin Schmidt <fschmidt@gmail.com>
date Mon, 17 Oct 2016 05:39:54 -0600
parents 13beba0c127f
children 83cc6e05a58f
line wrap: on
line diff
--- a/src/org/eclipse/jetty/server/Request.java	Mon Oct 17 05:22:25 2016 -0600
+++ b/src/org/eclipse/jetty/server/Request.java	Mon Oct 17 05:39:54 2016 -0600
@@ -157,7 +157,6 @@
 	private String _readerEncoding;
 	private String _remoteAddr;
 	private String _remoteHost;
-	private Object _requestAttributeListeners;
 	private String _requestURI;
 	private String _scheme = URIUtil.HTTP;
 	private String _serverName;
@@ -180,15 +179,6 @@
 	}
 
 	/* ------------------------------------------------------------ */
-	public void addEventListener(final EventListener listener)
-	{
-		if (listener instanceof ServletRequestAttributeListener)
-			_requestAttributeListeners = LazyList.add(_requestAttributeListeners,listener);
-		if (listener instanceof AsyncListener)
-			throw new IllegalArgumentException(listener.getClass().toString());
-	}
-
-	/* ------------------------------------------------------------ */
 	/**
 	 * Extract Parameters from query string and/or form _content.
 	 */
@@ -1295,34 +1285,8 @@
 	 */
 	public void removeAttribute(String name)
 	{
-		Object old_value = _attributes == null?null:_attributes.getAttribute(name);
-
 		if (_attributes != null)
 			_attributes.removeAttribute(name);
-
-		if (old_value != null)
-		{
-			if (_requestAttributeListeners != null)
-			{
-				final ServletRequestAttributeEvent event = new ServletRequestAttributeEvent(_context,this,name,old_value);
-				final int size = LazyList.size(_requestAttributeListeners);
-				for (int i = 0; i < size; i++)
-				{
-					final EventListener listener = (ServletRequestAttributeListener)LazyList.get(_requestAttributeListeners,i);
-					if (listener instanceof ServletRequestAttributeListener)
-					{
-						final ServletRequestAttributeListener l = (ServletRequestAttributeListener)listener;
-						l.attributeRemoved(event);
-					}
-				}
-			}
-		}
-	}
-
-	/* ------------------------------------------------------------ */
-	public void removeEventListener(final EventListener listener)
-	{
-		_requestAttributeListeners = LazyList.remove(_requestAttributeListeners,listener);
 	}
 
 	/* ------------------------------------------------------------ */
@@ -1385,27 +1349,6 @@
 		if (_attributes == null)
 			_attributes = new AttributesMap();
 		_attributes.setAttribute(name,value);
-
-		if (_requestAttributeListeners != null)
-		{
-			final ServletRequestAttributeEvent event = new ServletRequestAttributeEvent(_context,this,name,old_value == null?value:old_value);
-			final int size = LazyList.size(_requestAttributeListeners);
-			for (int i = 0; i < size; i++)
-			{
-				final EventListener listener = (ServletRequestAttributeListener)LazyList.get(_requestAttributeListeners,i);
-				if (listener instanceof ServletRequestAttributeListener)
-				{
-					final ServletRequestAttributeListener l = (ServletRequestAttributeListener)listener;
-
-					if (old_value == null)
-						l.attributeAdded(event);
-					else if (value == null)
-						l.attributeRemoved(event);
-					else
-						l.attributeReplaced(event);
-				}
-			}
-		}
 	}
 
 	/* ------------------------------------------------------------ */