events: use kEmptyObject

PR-URL: nodejs/node#43159 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
noslate-project · Oct 10, 2022 · 0a985b0 · 0a985b0
1 parent 0b94ec8
commit 0a985b0
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/lib/events.js b/lib/events.js
@@ -52,6 +52,9 @@ const {
   SymbolAsyncIterator,
 } = primordials;
 const kRejection = SymbolFor('nodejs.rejection');
+
+const { kEmptyObject } = require('internal/util');
+
 const { inspect } = require('internal/util/inspect');
 
 let spliceOne;
@@ -938,7 +941,7 @@ function getEventListeners(emitterOrTarget, type) {
  * @param {{ signal: AbortSignal; }} [options]
  * @returns {Promise}
  */
-async function once(emitter, name, options = {}) {
+async function once(emitter, name, options = kEmptyObject) {
   const signal = options?.signal;
   validateAbortSignal(signal, 'options.signal');
   if (signal?.aborted)

diff --git a/lib/internal/event_target.js b/lib/internal/event_target.js
@@ -35,7 +35,11 @@ const {
 } = require('internal/errors');
 const { validateObject, validateString } = require('internal/validators');
 
-const { customInspectSymbol, kEnumerableProperty } = require('internal/util');
+const {
+  customInspectSymbol,
+  kEmptyObject,
+  kEnumerableProperty,
+} = require('internal/util');
 const { inspect } = require('util');
 
 const kIsEventTarget = SymbolFor('nodejs.event_target');
@@ -455,7 +459,7 @@ class EventTarget {
    *   signal?: AbortSignal
    * }} [options]
    */
-  addEventListener(type, listener, options = {}) {
+  addEventListener(type, listener, options = kEmptyObject) {
     if (!isEventTarget(this))
       throw new ERR_INVALID_THIS('EventTarget');
     if (arguments.length < 2)
@@ -542,7 +546,7 @@ class EventTarget {
    *   capture?: boolean,
    * }} [options]
    */
-  removeEventListener(type, listener, options = {}) {
+  removeEventListener(type, listener, options = kEmptyObject) {
     if (!isEventTarget(this))
       throw new ERR_INVALID_THIS('EventTarget');
     if (!shouldAddListener(listener))
@@ -870,7 +874,7 @@ function validateEventListenerOptions(options) {
     return { capture: options };
 
   if (options === null)
-    return {};
+    return kEmptyObject;
   validateObject(options, 'options', {
     allowArray: true, allowFunction: true,
   });