From 9d29abdfa04241ec206d86e072818138f0846153 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Thu, 9 Feb 2023 12:34:41 +0100 Subject: [PATCH] Change the `LoopbackPort` class to use a Set internally This is a tiny bit more compact, thanks to the `Set.prototype.delete` method. --- src/display/api.js | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/display/api.js b/src/display/api.js index df89bcbfa..c7cb81007 100644 --- a/src/display/api.js +++ b/src/display/api.js @@ -1909,7 +1909,7 @@ class PDFPageProxy { } class LoopbackPort { - #listeners = []; + #listeners = new Set(); #deferred = Promise.resolve(); @@ -1926,16 +1926,15 @@ class LoopbackPort { } addEventListener(name, listener) { - this.#listeners.push(listener); + this.#listeners.add(listener); } removeEventListener(name, listener) { - const i = this.#listeners.indexOf(listener); - this.#listeners.splice(i, 1); + this.#listeners.delete(listener); } terminate() { - this.#listeners.length = 0; + this.#listeners.clear(); } }