You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2001/01/10 06:11:40 UTC

cvs commit: jakarta-tomcat-4.1/catalina/src/share/org/apache/catalina/startup Bootstrap.java

remm        01/01/09 21:11:40

  Modified:    catalina/src/share/org/apache/catalina/startup
                        Bootstrap.java
  Log:
  - Cosmetic issue : change the name of systemLoader -> commonLoader.
    I had forgotten to do it earlier.
  - Craig : Do I roll that back to TC 4.0 ?
  
  Revision  Changes    Path
  1.9       +9 -9      jakarta-tomcat-4.1/catalina/src/share/org/apache/catalina/startup/Bootstrap.java
  
  Index: Bootstrap.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.1/catalina/src/share/org/apache/catalina/startup/Bootstrap.java,v
  retrieving revision 1.8
  retrieving revision 1.9
  diff -u -r1.8 -r1.9
  --- Bootstrap.java	2000/12/21 23:47:20	1.8
  +++ Bootstrap.java	2001/01/10 05:11:40	1.9
  @@ -1,7 +1,7 @@
   /*
  - * $Header: /home/cvs/jakarta-tomcat-4.1/catalina/src/share/org/apache/catalina/startup/Bootstrap.java,v 1.8 2000/12/21 23:47:20 craigmcc Exp $
  - * $Revision: 1.8 $
  - * $Date: 2000/12/21 23:47:20 $
  + * $Header: /home/cvs/jakarta-tomcat-4.1/catalina/src/share/org/apache/catalina/startup/Bootstrap.java,v 1.9 2001/01/10 05:11:40 remm Exp $
  + * $Revision: 1.9 $
  + * $Date: 2001/01/10 05:11:40 $
    *
    * ====================================================================
    *
  @@ -85,7 +85,7 @@
    * class path and therefore not visible to application level classes.
    *
    * @author Craig R. McClanahan
  - * @version $Revision: 1.8 $ $Date: 2000/12/21 23:47:20 $
  + * @version $Revision: 1.9 $ $Date: 2001/01/10 05:11:40 $
    */
   
   public final class Bootstrap {
  @@ -117,9 +117,9 @@
           }
   
           // Construct the class loaders we will need
  -        ClassLoader systemLoader = createSystemLoader();
  -        ClassLoader catalinaLoader = createCatalinaLoader(systemLoader);
  -        ClassLoader sharedLoader = createSharedLoader(systemLoader);
  +        ClassLoader commonLoader = createCommonLoader();
  +        ClassLoader catalinaLoader = createCatalinaLoader(commonLoader);
  +        ClassLoader sharedLoader = createSharedLoader(commonLoader);
   
   	// Load our startup class and call its process() method
   	try {
  @@ -171,10 +171,10 @@
        * Construct and return the class loader to be used for loading
        * of the shared system classes.
        */
  -    private static ClassLoader createSystemLoader() {
  +    private static ClassLoader createCommonLoader() {
   
           if (debug >= 1)
  -            log("Creating SYSTEM class loader");
  +            log("Creating COMMON class loader");
   
           // Construct the "class path" for this class loader
           ArrayList list = new ArrayList();
  
  
  

Re: cvs commit: jakarta-tomcat-4.1/catalina/src/share/org/apache/catalina/startup Bootstrap.java

Posted by "Craig R. McClanahan" <Cr...@eng.sun.com>.
remm@apache.org wrote:

> remm        01/01/09 21:11:40
>
>   Modified:    catalina/src/share/org/apache/catalina/startup
>                         Bootstrap.java
>   Log:
>   - Cosmetic issue : change the name of systemLoader -> commonLoader.
>     I had forgotten to do it earlier.
>   - Craig : Do I roll that back to TC 4.0 ?
>
>

I wouldn't ... it's not worth making cosmetic changes unless it will affect our ability to do bug fixes, and this doesn't seem likely to matter.

Craig