You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@felix.apache.org by pd...@apache.org on 2010/10/16 21:26:26 UTC

svn commit: r1023358 - /felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ComponentImpl.java

Author: pderop
Date: Sat Oct 16 19:26:25 2010
New Revision: 1023358

URL: http://svn.apache.org/viewvc?rev=1023358&view=rev
Log:
Fetch again the current state in the add(List) method, just before invoking calculateStateChanges method.

Modified:
    felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ComponentImpl.java

Modified: felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ComponentImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ComponentImpl.java?rev=1023358&r1=1023357&r2=1023358&view=diff
==============================================================================
--- felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ComponentImpl.java (original)
+++ felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ComponentImpl.java Sat Oct 16 19:26:25 2010
@@ -309,6 +309,7 @@ public class ComponentImpl implements Co
         }
 
         synchronized (m_dependencies) {
+            oldState = m_state;
             // starting the dependency above might have triggered another state change, so
             // we have to fetch the current state again
             newState = new State((List) m_dependencies.clone(), !oldState.isInactive(), m_isInstantiated, m_isBound);