You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@velocity.apache.org by nb...@apache.org on 2007/05/17 04:53:32 UTC

svn commit: r538792 - /velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/ToolInfo.java

Author: nbubna
Date: Wed May 16 19:53:32 2007
New Revision: 538792

URL: http://svn.apache.org/viewvc?view=rev&rev=538792
Log:
catch all Throwables when testing newInstance() on tool class

Modified:
    velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/ToolInfo.java

Modified: velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/ToolInfo.java
URL: http://svn.apache.org/viewvc/velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/ToolInfo.java?view=diff&rev=538792&r1=538791&r2=538792
==============================================================================
--- velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/ToolInfo.java (original)
+++ velocity/tools/branches/2.x/src/main/java/org/apache/velocity/tools/ToolInfo.java Wed May 16 19:53:32 2007
@@ -85,9 +85,9 @@
             // ok, we'll accept this one
             this.clazz = clazz;
         }
-        catch (Exception e)
+        catch (Throwable t)
         {
-            throw new IllegalArgumentException("Could not create an instance of "+clazz, e);
+            throw new IllegalArgumentException("Could not create an instance of "+clazz, t);
         }
 
         // search for a configure(Map params) method in the class