Author: rickhall
Date: Wed Sep 17 07:05:16 2008
New Revision: 696309

URL: http://svn.apache.org/viewvc?rev=696309&view=rev
Log:
Applied patch (FELIX-728) to fix an issue where requests to Start Level
and Package Admin threads were not being immediately released, which was
inhibiting garbage collection.

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

Modified: 
felix/trunk/framework/src/main/java/org/apache/felix/framework/PackageAdminImpl.java
URL: 
http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/PackageAdminImpl.java?rev=696309&r1=696308&r2=696309&view=diff
==============================================================================
--- 
felix/trunk/framework/src/main/java/org/apache/felix/framework/PackageAdminImpl.java
 (original)
+++ 
felix/trunk/framework/src/main/java/org/apache/felix/framework/PackageAdminImpl.java
 Wed Sep 17 07:05:16 2008
@@ -227,9 +227,9 @@
     {
         // This thread loops forever, thus it should
         // be a daemon thread.
-        Bundle[] bundles = null;
         while (true)
         {
+            Bundle[] bundles = null;
             synchronized (this)
             {
                 // Wait for a refresh request.

Modified: 
felix/trunk/framework/src/main/java/org/apache/felix/framework/StartLevelImpl.java
URL: 
http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/StartLevelImpl.java?rev=696309&r1=696308&r2=696309&view=diff
==============================================================================
--- 
felix/trunk/framework/src/main/java/org/apache/felix/framework/StartLevelImpl.java
 (original)
+++ 
felix/trunk/framework/src/main/java/org/apache/felix/framework/StartLevelImpl.java
 Wed Sep 17 07:05:16 2008
@@ -226,12 +226,11 @@
 
     public void run()
     {
-        Object request = null;
-
         // This thread loops forever, thus it should
         // be a daemon thread.
         while (true)
         {
+            Object request = null;
             synchronized (m_requestList)
             {
                 // Wait for a request.


Reply via email to