At least a quick summary of caching would be that: if caching is
disabled, the tag object is renewed at the beginning of every new run.
(where I understood disabled caching would imply the tag would
disappear much earlier).
Maybe that's a simple formulation...
paul
Le 15 août 05, à 02:49, Hans Gilde a écrit :
Yes, you're running into a fundamental problem: there's no difference
between the "cache" and the mechanism that tags use to look up their
parents.
The interface solution will be a partial fix, because tags that are
never
looked up by their children, never need caching.
-----Original Message-----
From: Paul Libbrecht [mailto:[EMAIL PROTECTED]
Sent: Saturday, August 13, 2005 6:31 PM
To: Jakarta Commons Developers List
Subject: Re: [jelly] 1.0-final problem wrt tag caching
Indeed... it is running by me as well but I am fully puzzled.
I had added, after your commit, in my version the following which I
believed was innocent... it wasn't: In TagScript.java, replace
Tag tag = (Tag) threadLocalTagCache.get(t);
with
Tag tag = null;
if(context.isCacheTags())
tag = (Tag) threadLocalTagCache.get(t);
Indeed, otherwise, the tag is only cleared at the start of next run.
Well, precisely this change made everything fail by me.
I'll have to evaluate further for more tags-that-wish-caching but for
now... fine!
paul
Le 13 août 05, à 18:25, peter royal a écrit :
On Aug 10, 2005, at 6:14 PM, Paul Libbrecht wrote:
I'm annoyed... I don't obtain the same results as you do.
With the default cacheTags value as false, I obtain only your test
failing.
With the default cacheTags value as true, I obtain many failed tests.
Did you not say that with your commit, all jelly-core tests were
passing ?
If not then we really need to push the NeedCaching and RefusesCaching
interfaces. Right ?
yes, with the change I made, all the tests in jelly-core were passing.
can you give an example or two of a test that's failing for you?
-pete
--
peter royal
---------------------------------------------------------------------
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]