You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by bd...@apache.org on 2008/11/25 18:11:25 UTC
svn commit: r720542 -
/incubator/sling/trunk/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/SlingServletResolver.java
Author: bdelacretaz
Date: Tue Nov 25 09:11:25 2008
New Revision: 720542
URL: http://svn.apache.org/viewvc?rev=720542&view=rev
Log:
SLING-742 - SlingServletResolver.deactivate() synchronizes too much
Modified:
incubator/sling/trunk/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/SlingServletResolver.java
Modified: incubator/sling/trunk/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/SlingServletResolver.java
URL: http://svn.apache.org/viewvc/incubator/sling/trunk/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/SlingServletResolver.java?rev=720542&r1=720541&r2=720542&view=diff
==============================================================================
--- incubator/sling/trunk/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/SlingServletResolver.java (original)
+++ incubator/sling/trunk/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/SlingServletResolver.java Tue Nov 25 09:11:25 2008
@@ -529,15 +529,16 @@
protected void deactivate(ComponentContext context) {
- // destroy all active servlets
+ // Copy the list of servlets first, to minimize the need for synchronization
Collection<ServiceReference> refs;
synchronized (this) {
refs = new ArrayList<ServiceReference>(servletsByReference.keySet());
- // destroy all servlets
- destroyAllServlets(refs);
- this.context = null;
- this.servletResourceProviderFactory = null;
}
+
+ // destroy all servlets
+ destroyAllServlets(refs);
+ this.context = null;
+ this.servletResourceProviderFactory = null;
}
protected synchronized void bindServlet(ServiceReference reference) {