felix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From djen...@apache.org
Subject svn commit: r1462798 - /felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java
Date Sat, 30 Mar 2013 16:58:24 GMT
Author: djencks
Date: Sat Mar 30 16:58:23 2013
New Revision: 1462798

URL: http://svn.apache.org/r1462798
Log:
FELIX-4000 avoid possibility of concurrentModificationException on m_dependencyManagers by
never changing it

Modified:
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java

Modified: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java?rev=1462798&r1=1462797&r2=1462798&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java
(original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/manager/AbstractComponentManager.java
Sat Mar 30 16:58:23 2013
@@ -877,7 +877,7 @@ public abstract class AbstractComponentM
     }
 
 
-    void clear()
+    synchronized void clear()
     {
         // for some testing, the activator may be null
         if ( m_activator != null )
@@ -885,8 +885,6 @@ public abstract class AbstractComponentM
             m_activator.unregisterComponentId( this );
             m_activator = null;
         }
-
-        m_dependencyManagers.clear();
     }
 
     /**



Mime
View raw message