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/11 17:24:23 UTC

svn commit: r1021383 - /felix/trunk/dependencymanager/runtime/src/main/java/org/apache/felix/dm/runtime/ServiceLifecycleHandler.java

Author: pderop
Date: Mon Oct 11 15:24:22 2010
New Revision: 1021383

URL: http://svn.apache.org/viewvc?rev=1021383&view=rev
Log:
Add extra dependencies in one shot, using the Service.add(List) signature, in order to calculate state changes atomically

Modified:
    felix/trunk/dependencymanager/runtime/src/main/java/org/apache/felix/dm/runtime/ServiceLifecycleHandler.java

Modified: felix/trunk/dependencymanager/runtime/src/main/java/org/apache/felix/dm/runtime/ServiceLifecycleHandler.java
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/runtime/src/main/java/org/apache/felix/dm/runtime/ServiceLifecycleHandler.java?rev=1021383&r1=1021382&r2=1021383&view=diff
==============================================================================
--- felix/trunk/dependencymanager/runtime/src/main/java/org/apache/felix/dm/runtime/ServiceLifecycleHandler.java (original)
+++ felix/trunk/dependencymanager/runtime/src/main/java/org/apache/felix/dm/runtime/ServiceLifecycleHandler.java Mon Oct 11 15:24:22 2010
@@ -212,8 +212,10 @@ public class ServiceLifecycleHandler
                                    dependency, m_srvMeta);
                 Dependency d = depBuilder.build(m_bundle, dm, true);
                 m_namedDeps.add(d);
-                service.add(d);
             }
+            
+            // Add all extra dependencies in one shot, in order to calculate state changes for all dependencies at a time.
+            service.add(m_namedDeps);
         }
     }