1 files changed, 9 insertions, 47 deletions

diff --git a/source/core/windows/slang-win-process-util.cpp b/source/core/windows/slang-win-process-util.cpp
index 680864acb..ac6468109 100644
--- a/source/core/windows/slang-win-process-util.cpp
+++ b/source/core/windows/slang-win-process-util.cpp
@@ -154,35 +154,7 @@ static DWORD WINAPI _readerThreadProc(LPVOID threadParam)
     return UnownedStringSlice::fromLiteral(".exe");
 }
 
-static bool _isLetter(char c)
-{
-    return (c >= 'a' && c <= 'z') || (c >= 'A' && c <= 'Z');
-}
-
-/* Special case handling for command line switches of the form /LINKPATH: that it is immediately followed
-potentially by a path. This detects that kind of switch, and returns the index of the first character after
-the :. If doesn't have the right format returns -1, to indicate that it's not this style of switch */
-static int _calcIndexOfColonSwitch(const UnownedStringSlice& slice)
-{
-    if (!slice.startsWith("/"))
-    {
-        return -1;
-    }
-    int index = slice.indexOf(':');
-    if (index > 0)
-    {
-        for (int i = 1; i < index; ++i)
-        {
-            if (!_isLetter(slice[i]))
-            {
-                return -1;
-            }
-        }
-    }
-    return index + 1;
-}
-
-static void _appendCommandLineEscaped(const UnownedStringSlice& slice, StringBuilder& out)
+/* static */void ProcessUtil::appendCommandLineEscaped(const UnownedStringSlice& slice, StringBuilder& out)
 {
     // TODO(JS): This escaping is not complete... !
 
@@ -218,23 +190,6 @@ static void _appendCommandLineEscaped(const UnownedStringSlice& slice, StringBui
     }
 }
 
-/* static */void ProcessUtil::appendCommandLineEscaped(const UnownedStringSlice& slice, StringBuilder& out)
-{
-    // Check if starts with a colon ending switch, as we need to special case escaping in this case
-    const int index = _calcIndexOfColonSwitch(slice);
-    if (index >= 0)
-    {
-        // Append the switch prefix as is (ie /linkpath:)
-        out << UnownedStringSlice(slice.begin(), index);
-        // Append the stuff after the prefix escaping if needed
-        _appendCommandLineEscaped(UnownedStringSlice(slice.begin() + index, slice.end()), out);
-    }
-    else
-    {
-        _appendCommandLineEscaped(slice, out);
-    } 
-}
-
 /* static */String ProcessUtil::getCommandLineString(const CommandLine& commandLine)
 {
     StringBuilder cmd;
@@ -242,7 +197,14 @@ static void _appendCommandLineEscaped(const UnownedStringSlice& slice, StringBui
     for (const auto& arg : commandLine.m_args)
     {
         cmd << " ";
-        appendCommandLineEscaped(arg.getUnownedSlice(), cmd);
+        if (arg.type == CommandLine::ArgType::Unescaped)
+        {
+            appendCommandLineEscaped(arg.value.getUnownedSlice(), cmd);
+        }
+        else
+        {
+            cmd << arg.value;
+        }
     }
     return cmd.ToString();
 }