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 2015/05/21 19:58:58 UTC

svn commit: r1680935 - in /felix/trunk/dependencymanager/org.apache.felix.dependencymanager: resources/changelog.txt src/org/apache/felix/dm/impl/ConfigurationDependencyImpl.java

Author: pderop
Date: Thu May 21 17:58:58 2015
New Revision: 1680935

URL: http://svn.apache.org/r1680935
Log:
uncommitted the patch for FELIX-4879, which is invalid.

Modified:
    felix/trunk/dependencymanager/org.apache.felix.dependencymanager/resources/changelog.txt
    felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ConfigurationDependencyImpl.java

Modified: felix/trunk/dependencymanager/org.apache.felix.dependencymanager/resources/changelog.txt
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/org.apache.felix.dependencymanager/resources/changelog.txt?rev=1680935&r1=1680934&r2=1680935&view=diff
==============================================================================
--- felix/trunk/dependencymanager/org.apache.felix.dependencymanager/resources/changelog.txt (original)
+++ felix/trunk/dependencymanager/org.apache.felix.dependencymanager/resources/changelog.txt Thu May 21 17:58:58 2015
@@ -8,7 +8,6 @@ Release org.apache.felix.dependencymanag
 ** Improvement
     * [FELIX-4614] - Factory create() method should have access to the component definition
     * [FELIX-4878] - Support more signatures for Dependency callbacks
-    * [FELIX-4879] - ConfigurationDependency should always "need instance".
     * [FELIX-4880] - Missing callback instance support for some adapters
     * [FELIX-4873] - Enhance DM API to get missing and circular dependencies
     

Modified: felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ConfigurationDependencyImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ConfigurationDependencyImpl.java?rev=1680935&r1=1680934&r2=1680935&view=diff
==============================================================================
--- felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ConfigurationDependencyImpl.java (original)
+++ felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ConfigurationDependencyImpl.java Thu May 21 17:58:58 2015
@@ -93,10 +93,7 @@ public class ConfigurationDependencyImpl
 
     @Override
     public boolean needsInstance() {
-    	// we need the component instances even if there is a callback instance, which could need to access to
-    	// component instances while being invoked in the updated callback. So we return true here, even if there
-    	// is a configured callback instance.
-        return true; 
+        return m_callbackInstance == null;
     }
 
     @Override