Author: markt
Date: Tue Feb 19 20:52:26 2013
New Revision: 1447919

URL: http://svn.apache.org/r1447919
Log:
Remove unused code

Modified:
    tomcat/trunk/java/org/apache/tomcat/util/digester/FactoryCreateRule.java

Modified: 
tomcat/trunk/java/org/apache/tomcat/util/digester/FactoryCreateRule.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/digester/FactoryCreateRule.java?rev=1447919&r1=1447918&r2=1447919&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/digester/FactoryCreateRule.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/digester/FactoryCreateRule.java 
Tue Feb 19 20:52:26 2013
@@ -60,21 +60,8 @@ public class FactoryCreateRule extends R
         this.ignoreCreateExceptions = ignoreCreateExceptions;
     }
 
-    // ----------------------------------------------------- Instance Variables
-
-
-    /**
-     * The attribute containing an override class name if it is present.
-     */
-    protected String attributeName = null;
-
-
-    /**
-     * The Java class name of the ObjectCreationFactory to be created.
-     * This class must have a no-arguments constructor.
-     */
-    protected String className = null;
 
+    // ----------------------------------------------------- Instance Variables
 
     /**
      * The object creation factory we will use to instantiate objects
@@ -102,7 +89,7 @@ public class FactoryCreateRule extends R
             }
 
             try {
-                Object instance = 
getFactory(attributes).createObject(attributes);
+                Object instance = creationFactory.createObject(attributes);
 
                 if (digester.log.isDebugEnabled()) {
                     digester.log.debug("[FactoryCreateRule]{" + digester.match 
+
@@ -124,7 +111,7 @@ public class FactoryCreateRule extends R
             }
 
         } else {
-            Object instance = getFactory(attributes).createObject(attributes);
+            Object instance = creationFactory.createObject(attributes);
 
             if (digester.log.isDebugEnabled()) {
                 digester.log.debug("[FactoryCreateRule]{" + digester.match +
@@ -172,11 +159,7 @@ public class FactoryCreateRule extends R
      */
     @Override
     public void finish() throws Exception {
-
-        if (attributeName != null) {
-            creationFactory = null;
-        }
-
+        // NO-OP
     }
 
 
@@ -187,52 +170,12 @@ public class FactoryCreateRule extends R
     public String toString() {
 
         StringBuilder sb = new StringBuilder("FactoryCreateRule[");
-        sb.append("className=");
-        sb.append(className);
-        sb.append(", attributeName=");
-        sb.append(attributeName);
         if (creationFactory != null) {
-            sb.append(", creationFactory=");
+            sb.append("creationFactory=");
             sb.append(creationFactory);
         }
         sb.append("]");
         return (sb.toString());
 
     }
-
-
-    // ------------------------------------------------------ Protected Methods
-
-
-    /**
-     * Return an instance of our associated object creation factory,
-     * creating one if necessary.
-     *
-     * @param attributes Attributes passed to our factory creation element
-     *
-     * @exception Exception if any error occurs
-     */
-    protected ObjectCreationFactory getFactory(Attributes attributes)
-            throws Exception {
-
-        if (creationFactory == null) {
-            String realClassName = className;
-            if (attributeName != null) {
-                String value = attributes.getValue(attributeName);
-                if (value != null) {
-                    realClassName = value;
-                }
-            }
-            if (digester.log.isDebugEnabled()) {
-                digester.log.debug("[FactoryCreateRule]{" + digester.match +
-                        "} New factory " + realClassName);
-            }
-            Class<?> clazz = 
digester.getClassLoader().loadClass(realClassName);
-            creationFactory = (ObjectCreationFactory)
-                    clazz.newInstance();
-            creationFactory.setDigester(digester);
-        }
-        return (creationFactory);
-
-    }
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to