Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Removed unused UnitsWaiting() function from State class. #502

Merged
merged 1 commit into from

4 participants

@tfarina

This function was added at 307f0bb("and some basic implementation"), but nobody
calls it anymore.

Signed-off-by: Thiago Farina tfarina@chromium.org

@tfarina tfarina Removed unused UnitsWaiting() function from State class.
This function was added at 307f0bb("and some basic implementation"), but nobody
calls it anymore.

Signed-off-by: Thiago Farina <tfarina@chromium.org>
855f4eb
@buildhive

Evan Martin » ninja #319 SUCCESS
This pull request looks good
(what's this?)

@riannucci

This lgtm. Originally this was going to be used in Pool::Dump, but current_use_ is used instead (which, when non-unit-weight edges are implemented, is a more useful counter). Thanks for the cleanup.

@martine martine merged commit 45e7d32 into martine:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Feb 19, 2013
  1. @tfarina

    Removed unused UnitsWaiting() function from State class.

    tfarina authored
    This function was added at 307f0bb("and some basic implementation"), but nobody
    calls it anymore.
    
    Signed-off-by: Thiago Farina <tfarina@chromium.org>
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 3 deletions.
  1. +1 −3 src/state.h
View
4 src/state.h
@@ -66,9 +66,7 @@ struct Pool {
/// Dump the Pool and its edges (useful for debugging).
void Dump() const;
-private:
- int UnitsWaiting() { return delayed_.size(); }
-
+ private:
string name_;
/// |current_use_| is the total of the weights of the edges which are
Something went wrong with that request. Please try again.