aboutsummaryrefslogtreecommitdiff
path: root/build2/test
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2016-03-31 10:59:45 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2016-03-31 10:59:45 +0200
commit7253ffee27f6cae34e63a72b2d3d10db10571ecc (patch)
treec86b321ad4d2bcb05eda451ad3cae5ac8429cabf /build2/test
parent6417a4e6af2b7732ec0da6af24f1a56f7cdada3f (diff)
Clean up variable lookup interfaces
Diffstat (limited to 'build2/test')
-rw-r--r--build2/test/rule.cxx13
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)
{