You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by cl...@apache.org on 2013/10/08 09:58:19 UTC

svn commit: r1530181 - /felix/trunk/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/IPojoFactory.java

Author: clement
Date: Tue Oct  8 07:58:19 2013
New Revision: 1530181

URL: http://svn.apache.org/r1530181
Log:
Let the getRequiredHandlerList method throw ConfigurationException.

Modified:
    felix/trunk/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/IPojoFactory.java

Modified: felix/trunk/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/IPojoFactory.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/IPojoFactory.java?rev=1530181&r1=1530180&r2=1530181&view=diff
==============================================================================
--- felix/trunk/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/IPojoFactory.java (original)
+++ felix/trunk/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/IPojoFactory.java Tue Oct  8 07:58:19 2013
@@ -209,8 +209,9 @@ public abstract class IPojoFactory imple
      * Computes the required handler list.
      * Each sub-type must override this method.
      * @return the required handler list
+     * @throws ConfigurationException when the list of handler cannot be computed.
      */
-    public abstract List<RequiredHandler> getRequiredHandlerList();
+    public abstract List<RequiredHandler> getRequiredHandlerList() throws ConfigurationException;
 
     /**
      * Creates an instance.
@@ -701,7 +702,11 @@ public abstract class IPojoFactory imple
     public void restart() {
     	// Call sub-class to get the list of required handlers.
         m_requiredHandlers.clear();
-        m_requiredHandlers.addAll(getRequiredHandlerList());
+        try {
+            m_requiredHandlers.addAll(getRequiredHandlerList());
+        } catch (ConfigurationException e) {
+            // Swallow the exception.
+        }
     }
 
     /**