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 2014/07/02 14:56:07 UTC

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

Author: pderop
Date: Wed Jul  2 12:56:06 2014
New Revision: 1607354

URL: http://svn.apache.org/r1607354
Log:
FELIX-4002: Fixed addStateListener method: we don't have to call any missed listener callbacks if the component is stopped.

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

Modified: felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ComponentImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ComponentImpl.java?rev=1607354&r1=1607353&r2=1607354&view=diff
==============================================================================
--- felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ComponentImpl.java (original)
+++ felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ComponentImpl.java Wed Jul  2 12:56:06 2014
@@ -572,11 +572,7 @@ public class ComponentImpl implements Co
                         listener.stopping(ComponentImpl.this);
                         break;
                     case LISTENER_STOPPED:
-                        // this new listener missed the starting/started/stopping/stopped cb
-                        listener.starting(ComponentImpl.this);
-                        listener.started(ComponentImpl.this);
-                        listener.stopping(ComponentImpl.this);
-                        listener.stopped(ComponentImpl.this);
+                        // no need to call missed listener callbacks
                         break;
                 }
             }