write XML output file only once

Christian Thalinger twisti@complang.tuwien.ac.at
Thu May 15 10:36:00 GMT 2008


Hi!

This patch only writes the XML output file once after the Harness sends
"_dump_data_" and not after each finished testlet.  This can be a huge
amount of data writes when running e.g. on java.util.

OK to commit?

- twisti

---

Index: RunnerProcess.java
===================================================================
RCS file: /cvs/mauve/mauve/RunnerProcess.java,v
retrieving revision 1.16
diff -u -3 -p -r1.16 RunnerProcess.java
--- RunnerProcess.java  26 Nov 2006 23:12:41 -0000      1.16
+++ RunnerProcess.java  15 May 2008 10:33:20 -0000
@@ -214,6 +214,21 @@ public class RunnerProcess
             System.exit(0);
           if (testname.equals("_dump_data_"))
             {              
+              // Print the report if necessary.
+              if (report != null)
+                {
+                  File f = new File(xmlfile);
+                  try
+                    {
+                      report.writeXml(f);
+                    }
+                  catch (IOException e)
+                    {
+                      throw new Error("Failed to write data to xml file: "
+                                      + e.getMessage());
+                    }
+                }
+
               if (useEMMA)
                 dumpCoverageData();
               else
@@ -433,20 +448,6 @@ public class RunnerProcess
     
     // Print out a summary.
     int temp = harness.done();
-    // Print the report if necessary.
-    if (report != null)
-      {
-        File f = new File(xmlfile);
-        try
-          {
-            report.writeXml(f);
-          }
-        catch (IOException e)
-          {
-            throw new Error("Failed to write data to xml file: "
-                            + e.getMessage());
-          }
-      }
     
     // Report back to Harness that we've finished properly, whether the test
     // passed or failed.  Harness will wait for a message starting with 




More information about the Mauve-patches mailing list