eliott/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird synced 2026-04-30 19:47:17 +02:00
Code Issues Packages Projects Releases Wiki Activity

LibJS: Hide all the constructors!

Browse Source 
Now that the GC allocator is able to invoke Cell subclass constructors
directly via friendship, we no longer need to keep them public. :^)
This commit is contained in:
Andreas Kling
2022-08-28 23:51:28 +02:00
parent d54ba587f3
commit 35c9aa7c05
Notes: sideshowbarker 2024-07-17 07:38:42 +09:00 
Author: https://github.com/awesomekling
Commit: https://github.com/SerenityOS/serenity/commit/35c9aa7c05
Pull-request: https://github.com/SerenityOS/serenity/pull/15061
Reviewed-by: https://github.com/linusg ✅
196 changed files with 456 additions and 242 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
Userland/Libraries/LibJS/Runtime/ModuleNamespaceObject.h
View File

@@ -16,8 +16,6 @@ class ModuleNamespaceObject final : public Object {
JS_OBJECT(ModuleNamespaceObject, Object);
public:
ModuleNamespaceObject(Realm&, Module* module, Vector<FlyString> exports);
// 10.4.6 Module Namespace Exotic Objects, https://tc39.es/ecma262/#sec-module-namespace-exotic-objects
virtual ThrowCompletionOr<Object*> internal_get_prototype_of() const override;
@@ -34,6 +32,8 @@ public:
virtual void initialize(Realm&) override;
private:
ModuleNamespaceObject(Realm&, Module* module, Vector<FlyString> exports);
// FIXME: UHHH how do we want to store this to avoid cycles but be safe??
Module* m_module; // [[Module]]
Vector<FlyString> m_exports; // [[Exports]]