You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by hg...@apache.org on 2005/01/20 06:19:04 UTC
cvs commit: jakarta-commons/jelly/src/test/org/apache/commons/jelly/core BaseMemoryLeakTest.java
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: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org