You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ma...@apache.org on 2010/04/04 20:37:44 UTC

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

Author: marrs
Date: Sun Apr  4 18:37:44 2010
New Revision: 930730

URL: http://svn.apache.org/viewvc?rev=930730&view=rev
Log:
Changed the exception handling after Friday's refactoring. No such method exceptions should be ignored here.

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

Modified: felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ServiceImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ServiceImpl.java?rev=930730&r1=930729&r2=930730&view=diff
==============================================================================
--- felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ServiceImpl.java (original)
+++ felix/trunk/dependencymanager/core/src/main/java/org/apache/felix/dm/impl/ServiceImpl.java Sun Apr  4 18:37:44 2010
@@ -591,8 +591,12 @@ public class ServiceImpl implements Serv
                     new Class[][] {{ Object.class, DependencyManager.class, Service.class }, { DependencyManager.class, Service.class }, { Object.class }, {}}, 
                     new Object[][] {{ m_serviceInstance, m_manager, this }, { m_manager, this }, { m_serviceInstance }, {}});
             }
+            catch (NoSuchMethodException e) {
+            	// we ignore the fact that the method was not found
+            }
             catch (Exception e) {
-                m_logger.log(Logger.LOG_ERROR, "Error trying to invoke method named " + name + ".", e);
+            	// but any other exception means that the method was invoked but somehow failed
+                m_logger.log(Logger.LOG_WARNING, "Error trying to invoke method named " + name + ".", e);
             }
         }
     }