hgilde 2005/01/19 21:19:04
Modified: jelly/src/test/org/apache/commons/jelly/core
BaseMemoryLeakTest.java
Log:
PR: JELLY-166
Submitted by: Hans Gilde
Revision Changes Path
1.3 +5 -1
jakarta-commons/jelly/src/test/org/apache/commons/jelly/core/BaseMemoryLeakTest.java
Index: BaseMemoryLeakTest.java
===================================================================
RCS file:
/home/cvs/jakarta-commons/jelly/src/test/org/apache/commons/jelly/core/BaseMemoryLeakTest.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- BaseMemoryLeakTest.java 28 Dec 2004 01:46:36 -0000 1.2
+++ BaseMemoryLeakTest.java 20 Jan 2005 05:19:04 -0000 1.3
@@ -117,6 +117,10 @@
// PL: I don't see why but removing the clear here
// does make the test fail!
// As if the WeakHashMap wasn't weak enough...
+
+ //Hans: The structure of the relationship
+ // between TagScript and Tag prevents WeakHashMap
+ // from working in this case, which is why I removed it.
jc.clear();
if (outputEveryXIterations != 0 && i % outputEveryXIterations ==
0) {
@@ -126,7 +130,7 @@
rt.runFinalization();
rt.gc();
long middle = rt.totalMemory() - rt.freeMemory();
- log.info("TagHolderMap has " + jc.getTagHolderMap().size() +
" entries.");
+ log.info("TagHolderMap has " +
jc.getThreadScriptDataMap().size() + " entries.");
log.info("Memory test after " + i + " runs: "
+ (middle - start));
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
