Changeset 35418 in webkit for trunk/JavaScriptCore/kjs/identifier.cpp

Timestamp:

Jul 29, 2008, 1:16:17 AM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Oliver Hunt.

Store UString::Rep::isStatic bit in identifierTable pointer instead of reportedCost for
slightly nicer code and a 0.5% SunSpider improvement.

File:

• trunk/JavaScriptCore/kjs/identifier.cpp (modified) (5 diffs)

trunk/JavaScriptCore/kjs/identifier.cpp

@@ -41 +41 @@
         HashSet<UString::Rep*>::iterator end = m_table.end();
         for (HashSet<UString::Rep*>::iterator iter = m_table.begin(); iter != end; ++iter)
-            (*iter)->identifierTable = 0;
+            (*iter)->setIdentifierTable(0);
     }
     
@@ -47 +47 @@
     {
         std::pair<HashSet<UString::Rep*>::iterator, bool> result = m_table.add(value);
-        (*result.first)->identifierTable = this;
+        (*result.first)->setIdentifierTable(this);
         return result;
     }
@@ -55 +55 @@
     {
         std::pair<HashSet<UString::Rep*>::iterator, bool> result = m_table.add<U, V>(value);
-        (*result.first)->identifierTable = this;
+        (*result.first)->setIdentifierTable(this);
         return result;
     }
@@ -205 +205 @@
 PassRefPtr<UString::Rep> Identifier::addSlowCase(JSGlobalData* globalData, UString::Rep* r)
 {
-    ASSERT(!r->identifierTable);
+    ASSERT(!r->identifierTable());
 
     if (r->len == 0) {
@@ -222 +222 @@
 void Identifier::remove(UString::Rep *r)
 {
-    r->identifierTable->remove(r);
+    r->identifierTable()->remove(r);
 }