Changeset 58003 in webkit for trunk/JavaScriptCore/wtf/WTFThreadData.h

Timestamp:

Apr 21, 2010, 12:40:20 PM (15 years ago)

Author:

[email protected]

Message:

• API/JSValueRef.cpp:
• JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
• runtime/Identifier.cpp:

(JSC::IdentifierTable::~IdentifierTable):
(JSC::IdentifierTable::add):

• runtime/Identifier.h:
• wtf/WTFThreadData.h:

(JSC::IdentifierTable::remove):
(JSC::IdentifierTable::literalTable):

• wtf/text/StringImpl.cpp:

(WebCore::StringImpl::~StringImpl):

Reviewed by NOBODY (windows build fix).

File:

• trunk/JavaScriptCore/wtf/WTFThreadData.h (modified) (2 diffs)

trunk/JavaScriptCore/wtf/WTFThreadData.h

@@ -31 +31 @@
 #include <wtf/HashSet.h>
 #include <wtf/Noncopyable.h>
+#include <wtf/text/StringHash.h>
 
 // This was ENABLE(WORKERS) in WebCore, but this is not defined when compiling JSC.
@@ -46 +47 @@
 
 // FIXME: This is a temporary layering violation while we move more string code to WTF.
-namespace JSC {
-class IdentifierTable;
-}
-
-// FIXME: This is a temporary layering violation while we move more string code to WTF.
 namespace WebCore {
 class AtomicStringTable;
+class StringImpl;
 }
+using WebCore::StringImpl;
 
 typedef void (*AtomicStringTableDestructor)(WebCore::AtomicStringTable*);
+
+#if USE(JSC)
+// FIXME: This is a temporary layering violation while we move more string code to WTF.
+namespace JSC {
+
+typedef HashMap<const char*, RefPtr<StringImpl>, PtrHash<const char*> > LiteralIdentifierTable;
+
+class IdentifierTable : public FastAllocBase {
+public:
+    ~IdentifierTable();
+
+    std::pair<HashSet<StringImpl*>::iterator, bool> add(StringImpl* value);
+    template<typename U, typename V>
+    std::pair<HashSet<StringImpl*>::iterator, bool> add(U value);
+
+    void remove(StringImpl* r) { m_table.remove(r); }
+
+    LiteralIdentifierTable& literalTable() { return m_literalTable; }
+
+private:
+    HashSet<StringImpl*> m_table;
+    LiteralIdentifierTable m_literalTable;
+};
+
+}
+#endif
 
 namespace WTF {