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 2013/02/07 15:30:53 UTC

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

Author: rickhall
Date: Thu Feb  7 14:30:52 2013
New Revision: 1443503

URL: http://svn.apache.org/viewvc?rev=1443503&view=rev
Log:
Fix logic bug in start level change handling where the active start level
was not getting initialized properly, thus causing it to improperly re-process
bundles on the boundaries of the start level change range. (FELIX-3411)

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=1443503&r1=1443502&r2=1443503&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 Thu Feb  7 14:30:52 2013
@@ -1195,6 +1195,7 @@ public class Felix extends BundleImpl im
             // Determine the range of start levels to process.
             int low = (isLowering) ? m_targetStartLevel + 1 : m_activeStartLevel + 1;
             int high = (isLowering) ? m_activeStartLevel : m_targetStartLevel;
+            m_activeStartLevel = (isLowering) ? high : low;
 
             // Process bundles and stop or start them accordingly.
             while (bundlesRemaining)