From 9ca42d5da20cbbb76cd2f35f2a442d70505bf862 Mon Sep 17 00:00:00 2001
From: Robert Hensing <robert@roberthensing.nl>
Date: Sat, 13 Jul 2024 01:43:11 +0200
Subject: [PATCH] dropEmptyInitThenConcatStringsSep -> concatStringSep: setting
 value was already harmed

Considering that `value` was probably parsed with tokenizeString
prior, it's unlikely to contain empty strings, and we have no
reason to remove them either.
---
 src/libutil/config.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/libutil/config.cc b/src/libutil/config.cc
index 6d929b7f7..726e5091e 100644
--- a/src/libutil/config.cc
+++ b/src/libutil/config.cc
@@ -320,7 +320,7 @@ template<> void BaseSetting<Strings>::appendOrSet(Strings newValue, bool append)
 
 template<> std::string BaseSetting<Strings>::to_string() const
 {
-    return dropEmptyInitThenConcatStringsSep(" ", value);
+    return concatStringsSep(" ", value);
 }
 
 template<> StringSet BaseSetting<StringSet>::parse(const std::string & str) const
@@ -336,7 +336,7 @@ template<> void BaseSetting<StringSet>::appendOrSet(StringSet newValue, bool app
 
 template<> std::string BaseSetting<StringSet>::to_string() const
 {
-    return dropEmptyInitThenConcatStringsSep(" ", value);
+    return concatStringsSep(" ", value);
 }
 
 template<> std::set<ExperimentalFeature> BaseSetting<std::set<ExperimentalFeature>>::parse(const std::string & str) const