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]

Reply via email to