diff options
| author | Linus Groh <mail@linusgroh.de> | 2023-04-13 00:47:15 +0200 |
|---|---|---|
| committer | Linus Groh <mail@linusgroh.de> | 2023-04-13 14:29:42 +0200 |
| commit | b84f8fb55b2f926970af1d657314bb82a905ae1d (patch) | |
| tree | bef32f893a3077f71b29e955fbf036602e6e83e5 /Userland/Libraries/LibJS/Runtime/ReflectObject.cpp | |
| parent | ed9e2366da6a28ad39bf6f71d70201d59fa79220 (diff) | |
| download | serenity-b84f8fb55b2f926970af1d657314bb82a905ae1d.zip | |
LibJS: Make intrinsics getters return NonnullGCPtr
Some of these are allocated upon initialization of the intrinsics, and
some lazily, but in neither case the getters actually return a nullptr.
This saves us a whole bunch of pointer dereferences (as NonnullGCPtr has
an `operator T&()`), and also has the interesting side effect of forcing
us to explicitly use the FunctionObject& overload of call(), as passing
a NonnullGCPtr is ambigous - it could implicitly be turned into a Value
_or_ a FunctionObject& (so we have to dereference manually).
Diffstat (limited to 'Userland/Libraries/LibJS/Runtime/ReflectObject.cpp')
| -rw-r--r-- | Userland/Libraries/LibJS/Runtime/ReflectObject.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Userland/Libraries/LibJS/Runtime/ReflectObject.cpp b/Userland/Libraries/LibJS/Runtime/ReflectObject.cpp index 12abd35fc0..f3ba6ed5ab 100644 --- a/Userland/Libraries/LibJS/Runtime/ReflectObject.cpp +++ b/Userland/Libraries/LibJS/Runtime/ReflectObject.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2020-2022, Linus Groh <linusg@serenityos.org> + * Copyright (c) 2020-2023, Linus Groh <linusg@serenityos.org> * * SPDX-License-Identifier: BSD-2-Clause */ @@ -16,7 +16,7 @@ namespace JS { ReflectObject::ReflectObject(Realm& realm) - : Object(ConstructWithPrototypeTag::Tag, *realm.intrinsics().object_prototype()) + : Object(ConstructWithPrototypeTag::Tag, realm.intrinsics().object_prototype()) { } |