From 1601401888d9ee84353d8d66084bb7f03c14dbc6 Mon Sep 17 00:00:00 2001 From: Eric Wheeler Date: Sat, 8 Mar 2025 14:24:54 -0800 Subject: [PATCH] cleanup: remove unused stream property from Terminal @cte reported that the stream property is not used anywhere in the codebase. The stream is passed directly to the process via event emitter and does not need to be stored. Signed-off-by: Eric Wheeler --- src/integrations/terminal/Terminal.ts | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/src/integrations/terminal/Terminal.ts b/src/integrations/terminal/Terminal.ts index 01e6ccfcc8..7233b275b2 100644 --- a/src/integrations/terminal/Terminal.ts +++ b/src/integrations/terminal/Terminal.ts @@ -6,7 +6,6 @@ export class Terminal { public terminal: vscode.Terminal public busy: boolean public id: number - private stream?: AsyncIterable public running: boolean private streamClosed: boolean public process?: TerminalProcess @@ -40,13 +39,6 @@ export class Terminal { } } - /** - * Gets the terminal's stream - */ - public getStream(): AsyncIterable | undefined { - return this.stream - } - /** * Checks if the stream is closed */ @@ -60,8 +52,6 @@ export class Terminal { * @throws Error if process is undefined when a stream is provided */ public setActiveStream(stream: AsyncIterable | undefined): void { - this.stream = stream - if (stream) { // New stream is available if (!this.process) {