[cairo-commit] perf/cairo-perf-diff-files.c

Carl Worth cworth at kemper.freedesktop.org
Thu Apr 26 11:30:37 PDT 2007


 perf/cairo-perf-diff-files.c |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

New commits:
diff-tree 050dad71734739f1eaf8976ca5bb88e47c76409c (from 6035d3b47e635f53b361dc7924e161a214e038b3)
Author: Carl Worth <cworth at cworth.org>
Date:   Thu Apr 26 11:30:27 2007 -0700

    cairo-perf-diff-files: Always print old and new configuration names
    
    Previously, if the change in the first test case was small enough
    to be considered insignificant, then the header lines showing the
    names of the old and new configurations would be omitted. This
    commit fixes that bug.

diff --git a/perf/cairo-perf-diff-files.c b/perf/cairo-perf-diff-files.c
index d534fb7..adbc63b 100644
--- a/perf/cairo-perf-diff-files.c
+++ b/perf/cairo-perf-diff-files.c
@@ -698,6 +698,12 @@ cairo_perf_reports_compare (cairo_perf_r
 	    max_change = fabs (diffs[i].change);
     }
 
+    if (num_reports == 2 )
+	printf ("old: %s\n"
+		"new: %s\n",
+		diffs->tests[0]->configuration,
+		diffs->tests[1]->configuration);
+
     for (i = 0; i < num_diffs; i++) {
 	diff = &diffs[i];
 
@@ -708,12 +714,6 @@ cairo_perf_reports_compare (cairo_perf_r
 	    continue;
 
 	if (num_reports == 2) {
-	    if (i == 0) {
-		printf ("old: %s\n"
-			"new: %s\n",
-			diffs->tests[0]->configuration,
-			diffs->tests[1]->configuration);
-	    }
 	    if (diff->change > 1.0 && ! printed_speedup) {
 		printf ("Speedups\n"
 			"========\n");


More information about the cairo-commit mailing list