From 4e0804c920558575a4b3486df1e595445bf67555 Mon Sep 17 00:00:00 2001
From: Robert Hensing <robert@roberthensing.nl>
Date: Sun, 9 Apr 2023 22:42:20 +0200
Subject: Deduplicate string literal rendering, fix 4909

---
 src/libcmd/repl.cc | 20 ++++++--------------
 1 file changed, 6 insertions(+), 14 deletions(-)

(limited to 'src/libcmd/repl.cc')

diff --git a/src/libcmd/repl.cc b/src/libcmd/repl.cc
index 57848a5d3..1366622c7 100644
--- a/src/libcmd/repl.cc
+++ b/src/libcmd/repl.cc
@@ -40,6 +40,7 @@ extern "C" {
 #include "markdown.hh"
 #include "local-fs-store.hh"
 #include "progress-bar.hh"
+#include "value/print.hh"
 
 #if HAVE_BOEHMGC
 #define GC_INCLUDE_NEW
@@ -894,17 +895,6 @@ std::ostream & NixRepl::printValue(std::ostream & str, Value & v, unsigned int m
 }
 
 
-std::ostream & printStringValue(std::ostream & str, const char * string) {
-    str << "\"";
-    for (const char * i = string; *i; i++)
-        if (*i == '\"' || *i == '\\') str << "\\" << *i;
-        else if (*i == '\n') str << "\\n";
-        else if (*i == '\r') str << "\\r";
-        else if (*i == '\t') str << "\\t";
-        else str << *i;
-    str << "\"";
-    return str;
-}
 
 
 // FIXME: lot of cut&paste from Nix's eval.cc.
@@ -922,12 +912,14 @@ std::ostream & NixRepl::printValue(std::ostream & str, Value & v, unsigned int m
         break;
 
     case nBool:
-        str << ANSI_CYAN << (v.boolean ? "true" : "false") << ANSI_NORMAL;
+        str << ANSI_CYAN;
+        printLiteral(str, v.boolean);
+        str << ANSI_NORMAL;
         break;
 
     case nString:
         str << ANSI_WARNING;
-        printStringValue(str, v.string.s);
+        printLiteral(str, v.string.s);
         str << ANSI_NORMAL;
         break;
 
@@ -967,7 +959,7 @@ std::ostream & NixRepl::printValue(std::ostream & str, Value & v, unsigned int m
                 if (isVarName(i.first))
                     str << i.first;
                 else
-                    printStringValue(str, i.first.c_str());
+                    printLiteral(str, i.first);
                 str << " = ";
                 if (seen.count(i.second))
                     str << "«repeated»";
-- 
cgit v1.2.3


From 1e2dd669bcdd8df6cdaac061e035828626906447 Mon Sep 17 00:00:00 2001
From: Robert Hensing <robert@roberthensing.nl>
Date: Sun, 16 Apr 2023 12:56:31 +0200
Subject: printLiteral: Do not overload

---
 src/libcmd/repl.cc | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'src/libcmd/repl.cc')

diff --git a/src/libcmd/repl.cc b/src/libcmd/repl.cc
index 1366622c7..41cf77424 100644
--- a/src/libcmd/repl.cc
+++ b/src/libcmd/repl.cc
@@ -913,13 +913,13 @@ std::ostream & NixRepl::printValue(std::ostream & str, Value & v, unsigned int m
 
     case nBool:
         str << ANSI_CYAN;
-        printLiteral(str, v.boolean);
+        printLiteralBool(str, v.boolean);
         str << ANSI_NORMAL;
         break;
 
     case nString:
         str << ANSI_WARNING;
-        printLiteral(str, v.string.s);
+        printLiteralString(str, v.string.s);
         str << ANSI_NORMAL;
         break;
 
@@ -959,7 +959,7 @@ std::ostream & NixRepl::printValue(std::ostream & str, Value & v, unsigned int m
                 if (isVarName(i.first))
                     str << i.first;
                 else
-                    printLiteral(str, i.first);
+                    printLiteralString(str, i.first);
                 str << " = ";
                 if (seen.count(i.second))
                     str << "«repeated»";
-- 
cgit v1.2.3


From 28a5cdde02964306e7eb443f696c8d5d59ebf9e9 Mon Sep 17 00:00:00 2001
From: Robert Hensing <robert@roberthensing.nl>
Date: Sun, 16 Apr 2023 13:10:45 +0200
Subject: libexpr/value/print.* -> libexpr/print.*

Generalizes the file to sensibly allow printing any part of the
language syntax.
---
 src/libcmd/repl.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src/libcmd/repl.cc')

diff --git a/src/libcmd/repl.cc b/src/libcmd/repl.cc
index 41cf77424..806dce024 100644
--- a/src/libcmd/repl.cc
+++ b/src/libcmd/repl.cc
@@ -40,7 +40,7 @@ extern "C" {
 #include "markdown.hh"
 #include "local-fs-store.hh"
 #include "progress-bar.hh"
-#include "value/print.hh"
+#include "print.hh"
 
 #if HAVE_BOEHMGC
 #define GC_INCLUDE_NEW
-- 
cgit v1.2.3


From b6125772d7d5f82d48873fc93a7f261832154b14 Mon Sep 17 00:00:00 2001
From: Robert Hensing <robert@roberthensing.nl>
Date: Sun, 16 Apr 2023 14:07:35 +0200
Subject: libexpr: Move identifier-like printing to print.cc

---
 src/libcmd/repl.cc | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

(limited to 'src/libcmd/repl.cc')

diff --git a/src/libcmd/repl.cc b/src/libcmd/repl.cc
index 806dce024..80c08bf1c 100644
--- a/src/libcmd/repl.cc
+++ b/src/libcmd/repl.cc
@@ -426,6 +426,7 @@ StringSet NixRepl::completePrefix(const std::string & prefix)
 }
 
 
+// FIXME: DRY and match or use the parser
 static bool isVarName(std::string_view s)
 {
     if (s.size() == 0) return false;
@@ -956,10 +957,7 @@ std::ostream & NixRepl::printValue(std::ostream & str, Value & v, unsigned int m
                 sorted.emplace(state->symbols[i.name], i.value);
 
             for (auto & i : sorted) {
-                if (isVarName(i.first))
-                    str << i.first;
-                else
-                    printLiteralString(str, i.first);
+                printAttributeName(str, i.first);
                 str << " = ";
                 if (seen.count(i.second))
                     str << "«repeated»";
-- 
cgit v1.2.3