commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From skitch...@apache.org
Subject svn commit: r153490 - in jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2: Digester.java SAXHandler.java
Date Sat, 12 Feb 2005 09:27:16 GMT
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 {@link #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 {@link 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 {@link #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 {@link #getClassLoader} for more information.
      *
      * @param use determines whether to use Context ClassLoader.
      */



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message