diff options
| author | Carl Worth <cworth@cworth.org> | 2007-04-26 11:30:27 -0700 |
|---|---|---|
| committer | Carl Worth <cworth@cworth.org> | 2007-04-26 11:30:27 -0700 |
| commit | 050dad71734739f1eaf8976ca5bb88e47c76409c (patch) | |
| tree | 7d718eb13ba273ed6efd90182da56b1b67cba816 /perf/cairo-perf-diff-files.c | |
| parent | 6035d3b47e635f53b361dc7924e161a214e038b3 (diff) | |
| download | cairo-050dad71734739f1eaf8976ca5bb88e47c76409c.tar.gz | |
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.
Diffstat (limited to 'perf/cairo-perf-diff-files.c')
| -rw-r--r-- | perf/cairo-perf-diff-files.c | 12 |
1 files 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"); |