Skip to content

Commit

Permalink
Remove code that was never executed.
Browse files Browse the repository at this point in the history
total_time was always -1, because it computed a pointer difference,
not a time difference (should've been |*end_time - *start_time|).
  • Loading branch information
nico committed Jun 2, 2012
1 parent 28196d9 commit 555431a
Showing 1 changed file with 1 addition and 9 deletions.
10 changes: 1 addition & 9 deletions src/build.cc
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,6 @@ void BuildStatus::BuildEdgeFinished(Edge* edge,
RunningEdgeMap::iterator i = running_edges_.find(edge);
*start_time = i->second;
*end_time = (int)(now - start_time_millis_);
int total_time = end_time - start_time;
running_edges_.erase(i);

if (config_.verbosity == BuildConfig::QUIET)
Expand All @@ -93,14 +92,7 @@ void BuildStatus::BuildEdgeFinished(Edge* edge,
if (smart_terminal_)
PrintStatus(edge);

if (success && output.empty()) {
if (!smart_terminal_) {
if (total_time > 5*1000) {
printf("%.1f%% %d/%d\n", finished_edges_ * 100 / (float)total_edges_,
finished_edges_, total_edges_);
}
}
} else {
if (!success || !output.empty()) {
if (smart_terminal_)
printf("\n");

Expand Down

0 comments on commit 555431a

Please sign in to comment.