Skip to content

Commit

Permalink
events: add trailing commas in source files
Browse files Browse the repository at this point in the history
PR-URL: #46759
Reviewed-By: Deokjin Kim <deokjin81.kim@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
aduh95 authored and targos committed Mar 14, 2023
1 parent 6d32a16 commit 1a23eab
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 15 deletions.
2 changes: 2 additions & 0 deletions lib/.eslintrc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -269,13 +269,15 @@ overrides:
- ./cluster.js
- ./console.js
- ./constants.js
- ./events.js
- ./fs.js
- ./internal/assert.js
- ./internal/child_process.js
- ./internal/child_process/*.js
- ./internal/cli_table.js
- ./internal/cluster/*.js
- ./internal/debugger/*.js
- ./internal/event_target.js
- ./internal/events/*.js
- ./internal/fixed_queue.js
- ./internal/freelist.js
Expand Down
16 changes: 8 additions & 8 deletions lib/events.js
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ const {
ERR_INVALID_ARG_TYPE,
ERR_INVALID_THIS,
ERR_OUT_OF_RANGE,
ERR_UNHANDLED_ERROR
ERR_UNHANDLED_ERROR,
},
genericNodeError,
} = require('internal/errors');
Expand Down Expand Up @@ -94,7 +94,7 @@ let EventEmitterAsyncResource;
function lazyEventEmitterAsyncResource() {
if (EventEmitterAsyncResource === undefined) {
const {
AsyncResource
AsyncResource,
} = require('async_hooks');

const kEventEmitter = Symbol('kEventEmitter');
Expand Down Expand Up @@ -231,7 +231,7 @@ ObjectDefineProperty(EventEmitter, 'captureRejections', {

EventEmitter.prototype[kCapture] = value;
},
enumerable: true
enumerable: true,
});

ObjectDefineProperty(EventEmitter, 'EventEmitterAsyncResource', {
Expand All @@ -249,7 +249,7 @@ ObjectDefineProperty(EventEmitter.prototype, kCapture, {
__proto__: null,
value: false,
writable: true,
enumerable: false
enumerable: false,
});

EventEmitter.prototype._events = undefined;
Expand Down Expand Up @@ -278,7 +278,7 @@ ObjectDefineProperty(EventEmitter, 'defaultMaxListeners', {
arg);
}
defaultMaxListeners = arg;
}
},
});

ObjectDefineProperties(EventEmitter, {
Expand All @@ -295,7 +295,7 @@ ObjectDefineProperties(EventEmitter, {
enumerable: false,
configurable: false,
writable: false,
}
},
});

/**
Expand Down Expand Up @@ -479,7 +479,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.
Expand Down Expand Up @@ -1092,7 +1092,7 @@ function on(emitter, event, options = kEmptyObject) {

[SymbolAsyncIterator]() {
return this;
}
},
}, AsyncIteratorPrototype);

eventTargetAgnosticAddListener(emitter, event, eventHandler);
Expand Down
14 changes: 7 additions & 7 deletions lib/internal/event_target.js
Original file line number Diff line number Diff line change
Expand Up @@ -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');

Expand Down Expand Up @@ -75,7 +75,7 @@ const isTrustedSet = new SafeWeakSet();
const isTrusted = ObjectGetOwnPropertyDescriptor({
get isTrusted() {
return isTrustedSet.has(this);
}
},
}, 'isTrusted').get;

const isTrustedDescriptor = {
Expand Down Expand Up @@ -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({
Expand Down Expand Up @@ -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)}`;
Expand All @@ -801,7 +801,7 @@ ObjectDefineProperties(EventTarget.prototype, {
enumerable: false,
configurable: true,
value: 'EventTarget',
}
},
});

function initNodeEventTarget(self) {
Expand Down Expand Up @@ -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]),
};
}

Expand Down Expand Up @@ -1089,7 +1089,7 @@ function defineEventHandler(emitter, name, event = name) {
get,
set,
configurable: true,
enumerable: true
enumerable: true,
});
}

Expand Down

0 comments on commit 1a23eab

Please sign in to comment.