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 2011/03/29 22:21:42 UTC

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

Author: pderop
Date: Tue Mar 29 20:21:42 2011
New Revision: 1086721

URL: http://svn.apache.org/viewvc?rev=1086721&view=rev
Log:
Don't add any extra dependencies, if the list of extra dependencies is empty

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=1086721&r1=1086720&r2=1086721&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 Tue Mar 29 20:21:42 2011
@@ -205,7 +205,11 @@ public class ServiceLifecycleHandler
             }
             
             // Add all extra dependencies in one shot, in order to calculate state changes for all dependencies at a time.
-            service.add(m_namedDeps);
+            if (m_namedDeps.size() > 0) {
+                Log.instance().info("ServiceLifecycleHandler.init: adding extra/named dependencies %s",
+                                    m_namedDeps);
+                service.add(m_namedDeps);
+            }
         }
     }