From bd32ea47fd8f720b54f81a769dc007d1fdb4e5c1 Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Tue, 23 Aug 2016 09:05:54 +0200
Subject: Reverse empty simple values as an empty names sequence rather than
 empty name

This way we get:

config.import.foo =

Rather than:

config.import.foo = {}
---
 build2/variable.cxx | 27 ++++++++++++++++-----------
 1 file changed, 16 insertions(+), 11 deletions(-)

(limited to 'build2/variable.cxx')

diff --git a/build2/variable.cxx b/build2/variable.cxx
index 7a13618..0134683 100644
--- a/build2/variable.cxx
+++ b/build2/variable.cxx
@@ -643,7 +643,8 @@ namespace build2
   static names_view
   name_reverse (const value& v, names&)
   {
-    return names_view (&v.as<name> (), 1);
+    const name& n (v.as<name> ());
+    return n.empty () ? names_view (nullptr, 0) : names_view (&n, 1);
   }
 
   const char* const value_traits<name>::type_name = "name";
@@ -726,19 +727,23 @@ namespace build2
   static names_view
   process_path_reverse (const value& v, names& s)
   {
-    auto& pp (v.as<process_path> ());
-    s.reserve (pp.effect.empty () ? 1 : 2);
+    const process_path& x (v.as<process_path> ());
 
-    s.push_back (name (pp.recall.directory (),
-                       string (),
-                       pp.recall.leaf ().string ()));
-
-    if (!pp.effect.empty ())
+    if (!x.empty ())
     {
-      s.back ().pair = '@';
-      s.push_back (name (pp.effect.directory (),
+      s.reserve (x.effect.empty () ? 1 : 2);
+
+      s.push_back (name (x.recall.directory (),
                          string (),
-                         pp.effect.leaf ().string ()));
+                         x.recall.leaf ().string ()));
+
+      if (!x.effect.empty ())
+      {
+        s.back ().pair = '@';
+        s.push_back (name (x.effect.directory (),
+                           string (),
+                           x.effect.leaf ().string ()));
+      }
     }
 
     return s;
-- 
cgit v1.1