fergalmoran/ladybird
1
0
Fork 0
mirror of https://github.com/fergalmoran/ladybird.git synced 2025-12-26 11:20:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

LibJS: Remove two unnecessary 'vm.argument(0).value_or(js_undefined())'

Browse Source 
VM::argument() never returns an empty value.
This commit is contained in:
Linus Groh
2021-07-04 22:51:50 +01:00
committed by Andreas Kling
parent fdf701ed96
commit 57f7e6e775
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
Userland/Libraries/LibJS/Runtime/Promise.cpp
View File

@@ -63,7 +63,7 @@ Promise::ResolvingFunctions Promise::create_resolving_functions()
return js_undefined();
}
already_resolved.value = true;
auto resolution = vm.argument(0).value_or(js_undefined());
auto resolution = vm.argument(0);
if (resolution.is_object() && &resolution.as_object() == &promise) {
dbgln_if(PROMISE_DEBUG, "[Promise @ {} / PromiseResolvingFunction]: Promise can't be resolved with itself, rejecting with error", &promise);
auto* self_resolution_error = TypeError::create(global_object, "Cannot resolve promise with itself");
@@ -100,7 +100,7 @@ Promise::ResolvingFunctions Promise::create_resolving_functions()
if (already_resolved.value)
return js_undefined();
already_resolved.value = true;
auto reason = vm.argument(0).value_or(js_undefined());
auto reason = vm.argument(0);
return promise.reject(reason);
});