Nuke Joiner. Add wjoin()

4 files changed, 12 insertions, 36 deletions

diff --git a/toolsrc/src/SourceParagraph.cpp b/toolsrc/src/SourceParagraph.cpp
index 8bfe8e84d..4d144191f 100644
--- a/toolsrc/src/SourceParagraph.cpp
+++ b/toolsrc/src/SourceParagraph.cpp
@@ -53,8 +53,8 @@ namespace vcpkg
             const std::vector<std::string> remaining_fields = Maps::extract_keys(fields);
             const std::vector<std::string>& valid_fields = get_list_of_valid_fields();
 
-            const std::string remaining_fields_as_string = Strings::Joiner::on("\n    ").join(remaining_fields);
-            const std::string valid_fields_as_string = Strings::Joiner::on("\n    ").join(valid_fields);
+            const std::string remaining_fields_as_string = Strings::join("\n    ", remaining_fields);
+            const std::string valid_fields_as_string = Strings::join("\n    ", valid_fields);
 
             System::println(System::color::error, "Error: There are invalid fields in the Source Paragraph of %s", this->name);
             System::println("The following fields were not expected:\n\n    %s\n\n", remaining_fields_as_string);
diff --git a/toolsrc/src/commands_install.cpp b/toolsrc/src/commands_install.cpp
index 24fa5edc0..ad341c847 100644
--- a/toolsrc/src/commands_install.cpp
+++ b/toolsrc/src/commands_install.cpp
@@ -158,7 +158,7 @@ namespace vcpkg::Commands::Install
                             triplet_install_path.generic_string(),
                             binary_paragraph.spec);
             System::print("\n    ");
-            System::println(Strings::Joiner::on("\n    ").join(intersection));
+            System::println(Strings::join("\n    ", intersection));
             System::println("");
             exit(EXIT_FAILURE);
         }
diff --git a/toolsrc/src/commands_remove.cpp b/toolsrc/src/commands_remove.cpp
index 1daacb944..1b7b7923a 100644
--- a/toolsrc/src/commands_remove.cpp
+++ b/toolsrc/src/commands_remove.cpp
@@ -136,17 +136,17 @@ namespace vcpkg::Commands::Remove
         {
             sort_packages_by_name(&not_installed);
             System::println("The following packages are not installed, so not removed:\n%s",
-                            Strings::Joiner::on("\n    ").prefix("    ").join(not_installed, [](const package_spec_with_remove_plan* p)
-                                                                              {
-                                                                                  return p->spec.toString();
-                                                                              }));
+                            Strings::join("\n    ", not_installed, [](const package_spec_with_remove_plan* p)
+                                          {
+                                              return "    " + p->spec.toString();
+                                          }));
         }
 
         if (!remove.empty())
         {
             sort_packages_by_name(&remove);
             System::println("The following packages will be removed:\n%s",
-                            Strings::Joiner::on("\n").join(remove, [](const package_spec_with_remove_plan* p)
+                            Strings::join("\n", remove, [](const package_spec_with_remove_plan* p)
                                                            {
                                                                if (p->plan.request_type == Dependencies::request_type::AUTO_SELECTED)
                                                                {
diff --git a/toolsrc/src/vcpkg_Strings.cpp b/toolsrc/src/vcpkg_Strings.cpp
index a7805f211..044fd3c05 100644
--- a/toolsrc/src/vcpkg_Strings.cpp
+++ b/toolsrc/src/vcpkg_Strings.cpp
@@ -78,38 +78,14 @@ namespace vcpkg::Strings
         return output;
     }
 
-    std::string join(const std::vector<std::string>& v, const std::string& prefix, const std::string& delimiter, const std::string& suffix)
+    std::string join(const std::string& delimiter, const std::vector<std::string>& v)
     {
-        return join(v, prefix, delimiter, suffix, [](const std::string& i) -> std::string
-        {
-            return i;
-        });
-    }
-
-    Joiner Joiner::on(const std::string& delimiter)
-    {
-        return Joiner(delimiter);
-    }
-
-    Joiner& Joiner::prefix(const std::string& prefix)
-    {
-        this->m_prefix = prefix;
-        return *this;
-    }
-
-    Joiner& Joiner::suffix(const std::string& suffix)
-    {
-        this->m_suffix = suffix;
-        return *this;
-    }
-
-    std::string Joiner::join(const std::vector<std::string>& v) const
-    {
-        return Strings::join(v, this->m_prefix, this->m_delimiter, this->m_suffix);
+        return join(delimiter, v, [](const std::string& p) -> const std::string& { return p; });
     }
 
-    Joiner::Joiner(const std::string& delimiter) : m_prefix(""), m_delimiter(delimiter), m_suffix("")
+    std::wstring wjoin(const std::wstring& delimiter, const std::vector<std::wstring>& v)
     {
+        return wjoin(delimiter, v, [](const std::wstring& p) -> const std::wstring&{ return p; });
     }
 
     void trim(std::string* s)