From adc9a34ee759e31dc1f7f9b98629042be3819815 Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Wed, 7 Feb 2024 13:22:59 +0200
Subject: Use reverse to fundamental types semantics in $json.member_value()

Feels like this is an equivalent context to subscript/iteration.
---
 libbuild2/variable.cxx | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

(limited to 'libbuild2/variable.cxx')

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);
   }
-- 
cgit v1.1