Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(43)

Unified Diff: tools/gn/parse_tree.cc

Issue 2586073002: Revert GN declare_args() change. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/gn/functions_unittest.cc ('k') | tools/gn/scope.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/gn/parse_tree.cc
diff --git a/tools/gn/parse_tree.cc b/tools/gn/parse_tree.cc
index d1de9cbf3d78b6847557949f04418a12ecc985d4..ecc438b245aee02c62551efaee117796786dc02f 100644
--- a/tools/gn/parse_tree.cc
+++ b/tools/gn/parse_tree.cc
@@ -487,18 +487,13 @@ const IdentifierNode* IdentifierNode::AsIdentifier() const {
}
Value IdentifierNode::Execute(Scope* scope, Err* err) const {
- const Scope* found_in_scope = nullptr;
- const Value* value = scope->GetValueWithScope(value_.value(), true,
- &found_in_scope);
+ const Value* value = scope->GetValue(value_.value(), true);
Value result;
if (!value) {
*err = MakeErrorDescribing("Undefined identifier");
return result;
}
- if (!EnsureNotReadingFromSameDeclareArgs(this, scope, found_in_scope, err))
- return result;
-
result = *value;
result.set_origin(this);
return result;
« no previous file with comments | « tools/gn/functions_unittest.cc ('k') | tools/gn/scope.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698