aboutsummaryrefslogtreecommitdiff
path: root/libbuild2/variable.cxx
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2024-02-07 13:22:59 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2024-02-07 15:02:57 +0200
commitadc9a34ee759e31dc1f7f9b98629042be3819815 (patch)
tree7feef9c4305fda1020bc9b915dde721a29b69d29 /libbuild2/variable.cxx
parentfcc239ecdbd1467a4ac8b17a353e1b0ae7fd63a0 (diff)
Use reverse to fundamental types semantics in $json.member_value()
Feels like this is an equivalent context to subscript/iteration.
Diffstat (limited to 'libbuild2/variable.cxx')
-rw-r--r--libbuild2/variable.cxx5
1 files changed, 2 insertions, 3 deletions
diff --git a/libbuild2/variable.cxx b/libbuild2/variable.cxx
index 1c899f9..3db0fd0 100644
--- a/libbuild2/variable.cxx
+++ b/libbuild2/variable.cxx
@@ -1993,11 +1993,11 @@ namespace build2
// opposed to reverse() where it is used to build up values and thus
// needs things to be fully reversible). Once we add type hints, then
// this should become unnecessary and we should be able to just always
- // return json_value.
+ // return json_value. See also $json.member_value() where we do the
+ // same thing.
//
// @@ TODO: split this function into two (index/name) once get rid of this.
//
-#if 1
value r;
switch (jr.type)
{
@@ -2018,7 +2018,6 @@ namespace build2
case json_type::array:
case json_type::object: r = value (move (jr)); break;
}
-#endif
return make_pair (move (r), true);
}