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 2014/03/19 07:56:58 UTC

svn commit: r1579150 - /felix/sandbox/pderop/dependencymanager-prototype/dm/src/dm/impl/ComponentImpl.java

Author: pderop
Date: Wed Mar 19 06:56:58 2014
New Revision: 1579150

URL: http://svn.apache.org/r1579150
Log:
Fixed invokeRemoveOptionalDependencies() method, which must invoke unbind callback even if the dependency is in
instance bound state.

Modified:
    felix/sandbox/pderop/dependencymanager-prototype/dm/src/dm/impl/ComponentImpl.java

Modified: felix/sandbox/pderop/dependencymanager-prototype/dm/src/dm/impl/ComponentImpl.java
URL: http://svn.apache.org/viewvc/felix/sandbox/pderop/dependencymanager-prototype/dm/src/dm/impl/ComponentImpl.java?rev=1579150&r1=1579149&r2=1579150&view=diff
==============================================================================
--- felix/sandbox/pderop/dependencymanager-prototype/dm/src/dm/impl/ComponentImpl.java (original)
+++ felix/sandbox/pderop/dependencymanager-prototype/dm/src/dm/impl/ComponentImpl.java Wed Mar 19 06:56:58 2014
@@ -681,7 +681,7 @@ public class ComponentImpl implements Co
 
     private void invokeRemoveOptionalDependencies() { 
         for (DependencyContext d : m_dependencies) {
-            if (!d.isInstanceBound() && ! d.isRequired()) {
+            if (! d.isRequired()) {
                 for (Event e : m_dependencyEvents.get(d)) {
                     d.invokeRemove(e);
                 }