You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2007/02/22 13:24:04 UTC

svn commit: r510474 - /tomcat/tc6.0.x/trunk/java/org/apache/catalina/util/InstanceSupport.java

Author: remm
Date: Thu Feb 22 04:24:04 2007
New Revision: 510474

URL: http://svn.apache.org/viewvc?view=rev&rev=510474
Log:
- Remove sync + clone from InstanceSupport.

Modified:
    tomcat/tc6.0.x/trunk/java/org/apache/catalina/util/InstanceSupport.java

Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/util/InstanceSupport.java
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/util/InstanceSupport.java?view=diff&rev=510474&r1=510473&r2=510474
==============================================================================
--- tomcat/tc6.0.x/trunk/java/org/apache/catalina/util/InstanceSupport.java (original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/util/InstanceSupport.java Thu Feb 22 04:24:04 2007
@@ -121,10 +121,7 @@
             return;
 
         InstanceEvent event = new InstanceEvent(wrapper, filter, type);
-        InstanceListener interested[] = null;
-        synchronized (listeners) {
-            interested = (InstanceListener[]) listeners.clone();
-        }
+        InstanceListener interested[] = listeners;
         for (int i = 0; i < interested.length; i++)
             interested[i].instanceEvent(event);
 
@@ -148,10 +145,7 @@
 
         InstanceEvent event = new InstanceEvent(wrapper, filter, type,
                                                 exception);
-        InstanceListener interested[] = null;
-        synchronized (listeners) {
-            interested = (InstanceListener[]) listeners.clone();
-        }
+        InstanceListener interested[] = listeners;
         for (int i = 0; i < interested.length; i++)
             interested[i].instanceEvent(event);
 
@@ -177,10 +171,7 @@
 
         InstanceEvent event = new InstanceEvent(wrapper, filter, type,
                                                 request, response);
-        InstanceListener interested[] = null;
-        synchronized (listeners) {
-            interested = (InstanceListener[]) listeners.clone();
-        }
+        InstanceListener interested[] = listeners;
         for (int i = 0; i < interested.length; i++)
             interested[i].instanceEvent(event);
 
@@ -208,10 +199,7 @@
 
         InstanceEvent event = new InstanceEvent(wrapper, filter, type,
                                                 request, response, exception);
-        InstanceListener interested[] = null;
-        synchronized (listeners) {
-            interested = (InstanceListener[]) listeners.clone();
-        }
+        InstanceListener interested[] = listeners;
         for (int i = 0; i < interested.length; i++)
             interested[i].instanceEvent(event);
 
@@ -232,10 +220,7 @@
             return;
 
         InstanceEvent event = new InstanceEvent(wrapper, servlet, type);
-        InstanceListener interested[] = null;
-        synchronized (listeners) {
-            interested = (InstanceListener[]) listeners.clone();
-        }
+        InstanceListener interested[] = listeners;
         for (int i = 0; i < interested.length; i++)
             interested[i].instanceEvent(event);
 
@@ -259,10 +244,7 @@
 
         InstanceEvent event = new InstanceEvent(wrapper, servlet, type,
                                                 exception);
-        InstanceListener interested[] = null;
-        synchronized (listeners) {
-            interested = (InstanceListener[]) listeners.clone();
-        }
+        InstanceListener interested[] = listeners;
         for (int i = 0; i < interested.length; i++)
             interested[i].instanceEvent(event);
 
@@ -288,10 +270,7 @@
 
         InstanceEvent event = new InstanceEvent(wrapper, servlet, type,
                                                 request, response);
-        InstanceListener interested[] = null;
-        synchronized (listeners) {
-            interested = (InstanceListener[]) listeners.clone();
-        }
+        InstanceListener interested[] = listeners;
         for (int i = 0; i < interested.length; i++)
             interested[i].instanceEvent(event);
 
@@ -319,10 +298,7 @@
 
         InstanceEvent event = new InstanceEvent(wrapper, servlet, type,
                                                 request, response, exception);
-        InstanceListener interested[] = null;
-        synchronized (listeners) {
-            interested = (InstanceListener[]) listeners.clone();
-        }
+        InstanceListener interested[] = listeners;
         for (int i = 0; i < interested.length; i++)
             interested[i].instanceEvent(event);
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org