properly quote empty arguments; fixes microsoft#25098

weinand · weinand · commit fd05938a6d6c · 2019-06-12T15:57:59.000+02:00
diff --git a/src/vs/workbench/contrib/debug/node/terminals.ts b/src/vs/workbench/contrib/debug/node/terminals.ts
@@ -380,7 +380,7 @@ export function prepareCommand(args: DebugProtocol.RunInTerminalRequestArguments
 
 			quote = (s: string) => {
 				s = s.replace(/\"/g, '""');
-				return (s.indexOf(' ') >= 0 || s.indexOf('"') >= 0) ? `"${s}"` : s;
+				return (s.indexOf(' ') >= 0 || s.indexOf('"') >= 0 || s.length === 0) ? `"${s}"` : s;
 			};
 
 			if (args.cwd) {
@@ -410,7 +410,7 @@ export function prepareCommand(args: DebugProtocol.RunInTerminalRequestArguments
 
 			quote = (s: string) => {
 				s = s.replace(/([\"\\])/g, '\\$1');
-				return s.indexOf(' ') >= 0 ? `"${s}"` : s;
+				return (s.indexOf(' ') >= 0 || s.length === 0) ? `"${s}"` : s;
 			};
 
 			const hardQuote = (s: string) => {
