diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2016-03-31 10:59:45 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2016-03-31 10:59:45 +0200 |
commit | 7253ffee27f6cae34e63a72b2d3d10db10571ecc (patch) | |
tree | c86b321ad4d2bcb05eda451ad3cae5ac8429cabf /build2/test | |
parent | 6417a4e6af2b7732ec0da6af24f1a56f7cdada3f (diff) |
Clean up variable lookup interfaces
Diffstat (limited to 'build2/test')
-rw-r--r-- | build2/test/rule.cxx | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/build2/test/rule.cxx b/build2/test/rule.cxx index 063d148..2815cf9 100644 --- a/build2/test/rule.cxx +++ b/build2/test/rule.cxx @@ -26,7 +26,7 @@ namespace build2 // this is a test. So take care of that as well. // bool r (false); - lookup<const value> l; + lookup l; // @@ This logic doesn't take into account target type/pattern- // specific variables. @@ -44,7 +44,7 @@ namespace build2 // if (var.name == "test") { - l = lookup<const value> (val, t); + l = lookup (val, t); break; } @@ -126,6 +126,7 @@ namespace build2 // First check the target-specific vars since they override any // scope ones. // + //@@ OVR auto il (t.vars["test.input"]); auto ol (t.vars["test.output"]); auto rl (t.vars["test.roundtrip"]); @@ -162,12 +163,12 @@ namespace build2 // for (scope* s (&bs); s != nullptr; s = s->parent_scope ()) { - ol = s->vars[on]; + ol = s->vars[on]; //@@ OVR if (!al) // Not overriden at target level by test.arguments? { - il = s->vars[in]; - rl = s->vars[rn]; + il = s->vars[in]; //@@ OVR + rl = s->vars[rn]; //@@ OVR } if (il || ol || rl) @@ -289,7 +290,7 @@ namespace build2 string var ("test."); var += n; - auto l (t.vars[var]); + auto l (t.vars[var]); //@@ OVR if (!l) { |