Yes, it is the same.

Thank you.

James Mitchell
Software Engineer/Struts Evangelist
http://www.open-tools.org

"Only two things are infinite, the universe and human stupidity, and I'm not
sure about the former."
- Albert Einstein (1879-1955)


> -----Original Message-----
> From: Antoni Reus [mailto:antoni.reus@;wanadoo.es]
> Sent: Thursday, November 07, 2002 4:21 PM
> To: Struts Developers List
> Subject: Re: [Vote] Modify classloading in RequestUtils
>
>
> It seems that this is related with
>
> http://nagoya.apache.org/bugzilla/show_bug.cgi?id=13686
>
> a similar issue with iplanet
>
>
> A Dijous 07 Novembre 2002 17:13, James Mitchell va escriure:
> > Not sure if you caught the thread on the users list, or the bug recently
> > posted by Kjeld Froberg:
> >
> >  http://nagoya.apache.org/bugzilla/show_bug.cgi?id=14332
> >
> >
> > Here's the proposed code change:
> >
> > RequestUtils.java
> > -----------------------------------------------
> > From:
> >     public static Class applicationClass(String className)
> >         throws ClassNotFoundException {
> >
> >         // Look up the class loader to be used
> >         ClassLoader classLoader =
> >             Thread.currentThread().getContextClassLoader();
> >         if (classLoader == null) {
> >             classLoader = RequestUtils.class.getClassLoader();
> >         }
> >
> >         // Attempt to load the specified class
> >         return (classLoader.loadClass(className));
> >     }
> > -----------------------------------------------
> > To:
> >     public static Class applicationClass(String className)
> >         throws ClassNotFoundException {
> >             ClassLoader ctxLoader = null;
> >             try {
> >                     ctxLoader =
> Thread.currentThread().getContextClassLoader();
> >                     return Class.forName(className, true, ctxLoader);
> >
> >             } catch(ClassNotFoundException ex) {
> >                     if(ctxLoader == null) { throw ex; }
> >             } catch(SecurityException ex) {
> >
> >             }
> >             return Class.forName(className);
> >     }
> >
> > By changing these few lines, I am able to run the full test
> suite without
> > error. (And that includes the new test.tomcat.33)
> >
> > I'm not sure if this should be a [Vote] or a [commit and wait till they
> > complain], so I opted for the former.
> >
> > Even though I'm not truly sure of the consequences of this
> change, I'm +1.
> >
> >
> > James Mitchell
> > Software Engineer/Struts Evangelist
> > http://www.open-tools.org
> >
> > "Only two things are infinite, the universe and human stupidity, and I'm
> > not sure about the former."
> > - Albert Einstein (1879-1955)
>
>
> --
> To unsubscribe, e-mail:
> <mailto:struts-dev-unsubscribe@;jakarta.apache.org>
> For additional commands, e-mail:
> <mailto:struts-dev-help@;jakarta.apache.org>
>
>


--
To unsubscribe, e-mail:   <mailto:struts-dev-unsubscribe@;jakarta.apache.org>
For additional commands, e-mail: <mailto:struts-dev-help@;jakarta.apache.org>

Reply via email to