Fix _activeLinks not being disposed of

Looks like we were calling forEach but not actually invoking `dispose` on the items. Switch to use `dispose(...)` to avoid this

Author: mjbvz

src/vs/workbench/contrib/terminal/browser/links/terminalBaseLinkProvider.ts

@@ -3,14 +3,17 @@
  *  Licensed under the MIT License. See License.txt in the project root for license information.
  *--------------------------------------------------------------------------------------------*/
 
-import type { ILinkProvider } from 'xterm';
+import { dispose } from 'vs/base/common/lifecycle';
 import { TerminalLink } from 'vs/workbench/contrib/terminal/browser/links/terminalLink';
+import type { ILinkProvider } from 'xterm';
 
 export abstract class TerminalBaseLinkProvider implements ILinkProvider {
 	private _activeLinks: TerminalLink[] | undefined;
 
 	async provideLinks(bufferLineNumber: number, callback: (links: TerminalLink[] | undefined) => void): Promise<void> {
-		this._activeLinks?.forEach(l => l.dispose);
+		if (this._activeLinks) {
+			dispose(this._activeLinks);
+		}
 		this._activeLinks = await this._provideLinks(bufferLineNumber);
 		callback(this._activeLinks);
 	}