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: [email protected]
For additional commands, e-mail: [email protected]
