move listeners outside of onProcessReady block

microsoft · Oct 22, 2021 · 50404fe · 50404fe
1 parent 86fe048
commit 50404fe
Showing 1 changed file with 19 additions and 19 deletions.
diff --git a/src/vs/workbench/contrib/terminal/browser/terminalInstance.ts b/src/vs/workbench/contrib/terminal/browser/terminalInstance.ts
@@ -1275,27 +1275,27 @@ export class TerminalInstance extends Disposable implements ITerminalInstance {
 					});
 				});
 				this.refreshTabLabels(this._shellLaunchConfig.executable, TitleEventSource.Process);
-
-				this._processManager.onProcessData(ev => {
-					this._initialDataEvents?.push(ev.data);
-					this._onData.fire(ev.data);
-				});
-				this._processManager.onEnvironmentVariableInfoChanged(e => this._onEnvironmentVariableInfoChanged(e));
-				this._processManager.onPtyDisconnect(() => {
-					this._safeSetOption('disableStdin', true);
-					this.statusList.add({
-						id: TerminalStatus.Disconnected,
-						severity: Severity.Error,
-						icon: Codicon.debugDisconnect,
-						tooltip: nls.localize('disconnectStatus', "Lost connection to process")
-					});
-				});
-				this._processManager.onPtyReconnect(() => {
-					this._safeSetOption('disableStdin', false);
-					this.statusList.remove(TerminalStatus.Disconnected);
-				});
 			}
 		});
+
+		this._processManager.onProcessData(ev => {
+			this._initialDataEvents?.push(ev.data);
+			this._onData.fire(ev.data);
+		});
+		this._processManager.onEnvironmentVariableInfoChanged(e => this._onEnvironmentVariableInfoChanged(e));
+		this._processManager.onPtyDisconnect(() => {
+			this._safeSetOption('disableStdin', true);
+			this.statusList.add({
+				id: TerminalStatus.Disconnected,
+				severity: Severity.Error,
+				icon: Codicon.debugDisconnect,
+				tooltip: nls.localize('disconnectStatus', "Lost connection to process")
+			});
+		});
+		this._processManager.onPtyReconnect(() => {
+			this._safeSetOption('disableStdin', false);
+			this.statusList.remove(TerminalStatus.Disconnected);
+		});
 	}
 
 	private async _createProcess(): Promise<void> {