Skip to content

Add command line option to print user counters in tabular format #350

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
May 3, 2017
Merged

Add command line option to print user counters in tabular format #350

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
0de985a
Add command line option --benchmark_counters_tabular
biojppm Mar 2, 2017
cbb2eb8
Tabular counters: fix dangling reference to color_output_.
biojppm Mar 2, 2017
c846eed
Fix use of C-style cast.
biojppm Mar 2, 2017
d84d911
Tabular counters: fix missing cast on MSVC.
biojppm Mar 2, 2017
707dd89
User counters: document tabular output in README.md
biojppm Apr 1, 2017
6151517
Merge remote-tracking branch 'upstream/master' into compact
biojppm May 2, 2017
17a012d
Fix: --benchmark_counters_tabular was not being passed to tests.
biojppm May 2, 2017
c69b385
Add first unit test for benchmark_tabular_counters.
biojppm May 2, 2017
cf20dc9
Add test for tabular output of rate counters.
biojppm May 2, 2017
3db6254
Console reporter: add /s prefix to counter rates.
biojppm May 2, 2017
ea019f3
Allow different counter sets in CSV reporting.
biojppm May 2, 2017
2506044
Add unit test for counter sets.
biojppm May 2, 2017
020bac9
Extend tabular counter tests to different counter sets.
biojppm May 2, 2017
a310886
Fix (that is, ignore) clang compile error.
biojppm May 2, 2017
160770f
Fix dropped-style elses.
biojppm May 2, 2017
1735413
Fix pragma clang ignore with gcc.
biojppm May 2, 2017
ec6f035
Trying again to fix error caused by -Wunused-function.
biojppm May 2, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge remote-tracking branch 'upstream/master' into compact
  • Loading branch information
@biojppm
biojppm committed May 2, 2017
commit 615151723e43b4e925732e33ab428b6d9b3f8b43
23 changes: 16 additions & 7 deletions src/console_reporter.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,11 +53,18 @@ bool ConsoleReporter::ReportContext(const Context& context) {
}

void ConsoleReporter::PrintHeader(const Run& run) {
std::string str =
FormatString("%-*s %13s %13s %10s%s\n", static_cast<int>(name_field_width_),
"Benchmark", "Time", "CPU", "Iterations",
(run.counters.empty() ? "" : " UserCounters...")
);
std::string str = FormatString("%-*s %13s %13s %10s\n", static_cast<int>(name_field_width_),
"Benchmark", "Time", "CPU", "Iterations");
if(!run.counters.empty()) {
if(output_options_ & OO_Tabular) {
for(auto const& c : run.counters) {
str += FormatString(" %10s", c.first);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c.first.c_str() i think

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes! :-P

}
}
else {
str += " UserCounters...";
}
}
std::string line = std::string(str.length(), '-');
GetOutputStream() << line << "\n" << str << line << "\n";
}
Expand Down Expand Up @@ -142,10 +149,12 @@ void ConsoleReporter::PrintRunData(const Run& result) {

for (auto& c : result.counters) {
auto const& s = HumanReadableNumber(c.second.value);
const char* unit = (c.second.flags & Counter::kIsRate) ? "/s" : "";
if(output_options_ & OO_Tabular) {
printer(Out, COLOR_DEFAULT, " %10s", s.c_str());
printer(Out, COLOR_DEFAULT, " %10s%s", s.c_str(), unit);
} else {
printer(Out, COLOR_DEFAULT, " %s=%s", c.first.c_str(), s.c_str());
printer(Out, COLOR_DEFAULT, " %s=%s%s", c.first.c_str(), s.c_str(),
unit);
}
}

Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.