diff options
author | Thiago Farina <tfarina@chromium.org> | 2015-01-10 22:00:43 (GMT) |
---|---|---|
committer | Thiago Farina <tfarina@chromium.org> | 2015-01-10 22:00:43 (GMT) |
commit | 2908829b7c129b6d6f1c04e65893647d2e0e8b36 (patch) | |
tree | a4eb1436c5a46ff3a60d3fee71da7d622b218361 /src/manifest_parser.cc | |
parent | 82e372d5fd09ef37ab1b2f1c1cc175df26648653 (diff) | |
download | Ninja-2908829b7c129b6d6f1c04e65893647d2e0e8b36.zip Ninja-2908829b7c129b6d6f1c04e65893647d2e0e8b36.tar.gz Ninja-2908829b7c129b6d6f1c04e65893647d2e0e8b36.tar.bz2 |
Cleanup: Fix 'hasIdent' variable name/style.
Seems more correct to name it has_indent_token and to use the
unix_hacker style.
Signed-off-by: Thiago Farina <tfarina@chromium.org>
Diffstat (limited to 'src/manifest_parser.cc')
-rw-r--r-- | src/manifest_parser.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/manifest_parser.cc b/src/manifest_parser.cc index 388b5bc..7ee990b 100644 --- a/src/manifest_parser.cc +++ b/src/manifest_parser.cc @@ -298,16 +298,16 @@ bool ManifestParser::ParseEdge(string* err) { return false; // Bindings on edges are rare, so allocate per-edge envs only when needed. - bool hasIdent = lexer_.PeekToken(Lexer::INDENT); - BindingEnv* env = hasIdent ? new BindingEnv(env_) : env_; - while (hasIdent) { + bool has_indent_token = lexer_.PeekToken(Lexer::INDENT); + BindingEnv* env = has_indent_token ? new BindingEnv(env_) : env_; + while (has_indent_token) { string key; EvalString val; if (!ParseLet(&key, &val, err)) return false; env->AddBinding(key, val.Evaluate(env_)); - hasIdent = lexer_.PeekToken(Lexer::INDENT); + has_indent_token = lexer_.PeekToken(Lexer::INDENT); } Edge* edge = state_->AddEdge(rule); |