diff options
| author | Daniel Bertalan <dani@danielbertalan.dev> | 2022-01-28 21:22:13 +0100 |
|---|---|---|
| committer | Andreas Kling <kling@serenityos.org> | 2022-01-28 23:31:00 +0100 |
| commit | 7d11edbe173f49e1559ebb9bea8442bedd00fc0e (patch) | |
| tree | ecef929f166d3d56ce8f6c6b0f297f829f21cfd9 /Userland/Libraries/LibWeb/Loader | |
| parent | c1184c1fde517993e3f719c7e5aa636197cb2dfc (diff) | |
| download | serenity-7d11edbe173f49e1559ebb9bea8442bedd00fc0e.zip | |
Userland: Fix unnecessary heap allocation of singleton objects
In order to avoid having multiple instances, we were keeping a pointer
to these singleton objects and only allocating them when it was null.
We have `__cxa_guard_{acquire,release}` in the userland, so there's no
need to do this dance, as the compiler will ensure that the constructors
are only called once.
Diffstat (limited to 'Userland/Libraries/LibWeb/Loader')
| -rw-r--r-- | Userland/Libraries/LibWeb/Loader/ContentFilter.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Userland/Libraries/LibWeb/Loader/ContentFilter.cpp b/Userland/Libraries/LibWeb/Loader/ContentFilter.cpp index 5d1e1777fe..b725d859de 100644 --- a/Userland/Libraries/LibWeb/Loader/ContentFilter.cpp +++ b/Userland/Libraries/LibWeb/Loader/ContentFilter.cpp @@ -11,8 +11,8 @@ namespace Web { ContentFilter& ContentFilter::the() { - static ContentFilter* filter = new ContentFilter; - return *filter; + static ContentFilter filter; + return filter; } ContentFilter::ContentFilter() |