Author: skitching
Date: Sat Feb 12 01:27:14 2005
New Revision: 153490
URL: http://svn.apache.org/viewcvs?view=rev&rev=153490
Log:
Change default for useContextClassLoader to true. It would be a
very strange app where the classloader was set but shouldn't
be used to load user classes..
Modified:
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Digester.java
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/SAXHandler.java
Modified:
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Digester.java
URL:
http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Digester.java?view=diff&r1=153489&r2=153490
==============================================================================
---
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Digester.java
(original)
+++
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Digester.java
Sat Feb 12 01:27:14 2005
@@ -223,12 +223,13 @@
/**
* Set the class loader to be used for instantiating application objects
- * when required. If a non-null value is passed to this method, then
- * method [EMAIL PROTECTED] #setUseContextClassLoader} will have no effect.
+ * when required.
* <p>
* When an Action is executed due to some xml input, and that Action
* wishes to create an object to represent the input, then the class
* used will be loaded via the specified classloader.
+ * <p>
+ * See [EMAIL PROTECTED] SAXHandler#getClassLoader} for more information.
*
* @param classLoader The new class loader to use, or <code>null</code>
* to revert to the standard rules
@@ -339,9 +340,9 @@
* Determine whether to use the Context Classloader (the one found by
* calling <code>Thread.currentThread().getContextClassLoader()</code>)
* to resolve/load classes when an Action needs to create an instance of
- * an object to represent data in the xml input. If this is set to false,
- * and there is no explicit classloader set, then the same classloader
- * that loaded the Action class is used.
+ * an object to represent data in the xml input.
+ * <p>
+ * The default value is true.
* <p>
* See [EMAIL PROTECTED] #setExplicitClassLoader}.
*
Modified:
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/SAXHandler.java
URL:
http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/SAXHandler.java?view=diff&r1=153489&r2=153490
==============================================================================
---
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/SAXHandler.java
(original)
+++
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/SAXHandler.java
Sat Feb 12 01:27:14 2005
@@ -189,9 +189,9 @@
/**
* Do we want to use the Context classloader when loading classes for
- * instantiating new objects. Default is false.
+ * instantiating new objects. Default is true.
*/
- private boolean useContextClassLoader = false;
+ private boolean useContextClassLoader = true;
/**
* Has this instance had its initialize method called yet?
@@ -383,9 +383,11 @@
/**
* Determine whether to use the Context ClassLoader (the one found by
* calling <code>Thread.currentThread().getContextClassLoader()</code>)
- * to resolve/load classes that are defined in various rules. If not
- * using Context ClassLoader, then the class-loading defaults to
- * using the calling-class' ClassLoader.
+ * to resolve/load classes that are defined in various rules.
+ * <p>
+ * The default is true.
+ * <p>
+ * See [EMAIL PROTECTED] #getClassLoader} for more information.
*
* @param use determines whether to use Context ClassLoader.
*/
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
