diff options
| author | Andreas Kling <kling@serenityos.org> | 2020-10-04 17:36:12 +0200 |
|---|---|---|
| committer | Andreas Kling <kling@serenityos.org> | 2020-10-04 19:25:49 +0200 |
| commit | d01b746d8810890250995084dcd7b97f870286b1 (patch) | |
| tree | 0f92fcd30059992aa144b538ae62a34c73d7f665 /Libraries/LibJS/Runtime/StringOrSymbol.h | |
| parent | 3d053f244f0cb16be187a8935e8120e221e1c716 (diff) | |
| download | serenity-d01b746d8810890250995084dcd7b97f870286b1.zip | |
LibJS: Add StringOrSymbol constructor that takes a FlyString
This avoids refcount churn from implicit conversion in some places.
Diffstat (limited to 'Libraries/LibJS/Runtime/StringOrSymbol.h')
| -rw-r--r-- | Libraries/LibJS/Runtime/StringOrSymbol.h | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/Libraries/LibJS/Runtime/StringOrSymbol.h b/Libraries/LibJS/Runtime/StringOrSymbol.h index 6939194183..bb4aba3e7a 100644 --- a/Libraries/LibJS/Runtime/StringOrSymbol.h +++ b/Libraries/LibJS/Runtime/StringOrSymbol.h @@ -26,6 +26,7 @@ #pragma once +#include <AK/FlyString.h> #include <LibJS/Runtime/PrimitiveString.h> #include <LibJS/Runtime/Symbol.h> #include <LibJS/Runtime/Value.h> @@ -58,6 +59,13 @@ public: static_cast<const StringImpl*>(m_ptr)->ref(); } + StringOrSymbol(const FlyString& string) + : m_ptr(string.impl()) + { + ASSERT(!string.is_null()); + static_cast<const StringImpl*>(m_ptr)->ref(); + } + ~StringOrSymbol() { if (is_string()) |