Changeset 190829 in webkit for trunk/Source/JavaScriptCore/runtime/SmallStrings.h

Timestamp:

Oct 9, 2015, 4:17:59 PM (10 years ago)

Author:

[email protected]

Message:

No need to keep setting needsVisit flag in SmallStrings
​https://bugs.webkit.org/show_bug.cgi?id=149961

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-10-09
Reviewed by Andreas Kling.

SmallStrings are all initialized at once privately before the VM
enables Garbage Collection. There is no need to keep updating
this flag, as it couldn't have changed.

• runtime/SmallStrings.cpp:

(JSC::SmallStrings::createEmptyString):
(JSC::SmallStrings::createSingleCharacterString):
(JSC::SmallStrings::initialize):

• runtime/SmallStrings.h:

File:

• trunk/Source/JavaScriptCore/runtime/SmallStrings.h (modified) (2 diffs)

trunk/Source/JavaScriptCore/runtime/SmallStrings.h

@@ -49 +49 @@
 namespace JSC {
 
-class HeapRootVisitor;
 class VM;
 class JSString;
@@ -124 +123 @@
     static const unsigned singleCharacterStringCount = maxSingleCharacterString + 1;
 
-    JS_EXPORT_PRIVATE void createEmptyString(VM*);
-    JS_EXPORT_PRIVATE void createSingleCharacterString(VM*, unsigned char);
+    void createEmptyString(VM*);
+    void createSingleCharacterString(VM*, unsigned char);
 
     void initialize(VM*, JSString*&, const char* value);