Author: nbubna
Date: Fri Mar 30 15:04:23 2007
New Revision: 524292

URL: http://svn.apache.org/viewvc?view=rev&rev=524292
Log:
leave classname->Class conversion to configuration

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=524292&r1=524291&r2=524292
==============================================================================
--- 
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
 Fri Mar 30 15:04:23 2007
@@ -47,16 +47,6 @@
      * Creates a new instance using the minimum required info
      * necessary for a tool.
      */
-    public ToolInfo(String key, String classname)
-    {
-        setKey(key);
-        setType(classname);
-    }
-
-    /**
-     * Creates a new instance using the minimum required info
-     * necessary for a tool.
-     */
     public ToolInfo(String key, Class clazz)
     {
         setKey(key);
@@ -72,25 +62,6 @@
         if (this.key == null)
         {
             throw new NullPointerException("Key cannot be null");
-        }
-    }
-
-    /**
-     * If an instance of the tool cannot be created from
-     * the classname passed to this method, it will throw an exception.
-     *
-     * @param classname the fully qualified java.lang.Class name of the tool
-     */
-    public void setType(String classname)
-    {
-        try
-        {
-            // first, make sure we can get the Class
-            setClass(Utils.getClass(classname));
-        }
-        catch (ClassNotFoundException cnfe)
-        {
-            throw new IllegalArgumentException("Could not load Class: 
"+classname, cnfe);
         }
     }
 


Reply via email to