From 71447032738895248960a4662d7ef7026671a1c0 Mon Sep 17 00:00:00 2001 From: Aasim Khan Date: Fri, 28 Jul 2023 20:52:18 +0000 Subject: [PATCH] Commenting out listener threshold to fix extensions having mem leaks registering more listeners. (#24012) * Commenting out listener threshold * Adding SQL carbon edit --- src/vs/base/common/event.ts | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/vs/base/common/event.ts b/src/vs/base/common/event.ts index 92dc7c4e50..b6a178c82b 100644 --- a/src/vs/base/common/event.ts +++ b/src/vs/base/common/event.ts @@ -947,10 +947,11 @@ export class Emitter { this._listeners = new LinkedList(); } - if (this._leakageMon && this._listeners.size > this._leakageMon.threshold * 3) { - console.warn(`[${this._leakageMon.name}] REFUSES to accept new listeners because it exceeded its threshold by far`); - return Disposable.None; - } + // {{SQL CARBON EDIT}} Disabling threshold to unblock model view components to register more listeners. Due to the threshold, the components were not able to register listeners and were not able to handle click events. + // if (this._leakageMon && this._listeners.size > this._leakageMon.threshold * 3) { + // console.warn(`[${this._leakageMon.name}] REFUSES to accept new listeners because it exceeded its threshold by far`); + // return Disposable.None; + // } const firstListener = this._listeners.isEmpty();