You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by pd...@apache.org on 2016/09/27 13:22:13 UTC

svn commit: r1762474 - /felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java

Author: pderop
Date: Tue Sep 27 13:22:13 2016
New Revision: 1762474

URL: http://svn.apache.org/viewvc?rev=1762474&view=rev
Log:
FELIX-5274: avoid the state maching to be reentrant.

Modified:
    felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java

Modified: felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java?rev=1762474&r1=1762473&r2=1762474&view=diff
==============================================================================
--- felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java (original)
+++ felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java Tue Sep 27 13:22:13 2016
@@ -913,9 +913,9 @@ public class ComponentImpl implements Co
      * Runs the state machine, to see if a change event has to trigger some component state transition.
      */
     private void handleChange() {
-        // if (isHandlingChange()) {
-        //     return;
-        // }
+        if (isHandlingChange()) {
+            return;
+        }
         m_logger.debug("handleChanged");
     	handlingChange(true);
         try {