Author: woolfel Date: Mon Nov 14 06:20:48 2005 New Revision: 344127 URL: http://svn.apache.org/viewcvs?rev=344127&view=rev Log: thanks to sebb for catching the two bugs. it's fixed now.
peter Modified: jakarta/jmeter/branches/rel-2-1/src/reports/org/apache/jmeter/testelement/JTLData.java Modified: jakarta/jmeter/branches/rel-2-1/src/reports/org/apache/jmeter/testelement/JTLData.java URL: http://svn.apache.org/viewcvs/jakarta/jmeter/branches/rel-2-1/src/reports/org/apache/jmeter/testelement/JTLData.java?rev=344127&r1=344126&r2=344127&view=diff ============================================================================== --- jakarta/jmeter/branches/rel-2-1/src/reports/org/apache/jmeter/testelement/JTLData.java (original) +++ jakarta/jmeter/branches/rel-2-1/src/reports/org/apache/jmeter/testelement/JTLData.java Mon Nov 14 06:20:48 2005 @@ -87,7 +87,7 @@ if (itr.hasNext()) { SamplingStatCalculator row = (SamplingStatCalculator)itr.next(); if (row != null) { - items.add(items); + items.add(row); } } return items; @@ -153,11 +153,14 @@ rc.setListener(this); rc.clear(); rc.setListener(null); - rc = null; try { rc.loadExistingFile(); } catch (IOException e) { log.warn(e.getMessage()); + } finally { + // we clean up the ResultCollector to make sure there's + // no slow leaks + rc = null; } } } --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]