RE: [Barracuda-commit] CVS Update: Barracuda

Hey Jake,

I just saw your change to Classes (see commit message below)...this will
effectively undo your change to ApplicationGateway. Consider the following:

//jrk_20020414.1_start
    //Class.forName is java1. Since Java2, use
    //Thread.currentThread().getContextClassLoader().loadClass("MyClass")
    //This should fix some classloading issues in Engines with multiple
    //class loaders (eg.. Tomcat-3.3.x and Tomcat-4.x.x)
    //Class cl = Class.forName(eventName);
//csc_060903_3                Class cl =
Thread.currentThread().getContextClassLoader().loadClass(eventName);
                Class cl = Classes.getClass(eventName); //csc_060903_3
//jrk_20020414.1_end

In other words, you changed App gateway to use the longer method for getting
a class, and then I changed your code to call Classes. SO...if you change
Classes to use the .forName() approach, that effectively resets your change
in app gateway.

Just wanted to make sure you were aware of this. I'm ok with either method,
but the reason why I was using the longer version in Classes was because of
your previous change in app gateway...

Christian
----------------------------------------------
Christian Cryder
Internet Architect, ATMReports.com
Project Chair, BarracudaMVC - http://barracudamvc.org
----------------------------------------------
"Coffee? I could quit anytime, just not today"


> -----Original Message-----
> From: [email protected]
> [mailto:[email protected]]On Behalf Of
> [email protected]
> Sent: Wednesday, June 11, 2003 8:31 AM
> To: [email protected]
> Subject: [Barracuda-commit] CVS Update: Barracuda
>
>
> CVSROOT:	/var/cvs
> Module name:	Barracuda
> Repository:	Barracuda/src/org/enhydra/barracuda/plankton/
> Changes by:	[email protected].(none)	03/06/11 08:30:57
>
> Modified files:
> 	Barracuda/src/org/enhydra/barracuda/plankton/: Classes.java
>
> Log message:
> 	minor update to class loading using Class.forName(String,
> boolean, ClassLoader).  Just looks a bit cleaner and does the
> same job as the previous code.
>
> 	Jake
>
> ===================================================================
> File: no file Classes.java		Status: Needs Checkout
>
>    Working revision:	1.3	Wed Jun 11 14:30:56 2003
>    Repository revision:	1.3
> /var/cvs/Barracuda/src/org/enhydra/barracuda/plankton/Classes.java,v
>
>    Existing Tags:
> 	Barracuda_1_2_0          	(revision: 1.1)
> 	jrk_20030217             	(revision: 1.1)
> 	jrk_20030203             	(revision: 1.1)
> 	csc_122702_2             	(revision: 1.1)
>
> _______________________________________________
> Barracuda-commit mailing list
> [email protected]
> http://barracudamvc.org/lists/listinfo/barracuda-commit