update comment regarding usage of Argument vs ArgumentList

Author: jjw24

Plugins/Flow.Launcher.Plugin.Shell/Main.cs

@@ -194,18 +194,29 @@ private ProcessStartInfo PrepareProcessStartInfo(string command, bool runAsAdmin
 
             ProcessStartInfo info = new()
             {
-                Verb = runAsAdministratorArg, WorkingDirectory = workingDirectory,
+                Verb = runAsAdministratorArg,
+                WorkingDirectory = workingDirectory,
             };
             switch (_settings.Shell)
             {
                 case Shell.Cmd:
                     {
                         info.FileName = "cmd.exe";
                         info.Arguments = $"{(_settings.LeaveShellOpen ? "/k" : "/c")} {command}";
-                        
-                        // ArgumentList may break original shell command separation with quote.
+
+                        //// Use info.Arguments instead of info.ArgumentList to enable user better control over the argument they are writing.
+                        //// Previous code using ArgumentList, commands needed to be seperated correctly:                      
+                        //// Incorrect:
                         // info.ArgumentList.Add(_settings.LeaveShellOpen ? "/k" : "/c");
-                        // info.ArgumentList.Add(command);
+                        // info.ArgumentList.Add(command); //<== info.ArgumentList.Add("mkdir \"c:\\test new\"");
+
+                        //// Correct version should be:
+                        //info.ArgumentList.Add(_settings.LeaveShellOpen ? "/k" : "/c");
+                        //info.ArgumentList.Add("mkdir");
+                        //info.ArgumentList.Add(@"c:\test new");
+
+                        //https://docs.microsoft.com/en-us/dotnet/api/system.diagnostics.processstartinfo.argumentlist?view=net-6.0#remarks
+
                         break;
                     }