Skip to content

Commit

Permalink
lib: use predefined variable instead of bit operation
Browse files Browse the repository at this point in the history
PR-URL: #52580
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Daeyeon Jeong <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
  • Loading branch information
deokjinkim authored and marco-ippolito committed May 3, 2024
1 parent a4847c4 commit acd7758
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 6 deletions.
5 changes: 2 additions & 3 deletions lib/internal/abort_controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,7 @@ const {
validateAbortSignalArray,
validateObject,
validateUint32,
kValidateObjectAllowArray,
kValidateObjectAllowFunction,
kValidateObjectAllowObjects,
} = require('internal/validators');

const {
Expand Down Expand Up @@ -434,7 +433,7 @@ async function aborted(signal, resource) {
throw new ERR_INVALID_ARG_TYPE('signal', 'AbortSignal', signal);
}
validateAbortSignal(signal, 'signal');
validateObject(resource, 'resource', kValidateObjectAllowArray | kValidateObjectAllowFunction);
validateObject(resource, 'resource', kValidateObjectAllowObjects);
if (signal.aborted)
return PromiseResolve();
const abortPromise = createDeferredPromise();
Expand Down
5 changes: 2 additions & 3 deletions lib/internal/event_target.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,7 @@ const {
validateObject,
validateString,
validateInternalField,
kValidateObjectAllowArray,
kValidateObjectAllowFunction,
kValidateObjectAllowObjects,
} = require('internal/validators');

const {
Expand Down Expand Up @@ -1060,7 +1059,7 @@ function validateEventListenerOptions(options) {

if (options === null)
return kEmptyObject;
validateObject(options, 'options', kValidateObjectAllowArray | kValidateObjectAllowFunction);
validateObject(options, 'options', kValidateObjectAllowObjects);
return {
once: Boolean(options.once),
capture: Boolean(options.capture),
Expand Down

0 comments on commit acd7758

Please sign in to comment.