From 7253ffee27f6cae34e63a72b2d3d10db10571ecc Mon Sep 17 00:00:00 2001 From: Boris Kolpackov Date: Thu, 31 Mar 2016 10:59:45 +0200 Subject: Clean up variable lookup interfaces --- build2/test/rule.cxx | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) (limited to 'build2/test') 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 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 (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) { -- cgit v1.1