Microsoft/vscode
1
0
Fork 0
mirror of https://github.com/microsoft/vscode.git synced 2025-12-20 02:08:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

use mutable disposable, rm dispose func

Browse Source
This commit is contained in:
meganrogge
2023-09-21 12:16:18 -07:00
parent cd93f51433
commit 95675fae4f
1 changed files with 10 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
src/vs/workbench/contrib/tasks/browser/taskTerminalStatus.ts
View File

@@ -5,7 +5,7 @@
import * as nls from 'vs/nls';
import { Codicon } from 'vs/base/common/codicons';
import { Disposable, IDisposable } from 'vs/base/common/lifecycle';
import { Disposable, IDisposable, MutableDisposable, toDisposable } from 'vs/base/common/lifecycle';
import Severity from 'vs/base/common/severity';
import { AbstractProblemCollector, StartStopProblemCollector } from 'vs/workbench/contrib/tasks/common/problemCollectors';
import { ITaskGeneralEvent, ITaskProcessEndedEvent, ITaskProcessStartedEvent, TaskEventKind, TaskRunType } from 'vs/workbench/contrib/tasks/common/tasks';
@@ -23,7 +23,7 @@ interface ITerminalData {
status: ITerminalStatus;
problemMatcher: AbstractProblemCollector;
taskRunEnded: boolean;
disposeListener?: IDisposable;
disposeListener?: MutableDisposable<IDisposable>;
}
const TASK_TERMINAL_STATUS_ID = 'task_terminal_status';
@@ -50,14 +50,12 @@ export class TaskTerminalStatus extends Disposable {
case TaskEventKind.ProcessEnded: this.eventEnd(event); break;
}
}));
}
public override dispose(): void {
super.dispose();
for (const terminalData of this.terminalMap.values()) {
terminalData.disposeListener?.dispose();
}
this.terminalMap.clear();
this._register(toDisposable(() => {
for (const terminalData of this.terminalMap.values()) {
terminalData.disposeListener?.dispose();
}
this.terminalMap.clear();
}));
}
addTerminal(task: Task, terminal: ITerminalInstance, problemMatcher: AbstractProblemCollector) {
@@ -140,7 +138,8 @@ export class TaskTerminalStatus extends Disposable {
return;
}
if (!terminalData.disposeListener) {
terminalData.disposeListener = terminalData.terminal.onDisposed(() => {
terminalData.disposeListener = this._register(new MutableDisposable());
terminalData.disposeListener.value = terminalData.terminal.onDisposed(() => {
if (!event.terminalId) {
return;
}