From 54f452d76409d1c3e404fc6aacd957b678edf418 Mon Sep 17 00:00:00 2001 From: Antoine du Hamel Date: Fri, 24 Feb 2023 09:45:04 +0100 Subject: [PATCH] events: add trailing commas in source files MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR-URL: https://github.com/nodejs/node/pull/46759 Reviewed-By: Deokjin Kim Reviewed-By: Michaƫl Zasso Reviewed-By: Moshe Atlow Reviewed-By: James M Snell --- lib/.eslintrc.yaml | 2 ++ lib/events.js | 18 +++++++++--------- lib/internal/event_target.js | 14 +++++++------- 3 files changed, 18 insertions(+), 16 deletions(-) diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml index e7464551cca155..a10af356b9d356 100644 --- a/lib/.eslintrc.yaml +++ b/lib/.eslintrc.yaml @@ -271,6 +271,7 @@ overrides: - ./cluster.js - ./console.js - ./constants.js + - ./events.js - ./fs.js - ./internal/assert.js - ./internal/child_process.js @@ -278,6 +279,7 @@ overrides: - ./internal/cli_table.js - ./internal/cluster/*.js - ./internal/debugger/*.js + - ./internal/event_target.js - ./internal/events/*.js - ./internal/fixed_queue.js - ./internal/freelist.js diff --git a/lib/events.js b/lib/events.js index 75834ec106c72e..56615c13c49cc7 100644 --- a/lib/events.js +++ b/lib/events.js @@ -68,7 +68,7 @@ const { codes: { ERR_INVALID_ARG_TYPE, ERR_INVALID_THIS, - ERR_UNHANDLED_ERROR + ERR_UNHANDLED_ERROR, }, genericNodeError, } = require('internal/errors'); @@ -99,7 +99,7 @@ let EventEmitterAsyncResource; function lazyEventEmitterAsyncResource() { if (EventEmitterAsyncResource === undefined) { const { - AsyncResource + AsyncResource, } = require('async_hooks'); const kEventEmitter = Symbol('kEventEmitter'); @@ -236,7 +236,7 @@ ObjectDefineProperty(EventEmitter, 'captureRejections', { EventEmitter.prototype[kCapture] = value; }, - enumerable: true + enumerable: true, }); ObjectDefineProperty(EventEmitter, 'EventEmitterAsyncResource', { @@ -254,7 +254,7 @@ ObjectDefineProperty(EventEmitter.prototype, kCapture, { __proto__: null, value: false, writable: true, - enumerable: false + enumerable: false, }); EventEmitter.prototype._events = undefined; @@ -279,7 +279,7 @@ ObjectDefineProperty(EventEmitter, 'defaultMaxListeners', { set: function(arg) { validateNumber(arg, 'defaultMaxListeners', 0); defaultMaxListeners = arg; - } + }, }); ObjectDefineProperties(EventEmitter, { @@ -296,7 +296,7 @@ ObjectDefineProperties(EventEmitter, { enumerable: false, configurable: false, writable: false, - } + }, }); /** @@ -477,7 +477,7 @@ EventEmitter.prototype.emit = function emit(type, ...args) { ObjectDefineProperty(er, kEnhanceStackBeforeInspector, { __proto__: null, value: FunctionPrototypeBind(enhanceStackTrace, this, er, capture), - configurable: true + configurable: true, }); } catch { // Continue regardless of error. @@ -1111,7 +1111,7 @@ function on(emitter, event, options = kEmptyObject) { */ get isPaused() { return paused; - } + }, }, }, AsyncIteratorPrototype); @@ -1186,6 +1186,6 @@ function listenersController() { while (listeners.length > 0) { ReflectApply(eventTargetAgnosticRemoveListener, undefined, ArrayPrototypePop(listeners)); } - } + }, }; } diff --git a/lib/internal/event_target.js b/lib/internal/event_target.js index a8a7fb2ed9d88b..60c5eb80ceabd3 100644 --- a/lib/internal/event_target.js +++ b/lib/internal/event_target.js @@ -32,7 +32,7 @@ const { ERR_EVENT_RECURSION, ERR_MISSING_ARGS, ERR_INVALID_THIS, - } + }, } = require('internal/errors'); const { validateAbortSignal, validateObject, validateString, validateInternalField } = require('internal/validators'); @@ -75,7 +75,7 @@ const isTrustedSet = new SafeWeakSet(); const isTrusted = ObjectGetOwnPropertyDescriptor({ get isTrusted() { return isTrustedSet.has(this); - } + }, }, 'isTrusted').get; const isTrustedDescriptor = { @@ -150,7 +150,7 @@ class Event { return name; const opts = ObjectAssign({}, options, { - depth: NumberIsInteger(options.depth) ? options.depth - 1 : options.depth + depth: NumberIsInteger(options.depth) ? options.depth - 1 : options.depth, }); return `${name} ${inspect({ @@ -784,7 +784,7 @@ class EventTarget { return name; const opts = ObjectAssign({}, options, { - depth: NumberIsInteger(options.depth) ? options.depth - 1 : options.depth + depth: NumberIsInteger(options.depth) ? options.depth - 1 : options.depth, }); return `${name} ${inspect({}, opts)}`; @@ -801,7 +801,7 @@ ObjectDefineProperties(EventTarget.prototype, { enumerable: false, configurable: true, value: 'EventTarget', - } + }, }); function initNodeEventTarget(self) { @@ -1001,7 +1001,7 @@ function validateEventListenerOptions(options) { passive: Boolean(options.passive), signal: options.signal, weak: options[kWeakHandler], - isNodeStyleListener: Boolean(options[kIsNodeStyleListener]) + isNodeStyleListener: Boolean(options[kIsNodeStyleListener]), }; } @@ -1089,7 +1089,7 @@ function defineEventHandler(emitter, name, event = name) { get, set, configurable: true, - enumerable: true + enumerable: true, }); }