You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by fm...@apache.org on 2009/04/20 13:21:45 UTC

svn commit: r766654 - /felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DependencyManager.java

Author: fmeschbe
Date: Mon Apr 20 11:21:45 2009
New Revision: 766654

URL: http://svn.apache.org/viewvc?rev=766654&view=rev
Log:
FELIX-993 drop checking the target filter on service unregistration
(see issue for discussion)

Modified:
    felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DependencyManager.java

Modified: felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DependencyManager.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DependencyManager.java?rev=766654&r1=766653&r2=766654&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DependencyManager.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/impl/DependencyManager.java Mon Apr 20 11:21:45 2009
@@ -264,15 +264,6 @@
      */
     private void serviceRemoved( ServiceReference reference )
     {
-        // ignore the service, if it does not match the target filter
-        if ( !targetFilterMatch( reference ) )
-        {
-            m_componentManager.log( LogService.LOG_DEBUG, "Dependency Manager: Ignoring removed Service for "
-                + m_dependencyMetadata.getName() + " : Service " + reference.getProperty( Constants.SERVICE_ID )
-                + " does not match target filter " + getTarget(), m_componentManager.getComponentMetadata(), null );
-            return;
-        }
-
         // decrement the number of services
         m_size--;