Re: [lang] new method on ClassUtils
Rolled back :) I'll bring it up again post 2.1. Hen On Thu, 3 Feb 2005 14:09:21 -0500, Gary Gregory <[EMAIL PROTECTED]> wrote: > Waiting after 2.1: +1. > > "Release early, release often" ;-) > > Gary > > -Original Message- > From: Stephen Colebourne [mailto:[EMAIL PROTECTED] > Sent: Thursday, February 03, 2005 2:07 AM > To: Jakarta Commons Developers List > Subject: Re: [lang] new method on ClassUtils > > Can we not leave this until after 2.1?? Rushing > something like this is a Bad Idea... > > Stephen > > --- Henri Yandell <[EMAIL PROTECTED]> wrote: > > On Wed, 2 Feb 2005 00:19:06 -, Stephen > > Colebourne > > <[EMAIL PROTECTED]> wrote: > > > I think that we would have to ask for classloader > > advice ;-) > > > > Probably. > > > > > I suspect we would need: > > > > > > loadClassSystemClassLoader() - Class.forName > > > loadClassThreadContextClassLoader() - thread > > > loadClassLangClassLoader() - from ClassUtils > > > loadClass(ClassLoader) > > > > We could just do the latter? Let's all the options > > be used without us > > having to drag through a list of all the likely > > options. No matter > > what classloader advice we got, it's likely to be > > usable. > > > > Hen > > > > > - > > To unsubscribe, e-mail: > > [EMAIL PROTECTED] > > For additional commands, e-mail: > > [EMAIL PROTECTED] > > > > > > - > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > - > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
RE: [lang] new method on ClassUtils
Waiting after 2.1: +1. "Release early, release often" ;-) Gary -Original Message- From: Stephen Colebourne [mailto:[EMAIL PROTECTED] Sent: Thursday, February 03, 2005 2:07 AM To: Jakarta Commons Developers List Subject: Re: [lang] new method on ClassUtils Can we not leave this until after 2.1?? Rushing something like this is a Bad Idea... Stephen --- Henri Yandell <[EMAIL PROTECTED]> wrote: > On Wed, 2 Feb 2005 00:19:06 -, Stephen > Colebourne > <[EMAIL PROTECTED]> wrote: > > I think that we would have to ask for classloader > advice ;-) > > Probably. > > > I suspect we would need: > > > > loadClassSystemClassLoader() - Class.forName > > loadClassThreadContextClassLoader() - thread > > loadClassLangClassLoader() - from ClassUtils > > loadClass(ClassLoader) > > We could just do the latter? Let's all the options > be used without us > having to drag through a list of all the likely > options. No matter > what classloader advice we got, it's likely to be > usable. > > Hen > > - > To unsubscribe, e-mail: > [EMAIL PROTECTED] > For additional commands, e-mail: > [EMAIL PROTECTED] > > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [lang] new method on ClassUtils
Can we not leave this until after 2.1?? Rushing something like this is a Bad Idea... Stephen --- Henri Yandell <[EMAIL PROTECTED]> wrote: > On Wed, 2 Feb 2005 00:19:06 -, Stephen > Colebourne > <[EMAIL PROTECTED]> wrote: > > I think that we would have to ask for classloader > advice ;-) > > Probably. > > > I suspect we would need: > > > > loadClassSystemClassLoader() - Class.forName > > loadClassThreadContextClassLoader() - thread > > loadClassLangClassLoader() - from ClassUtils > > loadClass(ClassLoader) > > We could just do the latter? Let's all the options > be used without us > having to drag through a list of all the likely > options. No matter > what classloader advice we got, it's likely to be > usable. > > Hen > > - > To unsubscribe, e-mail: > [EMAIL PROTECTED] > For additional commands, e-mail: > [EMAIL PROTECTED] > > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [lang] new method on ClassUtils
Potential problem with the forName is bad issue, we already have a method using forName: convertClassNamesToClasses which turns a List of String into a List of Class using forName. So, a) this seems to be a problem? b) I'd like to point it at the new forName method (or however that gets solved). Hen On Tue, 1 Feb 2005 20:11:09 -0500, Henri Yandell <[EMAIL PROTECTED]> wrote: > On Wed, 2 Feb 2005 00:19:06 -, Stephen Colebourne > <[EMAIL PROTECTED]> wrote: > > I think that we would have to ask for classloader advice ;-) > > Probably. > > > I suspect we would need: > > > > loadClassSystemClassLoader() - Class.forName > > loadClassThreadContextClassLoader() - thread > > loadClassLangClassLoader() - from ClassUtils > > loadClass(ClassLoader) > > We could just do the latter? Let's all the options be used without us > having to drag through a list of all the likely options. No matter > what classloader advice we got, it's likely to be usable. > > Hen > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [lang] new method on ClassUtils
On Wed, 2 Feb 2005 00:19:06 -, Stephen Colebourne <[EMAIL PROTECTED]> wrote: > I think that we would have to ask for classloader advice ;-) Probably. > I suspect we would need: > > loadClassSystemClassLoader() - Class.forName > loadClassThreadContextClassLoader() - thread > loadClassLangClassLoader() - from ClassUtils > loadClass(ClassLoader) We could just do the latter? Let's all the options be used without us having to drag through a list of all the likely options. No matter what classloader advice we got, it's likely to be usable. Hen - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [lang] new method on ClassUtils
I think that we would have to ask for classloader advice ;-) I suspect we would need: loadClassSystemClassLoader() - Class.forName loadClassThreadContextClassLoader() - thread loadClassLangClassLoader() - from ClassUtils loadClass(ClassLoader) ;-) Stephen - Original Message - From: "Henri Yandell" <[EMAIL PROTECTED]> The important part for me is the munging that stops primitives being special and [] not working. I'm happy to replace the forName part with a better way and rename to getClass. Would Thread.currentThread().getContextClassLoader().loadClass(String) be acceptable instead of the forName call? Hen On Thu, 27 Jan 2005 23:43:58 -, Stephen Colebourne <[EMAIL PROTECTED]> wrote: While I support the addition of a get class method in principle, I am concerned that this brushes over the class loader issue. I would say that Class.forName() will often cause problems, so maybe this isn't the best way to code this. Stephen - Original Message - From: "Henri Yandell" <[EMAIL PROTECTED]> >I committed this btw. > > Hen > > On Wed, 26 Jan 2005 17:34:39 -0500, Henri Yandell <[EMAIL PROTECTED]> > wrote: >> I'd like to add ClassUtils.forName; to all intents and purposes the >> same as Class.forName except that: >> >> 1) It understands arrays ending with [] instead of the [L; >> ugliness. >> 2) It can handle primitives, "int" would correctly return int.class. >> 3) (perhaps?) null-safe. No idea really, just throwing it in :) >> >> Hen >> > > - > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [lang] new method on ClassUtils
The important part for me is the munging that stops primitives being special and [] not working. I'm happy to replace the forName part with a better way and rename to getClass. Would Thread.currentThread().getContextClassLoader().loadClass(String) be acceptable instead of the forName call? Hen On Thu, 27 Jan 2005 23:43:58 -, Stephen Colebourne <[EMAIL PROTECTED]> wrote: > While I support the addition of a get class method in principle, I am > concerned that this brushes over the class loader issue. I would say that > Class.forName() will often cause problems, so maybe this isn't the best way > to code this. > > Stephen > > - Original Message - > From: "Henri Yandell" <[EMAIL PROTECTED]> > >I committed this btw. > > > > Hen > > > > On Wed, 26 Jan 2005 17:34:39 -0500, Henri Yandell <[EMAIL PROTECTED]> > > wrote: > >> I'd like to add ClassUtils.forName; to all intents and purposes the > >> same as Class.forName except that: > >> > >> 1) It understands arrays ending with [] instead of the [L; > >> ugliness. > >> 2) It can handle primitives, "int" would correctly return int.class. > >> 3) (perhaps?) null-safe. No idea really, just throwing it in :) > >> > >> Hen > >> > > > > - > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [lang] new method on ClassUtils
While I support the addition of a get class method in principle, I am concerned that this brushes over the class loader issue. I would say that Class.forName() will often cause problems, so maybe this isn't the best way to code this. Stephen - Original Message - From: "Henri Yandell" <[EMAIL PROTECTED]> I committed this btw. Hen On Wed, 26 Jan 2005 17:34:39 -0500, Henri Yandell <[EMAIL PROTECTED]> wrote: I'd like to add ClassUtils.forName; to all intents and purposes the same as Class.forName except that: 1) It understands arrays ending with [] instead of the [L; ugliness. 2) It can handle primitives, "int" would correctly return int.class. 3) (perhaps?) null-safe. No idea really, just throwing it in :) Hen - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [lang] new method on ClassUtils
I committed this btw. Hen On Wed, 26 Jan 2005 17:34:39 -0500, Henri Yandell <[EMAIL PROTECTED]> wrote: > I'd like to add ClassUtils.forName; to all intents and purposes the > same as Class.forName except that: > > 1) It understands arrays ending with [] instead of the [L; ugliness. > 2) It can handle primitives, "int" would correctly return int.class. > 3) (perhaps?) null-safe. No idea really, just throwing it in :) > > Hen > - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]