On Tue, 07 Jun 2005 17:13:01 +0200, Nathan Bubna <[EMAIL PROTECTED]> wrote:
patches would help :)
yes - but before doing so just wanted to know if there were any argument
for
how it is done now ,)
-max
On 6/7/05, Max Rydahl Andersen <[EMAIL PROTECTED]> wrote:
On Mon, 06 Jun 2005 20:17:38 +0200, Shinobu Kawai
<[EMAIL PROTECTED]> wrote:
> Hi max,
>
>> Any particular reason why velocity only do Class.forName() where
>> it should first look in thread context classloader and THEN do
>> Class.forName() ?
>>
>> (it gives some issues when running in environments with changing
>> classloaders, like
>> J2EE and Eclipse)
>
> Are we talking about the ClasspathResourceLoader? Then it's fixed in
> the latest repository.
> http://issues.apache.org/bugzilla/show_bug.cgi?id=22419
> For the latest in the repository:
> http://cvs.apache.org/snapshots/velocity/
I'm talking about
FieldMethodizer
RuntimeInstance
LogManager
ResourceManagerImpl
ResourceLoaderFactory
Generator
While on the subject of things that "messes" up J2EE/Eclipse
environments.
Why is so much stuff logged as WARN/ERROR in Velocity ?
Like:
"ERROR: ResourceManager : unable to find resource 'VM_global_library.vm'
in any resource loader."
That should at least only be a WARN and it should be in a
ResourceManager
category so you could disable it.
And:
"Warning! The iterative is an Iterator in the #foreach() loop at [0,0]
in
template pojo/javaclass.vm. Because it's not resetable, if used in more
than once, this may lead to unexpected results."
Again, I undestand why this is there - but it should only be done on
objects that is directly in the context (if at all). Printing out this
warning
for every iterator is simply not very usefull... e.g. printing this
warning for a methodcall that does someList.iterator() is just noise.
I removed this last one by providing my own UberSpectImpl, but it should
not be necessary IMO.
/max
>
> Best regards,
> -- Shinobu
>
> --
> Shinobu Kawai <[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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
