commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brett Porter <br...@apache.org>
Subject Re: cvs commit: jakarta-commons/jelly/src/java/org/apache/commons/jelly/util TagUtils.java
Date Tue, 28 Dec 2004 09:13:54 GMT
Paul Libbrecht wrote:

> Mmmmh....
> can you document more your platform and VM...


Windows, Sun Java 1.4.2_06

> This test is based on freemem() and that makes, of course, the 
> assertion quite heuristic... if we all provide numbers for each 
> platforms, maybe we find a way to adjust the heuristic ?
>
> Also, maybe it should be renamed as OptionalTestCoreMemoryLeak and 
> only run when one wants ? (it is very slow anyways).

I think this is a good idea.

- Brett

>
> paul
>
>
> Le 28 déc. 04, à 02:48, Brett Porter a écrit :
>
>> The test is failing for me. Is that to be expected?
>> polx@apache.org wrote:
>>
>>>  Added:       jelly/src/test/org/apache/commons/jelly/core
>>>                        TestCoreMemoryLeak.java BaseMemoryLeakTest.java
>>>  Removed:     jelly/src/java/org/apache/commons/jelly/impl
>>>                        WeakReferenceWrapperScript.java
>>>  Log:
>>>  Tag caching is now done in the JellyContext.
>>>  The TestCoreMemoryLeak shows that, at least with some usage of 
>>> JellyContext.clear(), one avoids leaks.
>>>  Removing Tag-caching flag. Also, removing the 
>>> WeakReferenceWrapperScript, both were in the era of ThreadLocal 
>>> which is to be finished.
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
>
>


---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message