From 73386de65fd49e7690da5203074a5ea603551db6 Mon Sep 17 00:00:00 2001 From: Boris Kolpackov Date: Wed, 21 Feb 2024 14:21:23 +0200 Subject: Fix issue with json null representation in containers --- libbuild2/variable.cxx | 11 +++++++++-- libbuild2/variable.txx | 4 ++++ 2 files changed, 13 insertions(+), 2 deletions(-) (limited to 'libbuild2') diff --git a/libbuild2/variable.cxx b/libbuild2/variable.cxx index ab65237..078c13a 100644 --- a/libbuild2/variable.cxx +++ b/libbuild2/variable.cxx @@ -1799,9 +1799,16 @@ namespace build2 { case json_type::null: { - // Return empty to be consistent with other places. + // Note that here we cannot return empty (e.g., to be consistent with + // other places) because we treat empty name (as opposed to empty + // names) as string, not null (see to_json_value() above). // -#if 0 + // Thankfully this version of reverse() is only used when json_value + // representation is needed as part of a container. Which means in + // "consumption" contexts (e.g., result of subscript) null will still + // decay to empty. + // +#if 1 return name ("null"); #else return name (); diff --git a/libbuild2/variable.txx b/libbuild2/variable.txx index 12a2667..0b831e9 100644 --- a/libbuild2/variable.txx +++ b/libbuild2/variable.txx @@ -1347,6 +1347,10 @@ namespace build2 // value), but we don't yet have pair<> as value type so we let the generic // implementation return an untyped pair. // + // BTW, one negative consequence of returning untyped pair is that + // $first()/$second() don't return types values either, which is quite + // unfortunate for something like json_map. + // template const map_value_type value_traits>::value_type = build2::value_type // VC14 wants = -- cgit v1.1