You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@deltaspike.apache.org by Charles Moulliard <ch...@gmail.com> on 2012/09/12 10:38:45 UTC
Can we remove this comment ?
Hi,
In the ClassUtils of deltaspike core module, there is a mention of a
myfaces.jar file, can we remove it ?
/**
* Loads class for the given name
* @param name name of the class
* @return loaded class
* @throws ClassNotFoundException if the class can't be loadedx@
*/
public static Class loadClassForName(String name) throws
ClassNotFoundException
{
try
{
// Try WebApp ClassLoader first
return Class.forName(name, false, // do not initialize for faster
startup
getClassLoader(null));
}
catch (ClassNotFoundException ignore)
{
// fallback: Try ClassLoader for ClassUtils (i.e. the myfaces.jar
lib)
return Class.forName(name, false, // do not initialize for faster
startup
ClassUtils.class.getClassLoader());
}
}
Regards,
--
Charles Moulliard
Apache Committer / Sr. Pr. Consultant at FuseSource.com
Twitter : @cmoulliard
Blog : http://cmoulliard.blogspot.com
Re: Can we remove this comment ?
Posted by Romain Manni-Bucau <rm...@gmail.com>.
oki so forget my answer ;)
*Romain Manni-Bucau*
*Twitter: @rmannibucau*
*Blog: http://rmannibucau.wordpress.com*
2012/9/12 Charles Moulliard <ch...@gmail.com>
> No a code issue but clarification to be done in the code to improve it.
>
> On Wed, Sep 12, 2012 at 12:57 PM, Romain Manni-Bucau
> <rm...@gmail.com>wrote:
>
> > oh ok thought it was a code issue but just a comment one, right?
> >
> > *Romain Manni-Bucau*
> > *Twitter: @rmannibucau*
> > *Blog: http://rmannibucau.wordpress.com*
> >
> >
> >
> >
> > 2012/9/12 Charles Moulliard <ch...@gmail.com>
> >
> > > First we mention in the comment that we will try to use the "WebApp
> > > ClassLoader first" which is also not really true if we use DeltaSpike
> > with
> > > cdictrl
> > > Next, we mention "fallback: Try ClassLoader for ClassUtils (i.e. the >
> > > myfaces.jar lib)
> > >
> > > On Wed, Sep 12, 2012 at 12:53 PM, Romain Manni-Bucau
> > > <rm...@gmail.com>wrote:
> > >
> > > > what's the issue?
> > > >
> > > > *Romain Manni-Bucau*
> > > > *Twitter: @rmannibucau*
> > > > *Blog: http://rmannibucau.wordpress.com*
> > > >
> > > >
> > > >
> > > >
> > > > 2012/9/12 Charles Moulliard <ch...@gmail.com>
> > > >
> > > > > Nevertheless, we should perhaps adapt the code as ClassUtils could
> > also
> > > > be
> > > > > used using JavaSE ....
> > > > >
> > > > > public static Class loadClassForName(String name) throws
> > > > > ClassNotFoundException
> > > > > {
> > > > > try
> > > > > {
> > > > > // Try WebApp ClassLoader first
> > > > > return Class.forName(name, false, // do not initialize
> > for
> > > > > faster startup
> > > > > getClassLoader(null));
> > > > > }
> > > > > catch (ClassNotFoundException ignore)
> > > > > {
> > > > > // fallback: Try ClassLoader for ClassUtils (i.e. the
> > > > > myfaces.jar lib)
> > > > > return Class.forName(name, false, // do not initialize
> > for
> > > > > faster startup
> > > > > ClassUtils.class.getClassLoader());
> > > > > }
> > > > > }
> > > > >
> > > > >
> > > > >
> > > > > On Wed, Sep 12, 2012 at 11:59 AM, Mark Struberg <struberg@yahoo.de
> >
> > > > wrote:
> > > > >
> > > > > > Sure, go on!
> > > > > >
> > > > > > LieGrue,
> > > > > > strub
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > ----- Original Message -----
> > > > > > > From: Charles Moulliard <ch...@gmail.com>
> > > > > > > To: deltaspike-dev@incubator.apache.org
> > > > > > > Cc:
> > > > > > > Sent: Wednesday, September 12, 2012 10:38 AM
> > > > > > > Subject: Can we remove this comment ?
> > > > > > >
> > > > > > > Hi,
> > > > > > >
> > > > > > > In the ClassUtils of deltaspike core module, there is a mention
> > of
> > > a
> > > > > > > myfaces.jar file, can we remove it ?
> > > > > > >
> > > > > > > /**
> > > > > > > * Loads class for the given name
> > > > > > > * @param name name of the class
> > > > > > > * @return loaded class
> > > > > > > * @throws ClassNotFoundException if the class can't be loadedx@
> > > > > > > */
> > > > > > > public static Class loadClassForName(String name) throws
> > > > > > > ClassNotFoundException
> > > > > > > {
> > > > > > > try
> > > > > > > {
> > > > > > > // Try WebApp ClassLoader first
> > > > > > > return Class.forName(name, false, // do not initialize
> > for
> > > > > faster
> > > > > > > startup
> > > > > > > getClassLoader(null));
> > > > > > > }
> > > > > > > catch (ClassNotFoundException ignore)
> > > > > > > {
> > > > > > > // fallback: Try ClassLoader for ClassUtils (i.e. the
> > > > > myfaces.jar
> > > > > > > lib)
> > > > > > > return Class.forName(name, false, // do not initialize
> > for
> > > > > faster
> > > > > > > startup
> > > > > > > ClassUtils.class.getClassLoader());
> > > > > > > }
> > > > > > > }
> > > > > > >
> > > > > > > Regards,
> > > > > > >
> > > > > > > --
> > > > > > > Charles Moulliard
> > > > > > > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > > > > > > Twitter : @cmoulliard
> > > > > > > Blog : http://cmoulliard.blogspot.com
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Charles Moulliard
> > > > > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > > > > Twitter : @cmoulliard
> > > > > Blog : http://cmoulliard.blogspot.com
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Charles Moulliard
> > > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > > Twitter : @cmoulliard
> > > Blog : http://cmoulliard.blogspot.com
> > >
> >
>
>
>
> --
> Charles Moulliard
> Apache Committer / Sr. Pr. Consultant at FuseSource.com
> Twitter : @cmoulliard
> Blog : http://cmoulliard.blogspot.com
>
Re: Can we remove this comment ?
Posted by Charles Moulliard <ch...@gmail.com>.
No a code issue but clarification to be done in the code to improve it.
On Wed, Sep 12, 2012 at 12:57 PM, Romain Manni-Bucau
<rm...@gmail.com>wrote:
> oh ok thought it was a code issue but just a comment one, right?
>
> *Romain Manni-Bucau*
> *Twitter: @rmannibucau*
> *Blog: http://rmannibucau.wordpress.com*
>
>
>
>
> 2012/9/12 Charles Moulliard <ch...@gmail.com>
>
> > First we mention in the comment that we will try to use the "WebApp
> > ClassLoader first" which is also not really true if we use DeltaSpike
> with
> > cdictrl
> > Next, we mention "fallback: Try ClassLoader for ClassUtils (i.e. the >
> > myfaces.jar lib)
> >
> > On Wed, Sep 12, 2012 at 12:53 PM, Romain Manni-Bucau
> > <rm...@gmail.com>wrote:
> >
> > > what's the issue?
> > >
> > > *Romain Manni-Bucau*
> > > *Twitter: @rmannibucau*
> > > *Blog: http://rmannibucau.wordpress.com*
> > >
> > >
> > >
> > >
> > > 2012/9/12 Charles Moulliard <ch...@gmail.com>
> > >
> > > > Nevertheless, we should perhaps adapt the code as ClassUtils could
> also
> > > be
> > > > used using JavaSE ....
> > > >
> > > > public static Class loadClassForName(String name) throws
> > > > ClassNotFoundException
> > > > {
> > > > try
> > > > {
> > > > // Try WebApp ClassLoader first
> > > > return Class.forName(name, false, // do not initialize
> for
> > > > faster startup
> > > > getClassLoader(null));
> > > > }
> > > > catch (ClassNotFoundException ignore)
> > > > {
> > > > // fallback: Try ClassLoader for ClassUtils (i.e. the
> > > > myfaces.jar lib)
> > > > return Class.forName(name, false, // do not initialize
> for
> > > > faster startup
> > > > ClassUtils.class.getClassLoader());
> > > > }
> > > > }
> > > >
> > > >
> > > >
> > > > On Wed, Sep 12, 2012 at 11:59 AM, Mark Struberg <st...@yahoo.de>
> > > wrote:
> > > >
> > > > > Sure, go on!
> > > > >
> > > > > LieGrue,
> > > > > strub
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > ----- Original Message -----
> > > > > > From: Charles Moulliard <ch...@gmail.com>
> > > > > > To: deltaspike-dev@incubator.apache.org
> > > > > > Cc:
> > > > > > Sent: Wednesday, September 12, 2012 10:38 AM
> > > > > > Subject: Can we remove this comment ?
> > > > > >
> > > > > > Hi,
> > > > > >
> > > > > > In the ClassUtils of deltaspike core module, there is a mention
> of
> > a
> > > > > > myfaces.jar file, can we remove it ?
> > > > > >
> > > > > > /**
> > > > > > * Loads class for the given name
> > > > > > * @param name name of the class
> > > > > > * @return loaded class
> > > > > > * @throws ClassNotFoundException if the class can't be loadedx@
> > > > > > */
> > > > > > public static Class loadClassForName(String name) throws
> > > > > > ClassNotFoundException
> > > > > > {
> > > > > > try
> > > > > > {
> > > > > > // Try WebApp ClassLoader first
> > > > > > return Class.forName(name, false, // do not initialize
> for
> > > > faster
> > > > > > startup
> > > > > > getClassLoader(null));
> > > > > > }
> > > > > > catch (ClassNotFoundException ignore)
> > > > > > {
> > > > > > // fallback: Try ClassLoader for ClassUtils (i.e. the
> > > > myfaces.jar
> > > > > > lib)
> > > > > > return Class.forName(name, false, // do not initialize
> for
> > > > faster
> > > > > > startup
> > > > > > ClassUtils.class.getClassLoader());
> > > > > > }
> > > > > > }
> > > > > >
> > > > > > Regards,
> > > > > >
> > > > > > --
> > > > > > Charles Moulliard
> > > > > > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > > > > > Twitter : @cmoulliard
> > > > > > Blog : http://cmoulliard.blogspot.com
> > > > > >
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > Charles Moulliard
> > > > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > > > Twitter : @cmoulliard
> > > > Blog : http://cmoulliard.blogspot.com
> > > >
> > >
> >
> >
> >
> > --
> > Charles Moulliard
> > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > Twitter : @cmoulliard
> > Blog : http://cmoulliard.blogspot.com
> >
>
--
Charles Moulliard
Apache Committer / Sr. Pr. Consultant at FuseSource.com
Twitter : @cmoulliard
Blog : http://cmoulliard.blogspot.com
Re: Can we remove this comment ?
Posted by Romain Manni-Bucau <rm...@gmail.com>.
oh ok thought it was a code issue but just a comment one, right?
*Romain Manni-Bucau*
*Twitter: @rmannibucau*
*Blog: http://rmannibucau.wordpress.com*
2012/9/12 Charles Moulliard <ch...@gmail.com>
> First we mention in the comment that we will try to use the "WebApp
> ClassLoader first" which is also not really true if we use DeltaSpike with
> cdictrl
> Next, we mention "fallback: Try ClassLoader for ClassUtils (i.e. the >
> myfaces.jar lib)
>
> On Wed, Sep 12, 2012 at 12:53 PM, Romain Manni-Bucau
> <rm...@gmail.com>wrote:
>
> > what's the issue?
> >
> > *Romain Manni-Bucau*
> > *Twitter: @rmannibucau*
> > *Blog: http://rmannibucau.wordpress.com*
> >
> >
> >
> >
> > 2012/9/12 Charles Moulliard <ch...@gmail.com>
> >
> > > Nevertheless, we should perhaps adapt the code as ClassUtils could also
> > be
> > > used using JavaSE ....
> > >
> > > public static Class loadClassForName(String name) throws
> > > ClassNotFoundException
> > > {
> > > try
> > > {
> > > // Try WebApp ClassLoader first
> > > return Class.forName(name, false, // do not initialize for
> > > faster startup
> > > getClassLoader(null));
> > > }
> > > catch (ClassNotFoundException ignore)
> > > {
> > > // fallback: Try ClassLoader for ClassUtils (i.e. the
> > > myfaces.jar lib)
> > > return Class.forName(name, false, // do not initialize for
> > > faster startup
> > > ClassUtils.class.getClassLoader());
> > > }
> > > }
> > >
> > >
> > >
> > > On Wed, Sep 12, 2012 at 11:59 AM, Mark Struberg <st...@yahoo.de>
> > wrote:
> > >
> > > > Sure, go on!
> > > >
> > > > LieGrue,
> > > > strub
> > > >
> > > >
> > > >
> > > >
> > > > ----- Original Message -----
> > > > > From: Charles Moulliard <ch...@gmail.com>
> > > > > To: deltaspike-dev@incubator.apache.org
> > > > > Cc:
> > > > > Sent: Wednesday, September 12, 2012 10:38 AM
> > > > > Subject: Can we remove this comment ?
> > > > >
> > > > > Hi,
> > > > >
> > > > > In the ClassUtils of deltaspike core module, there is a mention of
> a
> > > > > myfaces.jar file, can we remove it ?
> > > > >
> > > > > /**
> > > > > * Loads class for the given name
> > > > > * @param name name of the class
> > > > > * @return loaded class
> > > > > * @throws ClassNotFoundException if the class can't be loadedx@
> > > > > */
> > > > > public static Class loadClassForName(String name) throws
> > > > > ClassNotFoundException
> > > > > {
> > > > > try
> > > > > {
> > > > > // Try WebApp ClassLoader first
> > > > > return Class.forName(name, false, // do not initialize for
> > > faster
> > > > > startup
> > > > > getClassLoader(null));
> > > > > }
> > > > > catch (ClassNotFoundException ignore)
> > > > > {
> > > > > // fallback: Try ClassLoader for ClassUtils (i.e. the
> > > myfaces.jar
> > > > > lib)
> > > > > return Class.forName(name, false, // do not initialize for
> > > faster
> > > > > startup
> > > > > ClassUtils.class.getClassLoader());
> > > > > }
> > > > > }
> > > > >
> > > > > Regards,
> > > > >
> > > > > --
> > > > > Charles Moulliard
> > > > > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > > > > Twitter : @cmoulliard
> > > > > Blog : http://cmoulliard.blogspot.com
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Charles Moulliard
> > > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > > Twitter : @cmoulliard
> > > Blog : http://cmoulliard.blogspot.com
> > >
> >
>
>
>
> --
> Charles Moulliard
> Apache Committer / Sr. Pr. Consultant at FuseSource.com
> Twitter : @cmoulliard
> Blog : http://cmoulliard.blogspot.com
>
Re: Can we remove this comment ?
Posted by Charles Moulliard <ch...@gmail.com>.
First we mention in the comment that we will try to use the "WebApp
ClassLoader first" which is also not really true if we use DeltaSpike with
cdictrl
Next, we mention "fallback: Try ClassLoader for ClassUtils (i.e. the >
myfaces.jar lib)
On Wed, Sep 12, 2012 at 12:53 PM, Romain Manni-Bucau
<rm...@gmail.com>wrote:
> what's the issue?
>
> *Romain Manni-Bucau*
> *Twitter: @rmannibucau*
> *Blog: http://rmannibucau.wordpress.com*
>
>
>
>
> 2012/9/12 Charles Moulliard <ch...@gmail.com>
>
> > Nevertheless, we should perhaps adapt the code as ClassUtils could also
> be
> > used using JavaSE ....
> >
> > public static Class loadClassForName(String name) throws
> > ClassNotFoundException
> > {
> > try
> > {
> > // Try WebApp ClassLoader first
> > return Class.forName(name, false, // do not initialize for
> > faster startup
> > getClassLoader(null));
> > }
> > catch (ClassNotFoundException ignore)
> > {
> > // fallback: Try ClassLoader for ClassUtils (i.e. the
> > myfaces.jar lib)
> > return Class.forName(name, false, // do not initialize for
> > faster startup
> > ClassUtils.class.getClassLoader());
> > }
> > }
> >
> >
> >
> > On Wed, Sep 12, 2012 at 11:59 AM, Mark Struberg <st...@yahoo.de>
> wrote:
> >
> > > Sure, go on!
> > >
> > > LieGrue,
> > > strub
> > >
> > >
> > >
> > >
> > > ----- Original Message -----
> > > > From: Charles Moulliard <ch...@gmail.com>
> > > > To: deltaspike-dev@incubator.apache.org
> > > > Cc:
> > > > Sent: Wednesday, September 12, 2012 10:38 AM
> > > > Subject: Can we remove this comment ?
> > > >
> > > > Hi,
> > > >
> > > > In the ClassUtils of deltaspike core module, there is a mention of a
> > > > myfaces.jar file, can we remove it ?
> > > >
> > > > /**
> > > > * Loads class for the given name
> > > > * @param name name of the class
> > > > * @return loaded class
> > > > * @throws ClassNotFoundException if the class can't be loadedx@
> > > > */
> > > > public static Class loadClassForName(String name) throws
> > > > ClassNotFoundException
> > > > {
> > > > try
> > > > {
> > > > // Try WebApp ClassLoader first
> > > > return Class.forName(name, false, // do not initialize for
> > faster
> > > > startup
> > > > getClassLoader(null));
> > > > }
> > > > catch (ClassNotFoundException ignore)
> > > > {
> > > > // fallback: Try ClassLoader for ClassUtils (i.e. the
> > myfaces.jar
> > > > lib)
> > > > return Class.forName(name, false, // do not initialize for
> > faster
> > > > startup
> > > > ClassUtils.class.getClassLoader());
> > > > }
> > > > }
> > > >
> > > > Regards,
> > > >
> > > > --
> > > > Charles Moulliard
> > > > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > > > Twitter : @cmoulliard
> > > > Blog : http://cmoulliard.blogspot.com
> > > >
> > >
> >
> >
> >
> > --
> > Charles Moulliard
> > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > Twitter : @cmoulliard
> > Blog : http://cmoulliard.blogspot.com
> >
>
--
Charles Moulliard
Apache Committer / Sr. Pr. Consultant at FuseSource.com
Twitter : @cmoulliard
Blog : http://cmoulliard.blogspot.com
Re: Can we remove this comment ?
Posted by Romain Manni-Bucau <rm...@gmail.com>.
what's the issue?
*Romain Manni-Bucau*
*Twitter: @rmannibucau*
*Blog: http://rmannibucau.wordpress.com*
2012/9/12 Charles Moulliard <ch...@gmail.com>
> Nevertheless, we should perhaps adapt the code as ClassUtils could also be
> used using JavaSE ....
>
> public static Class loadClassForName(String name) throws
> ClassNotFoundException
> {
> try
> {
> // Try WebApp ClassLoader first
> return Class.forName(name, false, // do not initialize for
> faster startup
> getClassLoader(null));
> }
> catch (ClassNotFoundException ignore)
> {
> // fallback: Try ClassLoader for ClassUtils (i.e. the
> myfaces.jar lib)
> return Class.forName(name, false, // do not initialize for
> faster startup
> ClassUtils.class.getClassLoader());
> }
> }
>
>
>
> On Wed, Sep 12, 2012 at 11:59 AM, Mark Struberg <st...@yahoo.de> wrote:
>
> > Sure, go on!
> >
> > LieGrue,
> > strub
> >
> >
> >
> >
> > ----- Original Message -----
> > > From: Charles Moulliard <ch...@gmail.com>
> > > To: deltaspike-dev@incubator.apache.org
> > > Cc:
> > > Sent: Wednesday, September 12, 2012 10:38 AM
> > > Subject: Can we remove this comment ?
> > >
> > > Hi,
> > >
> > > In the ClassUtils of deltaspike core module, there is a mention of a
> > > myfaces.jar file, can we remove it ?
> > >
> > > /**
> > > * Loads class for the given name
> > > * @param name name of the class
> > > * @return loaded class
> > > * @throws ClassNotFoundException if the class can't be loadedx@
> > > */
> > > public static Class loadClassForName(String name) throws
> > > ClassNotFoundException
> > > {
> > > try
> > > {
> > > // Try WebApp ClassLoader first
> > > return Class.forName(name, false, // do not initialize for
> faster
> > > startup
> > > getClassLoader(null));
> > > }
> > > catch (ClassNotFoundException ignore)
> > > {
> > > // fallback: Try ClassLoader for ClassUtils (i.e. the
> myfaces.jar
> > > lib)
> > > return Class.forName(name, false, // do not initialize for
> faster
> > > startup
> > > ClassUtils.class.getClassLoader());
> > > }
> > > }
> > >
> > > Regards,
> > >
> > > --
> > > Charles Moulliard
> > > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > > Twitter : @cmoulliard
> > > Blog : http://cmoulliard.blogspot.com
> > >
> >
>
>
>
> --
> Charles Moulliard
> Apache Committer / Sr. Pr. Consultant at FuseSource.com
> Twitter : @cmoulliard
> Blog : http://cmoulliard.blogspot.com
>
Re: Can we remove this comment ?
Posted by Charles Moulliard <ch...@gmail.com>.
Nevertheless, we should perhaps adapt the code as ClassUtils could also be
used using JavaSE ....
public static Class loadClassForName(String name) throws
ClassNotFoundException
{
try
{
// Try WebApp ClassLoader first
return Class.forName(name, false, // do not initialize for
faster startup
getClassLoader(null));
}
catch (ClassNotFoundException ignore)
{
// fallback: Try ClassLoader for ClassUtils (i.e. the
myfaces.jar lib)
return Class.forName(name, false, // do not initialize for
faster startup
ClassUtils.class.getClassLoader());
}
}
On Wed, Sep 12, 2012 at 11:59 AM, Mark Struberg <st...@yahoo.de> wrote:
> Sure, go on!
>
> LieGrue,
> strub
>
>
>
>
> ----- Original Message -----
> > From: Charles Moulliard <ch...@gmail.com>
> > To: deltaspike-dev@incubator.apache.org
> > Cc:
> > Sent: Wednesday, September 12, 2012 10:38 AM
> > Subject: Can we remove this comment ?
> >
> > Hi,
> >
> > In the ClassUtils of deltaspike core module, there is a mention of a
> > myfaces.jar file, can we remove it ?
> >
> > /**
> > * Loads class for the given name
> > * @param name name of the class
> > * @return loaded class
> > * @throws ClassNotFoundException if the class can't be loadedx@
> > */
> > public static Class loadClassForName(String name) throws
> > ClassNotFoundException
> > {
> > try
> > {
> > // Try WebApp ClassLoader first
> > return Class.forName(name, false, // do not initialize for faster
> > startup
> > getClassLoader(null));
> > }
> > catch (ClassNotFoundException ignore)
> > {
> > // fallback: Try ClassLoader for ClassUtils (i.e. the myfaces.jar
> > lib)
> > return Class.forName(name, false, // do not initialize for faster
> > startup
> > ClassUtils.class.getClassLoader());
> > }
> > }
> >
> > Regards,
> >
> > --
> > Charles Moulliard
> > Apache Committer / Sr. Pr. Consultant at FuseSource.com
> > Twitter : @cmoulliard
> > Blog : http://cmoulliard.blogspot.com
> >
>
--
Charles Moulliard
Apache Committer / Sr. Pr. Consultant at FuseSource.com
Twitter : @cmoulliard
Blog : http://cmoulliard.blogspot.com
Re: Can we remove this comment ?
Posted by Mark Struberg <st...@yahoo.de>.
Sure, go on!
LieGrue,
strub
----- Original Message -----
> From: Charles Moulliard <ch...@gmail.com>
> To: deltaspike-dev@incubator.apache.org
> Cc:
> Sent: Wednesday, September 12, 2012 10:38 AM
> Subject: Can we remove this comment ?
>
> Hi,
>
> In the ClassUtils of deltaspike core module, there is a mention of a
> myfaces.jar file, can we remove it ?
>
> /**
> * Loads class for the given name
> * @param name name of the class
> * @return loaded class
> * @throws ClassNotFoundException if the class can't be loadedx@
> */
> public static Class loadClassForName(String name) throws
> ClassNotFoundException
> {
> try
> {
> // Try WebApp ClassLoader first
> return Class.forName(name, false, // do not initialize for faster
> startup
> getClassLoader(null));
> }
> catch (ClassNotFoundException ignore)
> {
> // fallback: Try ClassLoader for ClassUtils (i.e. the myfaces.jar
> lib)
> return Class.forName(name, false, // do not initialize for faster
> startup
> ClassUtils.class.getClassLoader());
> }
> }
>
> Regards,
>
> --
> Charles Moulliard
> Apache Committer / Sr. Pr. Consultant at FuseSource.com
> Twitter : @cmoulliard
> Blog : http://cmoulliard.blogspot.com
>