events: remove redundant todo

gurgunday · islandryu · commit 8299e2e22475 · 2026-01-25T10:22:12.000+09:00
PR-URL: nodejs#60595 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
diff --git a/lib/internal/event_target.js b/lib/internal/event_target.js
@@ -639,8 +639,6 @@ class EventTarget {
         if (signal.aborted) {
           return;
         }
-        // TODO(benjamingr) make this weak somehow? ideally the signal would
-        // not prevent the event target from GC.
         signal.addEventListener('abort', () => {
           this.removeEventListener(type, listener, options);
         }, { __proto__: null, once: true, [kWeakHandler]: this, [kResistStopPropagation]: true });

Original file line number	Diff line number	Diff line change
`@@ -639,8 +639,6 @@ class EventTarget {`
`639`	`639`	`if (signal.aborted) {`
`640`	`640`	`return;`
`641`	`641`	`}`
`642`		`- // TODO(benjamingr) make this weak somehow? ideally the signal would`
`643`		`- // not prevent the event target from GC.`
`644`	`642`	`signal.addEventListener('abort', () => {`
`645`	`643`	`this.removeEventListener(type, listener, options);`
`646`	`644`	`}, { __proto__: null, once: true, [kWeakHandler]: this, [kResistStopPropagation]: true });`