You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@felix.apache.org by gn...@apache.org on 2013/11/25 15:55:09 UTC

svn commit: r1545309 - /felix/trunk/utils/src/main/java/org/apache/felix/utils/extender/AbstractExtender.java

Author: gnodet
Date: Mon Nov 25 14:55:09 2013
New Revision: 1545309

URL: http://svn.apache.org/r1545309
Log:
[FELIX-4326] Revert bad commit
We should not remove the BundleListener to make sure we still receive bundle STOPPING events while shutting down the extender

Modified:
    felix/trunk/utils/src/main/java/org/apache/felix/utils/extender/AbstractExtender.java

Modified: felix/trunk/utils/src/main/java/org/apache/felix/utils/extender/AbstractExtender.java
URL: http://svn.apache.org/viewvc/felix/trunk/utils/src/main/java/org/apache/felix/utils/extender/AbstractExtender.java?rev=1545309&r1=1545308&r2=1545309&view=diff
==============================================================================
--- felix/trunk/utils/src/main/java/org/apache/felix/utils/extender/AbstractExtender.java (original)
+++ felix/trunk/utils/src/main/java/org/apache/felix/utils/extender/AbstractExtender.java Mon Nov 25 14:55:09 2013
@@ -116,7 +116,6 @@ public abstract class AbstractExtender i
 
     public void stop(BundleContext context) throws Exception {
         stopping = true;
-        this.context.removeBundleListener(this);
         while (!extensions.isEmpty()) {
             Collection<Bundle> toDestroy = chooseBundlesToDestroy(extensions.keySet());
             if (toDestroy == null || toDestroy.isEmpty()) {