diff options
| author | Boris Kolpackov <boris@codesynthesis.com> | 2024-02-07 13:22:59 +0200 |
|---|---|---|
| committer | Boris Kolpackov <boris@codesynthesis.com> | 2024-02-07 15:02:57 +0200 |
| commit | adc9a34ee759e31dc1f7f9b98629042be3819815 (patch) | |
| tree | 7feef9c4305fda1020bc9b915dde721a29b69d29 /libbuild2/variable.cxx | |
| parent | fcc239ecdbd1467a4ac8b17a353e1b0ae7fd63a0 (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.cxx | 5 |
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); } |