You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ri...@apache.org on 2008/09/17 16:05:17 UTC
svn commit: r696309 - in
/felix/trunk/framework/src/main/java/org/apache/felix/framework:
PackageAdminImpl.java StartLevelImpl.java
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.