You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by cl...@apache.org on 2007/08/14 16:53:25 UTC
svn commit: r565785 -
/felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/handlers/dependency/Dependency.java
Author: clement
Date: Tue Aug 14 07:53:24 2007
New Revision: 565785
URL: http://svn.apache.org/viewvc?view=rev&rev=565785
Log:
Solve a bug in synchronization stuff. IPOJO released services when exiting methods. However, for service factory, it must not released services as the binding can be stateful.
Now, iPOJO releases services only when they disappears.
Modified:
felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/handlers/dependency/Dependency.java
Modified: felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/handlers/dependency/Dependency.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/handlers/dependency/Dependency.java?view=diff&rev=565785&r1=565784&r2=565785
==============================================================================
--- felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/handlers/dependency/Dependency.java (original)
+++ felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/handlers/dependency/Dependency.java Tue Aug 14 07:53:24 2007
@@ -135,7 +135,6 @@
*/
private ServiceContext m_serviceContext;
-
/**
* Thread Local.
*/
@@ -730,10 +729,6 @@
// Exit the method flow => Release all object
m_usage.getObjects().clear();
List refs = m_usage.getReferences();
- for (int i = 0; i < refs.size(); i++) {
- ServiceReference ref = (ServiceReference) refs.get(i);
- ungetService(ref);
- }
refs.clear();
}
}