diff options
author | Nico Weber <nicolasweber@gmx.de> | 2012-09-14 13:37:31 (GMT) |
---|---|---|
committer | Nico Weber <nicolasweber@gmx.de> | 2012-09-14 13:48:48 (GMT) |
commit | 48143c9b07eca30c99d47cadf3c0ae111b369f1b (patch) | |
tree | 664f292f0ca0797ecb02a5c7df0f5158eb36f51d /src/build.cc | |
parent | 06fa62352d1e9868409b299ffc8abc8f4cd9a39d (diff) | |
download | Ninja-48143c9b07eca30c99d47cadf3c0ae111b369f1b.zip Ninja-48143c9b07eca30c99d47cadf3c0ae111b369f1b.tar.gz Ninja-48143c9b07eca30c99d47cadf3c0ae111b369f1b.tar.bz2 |
Change rate measurement code.
For %o, remove a superfluous + 0.5: snprintf("%f") rounds already.
Remove some unnecessary code.
For %c, fix a TODO to add a sliding window and update after every
completed edge. Else, with -j50 and several files that take 3s to
compile each, this number would only update every 150s. Also give
the number one decimal place so that this can measure steps slower
than 1s.
Diffstat (limited to 'src/build.cc')
-rw-r--r-- | src/build.cc | 24 |
1 files changed, 9 insertions, 15 deletions
diff --git a/src/build.cc b/src/build.cc index e1aaad1..5a0c3b3 100644 --- a/src/build.cc +++ b/src/build.cc @@ -42,8 +42,7 @@ BuildStatus::BuildStatus(const BuildConfig& config) start_time_millis_(GetTimeMillis()), started_edges_(0), finished_edges_(0), total_edges_(0), have_blank_line_(true), progress_status_format_(NULL), - overall_rate_(), current_rate_(), - current_rate_average_count_(config.parallelism) { + overall_rate_(), current_rate_(config.parallelism) { #ifndef _WIN32 const char* term = getenv("TERM"); smart_terminal_ = isatty(1) && term && string(term) != "dumb"; @@ -136,7 +135,8 @@ void BuildStatus::BuildFinished() { printf("\n"); } -string BuildStatus::FormatProgressStatus(const char* progress_status_format) const { +string BuildStatus::FormatProgressStatus( + const char* progress_status_format) const { string out; char buf[32]; for (const char* s = progress_status_format; *s != '\0'; ++s) { @@ -177,30 +177,24 @@ string BuildStatus::FormatProgressStatus(const char* progress_status_format) con out += buf; break; - // Overall finished edges per second. + // Overall finished edges per second. case 'o': - overall_rate_.UpdateRate(finished_edges_, finished_edges_); + overall_rate_.UpdateRate(finished_edges_); overall_rate_.snprinfRate(buf, "%.1f"); out += buf; break; - // Current rate, average over the last '-j' jobs. + // Current rate, average over the last '-j' jobs. case 'c': - // TODO use sliding window? - if (finished_edges_ > current_rate_.last_update() && - finished_edges_ - current_rate_.last_update() == current_rate_average_count_) { - current_rate_.UpdateRate(current_rate_average_count_, finished_edges_); - current_rate_.Restart(); - } - current_rate_.snprinfRate(buf, "%.0f"); + current_rate_.UpdateRate(finished_edges_); + current_rate_.snprinfRate(buf, "%.1f"); out += buf; break; - default: { + default: Fatal("unknown placeholder '%%%c' in $NINJA_STATUS", *s); return ""; } - } } else { out.push_back(*s); } |