From 9cacec8e78fb359f7148b7848cd0b305bdc80732 Mon Sep 17 00:00:00 2001 From: Roman Lebedev Date: Thu, 18 Oct 2018 15:03:17 +0300 Subject: [PATCH] [NFC] RunBenchmarks(): s/has_repetitions/might_have_aggregates/ (#707) That is the real purpose of that bool. A follow-up change will make it consider something else other than repetitions. --- src/benchmark.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/benchmark.cc b/src/benchmark.cc index 67b6c7a7..a22967b7 100644 --- a/src/benchmark.cc +++ b/src/benchmark.cc @@ -228,18 +228,18 @@ void RunBenchmarks(const std::vector& benchmarks, CHECK(display_reporter != nullptr); // Determine the width of the name field using a minimum width of 10. - bool has_repetitions = FLAGS_benchmark_repetitions > 1; + bool might_have_aggregates = FLAGS_benchmark_repetitions > 1; size_t name_field_width = 10; size_t stat_field_width = 0; for (const BenchmarkInstance& benchmark : benchmarks) { name_field_width = std::max(name_field_width, benchmark.name.size()); - has_repetitions |= benchmark.repetitions > 1; + might_have_aggregates |= benchmark.repetitions > 1; for (const auto& Stat : *benchmark.statistics) stat_field_width = std::max(stat_field_width, Stat.name_.size()); } - if (has_repetitions) name_field_width += 1 + stat_field_width; + if (might_have_aggregates) name_field_width += 1 + stat_field_width; // Print header here BenchmarkReporter::Context context;