Remove resolved TODOs

Tyriar · Tyriar · commit af056cd159a0 · 2020-06-12T11:33:32.000-07:00
diff --git a/src/vs/workbench/contrib/terminal/node/terminalProcess.ts b/src/vs/workbench/contrib/terminal/node/terminalProcess.ts
@@ -78,19 +78,13 @@ export class TerminalProcess extends Disposable implements ITerminalChildProcess
 		const results = await Promise.all([this._validateCwd(), this._validateExecutable()]);
 		const firstError = results.find(r => r !== undefined);
 		if (firstError) {
-			// TODO: Anything else we need to do here to tear down the terminal instance?
-			// this._processStartupComplete = Promise.resolve(undefined);
-			// this._onProcessExit.fire(0);
-
 			return firstError;
 		}
 
 		try {
 			this.setupPtyProcess(this._shellLaunchConfig, this._ptyOptions);
 			return undefined;
 		} catch (err) {
-			// TODO: Anything else we need to do here to tear down the terminal instance?
-			// A native exception occurred
 			this._logService.trace('IPty#spawn native exception', err);
 			return { message: `A native exception occurred during launch (${err.message})` };
 		}
