From 050dad71734739f1eaf8976ca5bb88e47c76409c Mon Sep 17 00:00:00 2001 From: Carl Worth Date: Thu, 26 Apr 2007 11:30:27 -0700 Subject: [PATCH] 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. --- perf/cairo-perf-diff-files.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/perf/cairo-perf-diff-files.c b/perf/cairo-perf-diff-files.c index d534fb7d6..adbc63bfe 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_report_t *reports, 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_report_t *reports, 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");