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

Posted to commits@felix.apache.org by pa...@apache.org on 2008/10/10 23:32:52 UTC

svn commit: r703577 - /felix/trunk/framework/src/main/java/org/apache/felix/framework/Felix.java

Author: pauls
Date: Fri Oct 10 14:32:52 2008
New Revision: 703577

URL: http://svn.apache.org/viewvc?rev=703577&view=rev
Log:
Set the activator to null on Bundle.stop() to allow for earlier garbage collection. (FELIX-762)

Modified:
    felix/trunk/framework/src/main/java/org/apache/felix/framework/Felix.java

Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/Felix.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/Felix.java?rev=703577&r1=703576&r2=703577&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/Felix.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/Felix.java Fri Oct 10 14:32:52 2008
@@ -1644,6 +1644,9 @@
             ((BundleContextImpl) info.getBundleContext()).invalidate();
             info.setBundleContext(null);
 
+            // Clean up the bundle activator
+            info.setActivator(null);
+
             // Unregister any services offered by this bundle.
             m_registry.unregisterServices(bundle);
 
@@ -2034,6 +2037,9 @@
             ((BundleContextImpl) info.getBundleContext()).invalidate();
             info.setBundleContext(null);
 
+            // Clean up the bundle activator.
+            info.setActivator(null);
+
             // Unregister any services offered by this bundle.
             m_registry.unregisterServices(bundle);