language-servers/deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2025-08-04 19:08:15 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

refactor(ext/webidl): remove object from 'requiredArguments' (#18674)

Browse source 
This should produce a little less garbage and using an object here
wasn't really required.

---------

Co-authored-by: Aapo Alasuutari <aapo.alasuutari@gmail.com>
Co-authored-by: Leo Kettmeir <crowlkats@toaxl.com>
This commit is contained in:
Bartek Iwańczuk 2023-04-12 21:58:57 +02:00 • committed by GitHub
parent 9c255b2843
commit a3c5193a2e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
24 changed files with 114 additions and 114 deletions
Download patch file Download diff file Expand all files Collapse all files

38
ext/web/02_event.js
View file

@ -162,9 +162,11 @@ class Event {
this[_path] = [];
if (!eventInitDict[_skipInternalInit]) {
webidl.requiredArguments(arguments.length, 1, {
prefix: "Failed to construct 'Event'",
});
webidl.requiredArguments(
arguments.length,
1,
"Failed to construct 'Event'",
);
type = webidl.converters.DOMString(type, {
prefix: "Failed to construct 'Event'",
context: "Argument 1",
@ -958,9 +960,7 @@ class EventTarget {
webidl.assertBranded(self, EventTargetPrototype);
const prefix = "Failed to execute 'addEventListener' on 'EventTarget'";
webidl.requiredArguments(arguments.length, 2, {
prefix,
});
webidl.requiredArguments(arguments.length, 2, prefix);
options = webidl.converters.AddEventListenerOptions(options, {
prefix,
@ -1014,9 +1014,11 @@ class EventTarget {
) {
const self = this ?? globalThis_;
webidl.assertBranded(self, EventTargetPrototype);
webidl.requiredArguments(arguments.length, 2, {
prefix: "Failed to execute 'removeEventListener' on 'EventTarget'",
});
webidl.requiredArguments(
arguments.length,
2,
"Failed to execute 'removeEventListener' on 'EventTarget'",
);
const { listeners } = self[eventTargetData];
if (callback !== null && ReflectHas(listeners, type)) {
@ -1052,9 +1054,11 @@ class EventTarget {
// executed.
const self = this ?? globalThis_;
webidl.assertBranded(self, EventTargetPrototype);
webidl.requiredArguments(arguments.length, 1, {
prefix: "Failed to execute 'dispatchEvent' on 'EventTarget'",
});
webidl.requiredArguments(
arguments.length,
1,
"Failed to execute 'dispatchEvent' on 'EventTarget'",
);
// This is an optimization to avoid creating an event listener
// on each startup.
@ -1261,9 +1265,11 @@ class CustomEvent extends Event {
constructor(type, eventInitDict = {}) {
super(type, eventInitDict);
webidl.requiredArguments(arguments.length, 1, {
prefix: "Failed to construct 'CustomEvent'",
});
webidl.requiredArguments(
arguments.length,
1,
"Failed to construct 'CustomEvent'",
);
const { detail } = eventInitDict;
this.#detail = detail;
}
@ -1512,7 +1518,7 @@ function checkThis(thisArg) {
function reportError(error) {
checkThis(this);
const prefix = "Failed to call 'reportError'";
webidl.requiredArguments(arguments.length, 1, { prefix });
webidl.requiredArguments(arguments.length, 1, prefix);
reportException(error);
}