Forgot to address a review comment about better names for tracked objects...
authorap@webkit.org <ap@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Jun 2008 07:15:20 +0000 (07:15 +0000)
committerap@webkit.org <ap@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Jun 2008 07:15:20 +0000 (07:15 +0000)
        * kjs/JSGlobalData.cpp:
        (KJS::JSGlobalData::JSGlobalData):
        * kjs/JSGlobalData.h:
        * kjs/nodes.cpp:
        (KJS::ParserRefCounted::ParserRefCounted):
        (KJS::ParserRefCounted::ref):
        (KJS::ParserRefCounted::deref):
        (KJS::ParserRefCounted::hasOneRef):
        (KJS::ParserRefCounted::deleteNewObjects):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@34792 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/kjs/JSGlobalData.cpp
JavaScriptCore/kjs/JSGlobalData.h
JavaScriptCore/kjs/nodes.cpp

index 33bce0416d015535d73a223948d60c758e92fdd5..4755eda6c94b9ad1f320160ece84ab33055ec8a7 100644 (file)
@@ -1,3 +1,17 @@
+2008-06-25  Alexey Proskuryakov  <ap@webkit.org>
+
+        Forgot to address a review comment about better names for tracked objects, doing it now.
+
+        * kjs/JSGlobalData.cpp:
+        (KJS::JSGlobalData::JSGlobalData):
+        * kjs/JSGlobalData.h:
+        * kjs/nodes.cpp:
+        (KJS::ParserRefCounted::ParserRefCounted):
+        (KJS::ParserRefCounted::ref):
+        (KJS::ParserRefCounted::deref):
+        (KJS::ParserRefCounted::hasOneRef):
+        (KJS::ParserRefCounted::deleteNewObjects):
+
 2008-06-25  Alexey Proskuryakov  <ap@webkit.org>
 
         Reviewed by Geoff.
index 1844b066898c3361b707618f29878b54a86a763a..f3f951c85496ab5eab5d187c83563e9dd5722b5d 100644 (file)
@@ -77,8 +77,8 @@ JSGlobalData::JSGlobalData()
 #endif
     , identifierTable(createIdentifierTable())
     , propertyNames(new CommonIdentifiers(this))
-    , newTrackedObjects(0)
-    , trackedObjectExtraRefCounts(0)
+    , newParserObjects(0)
+    , parserObjectExtraRefCounts(0)
     , lexer(new Lexer(this))
     , parser(new Parser)
     , head(0)
index 06bb659bfbd6db5f62a03a20bbd74a02adeecaca..8f27123ad42b44aa7a626d5a7a7afad4c6ff32b5 100644 (file)
@@ -73,8 +73,8 @@ namespace KJS {
         const ArgList emptyList; // Lists are supposed to be allocated on the stack to have their elements properly marked, which is not the case here - but this list has nothing to mark.
 
         // Tracked object sets are transient, only needed when parsing.
-        HashSet<ParserRefCounted*>* newTrackedObjects;
-        HashCountedSet<ParserRefCounted*>* trackedObjectExtraRefCounts;
+        HashSet<ParserRefCounted*>* newParserObjects;
+        HashCountedSet<ParserRefCounted*>* parserObjectExtraRefCounts;
 
         Lexer* lexer;
         Parser* parser;
index e2a790028e2c63c2fbbc8bf5db2f1d875a7db528..f82c3ad96318f36a576af11e798e0492a9638a84 100644 (file)
@@ -112,10 +112,10 @@ ParserRefCounted::ParserRefCounted(JSGlobalData* globalData)
 #ifndef NDEBUG
     ParserRefCountedCounter::increment();
 #endif
-    if (!m_globalData->newTrackedObjects)
-        m_globalData->newTrackedObjects = new HashSet<ParserRefCounted*>;
-    m_globalData->newTrackedObjects->add(this);
-    ASSERT(m_globalData->newTrackedObjects->contains(this));
+    if (!m_globalData->newParserObjects)
+        m_globalData->newParserObjects = new HashSet<ParserRefCounted*>;
+    m_globalData->newParserObjects->add(this);
+    ASSERT(m_globalData->newParserObjects->contains(this));
 }
 
 ParserRefCounted::~ParserRefCounted()
@@ -128,66 +128,66 @@ ParserRefCounted::~ParserRefCounted()
 void ParserRefCounted::ref()
 {
     // bumping from 0 to 1 is just removing from the new nodes set
-    if (m_globalData->newTrackedObjects) {
-        HashSet<ParserRefCounted*>::iterator it = m_globalData->newTrackedObjects->find(this);
-        if (it != m_globalData->newTrackedObjects->end()) {
-            m_globalData->newTrackedObjects->remove(it);
-            ASSERT(!m_globalData->trackedObjectExtraRefCounts || !m_globalData->trackedObjectExtraRefCounts->contains(this));
+    if (m_globalData->newParserObjects) {
+        HashSet<ParserRefCounted*>::iterator it = m_globalData->newParserObjects->find(this);
+        if (it != m_globalData->newParserObjects->end()) {
+            m_globalData->newParserObjects->remove(it);
+            ASSERT(!m_globalData->parserObjectExtraRefCounts || !m_globalData->parserObjectExtraRefCounts->contains(this));
             return;
         }
     }
 
-    ASSERT(!m_globalData->newTrackedObjects || !m_globalData->newTrackedObjects->contains(this));
+    ASSERT(!m_globalData->newParserObjects || !m_globalData->newParserObjects->contains(this));
 
-    if (!m_globalData->trackedObjectExtraRefCounts)
-        m_globalData->trackedObjectExtraRefCounts = new HashCountedSet<ParserRefCounted*>;
-    m_globalData->trackedObjectExtraRefCounts->add(this);
+    if (!m_globalData->parserObjectExtraRefCounts)
+        m_globalData->parserObjectExtraRefCounts = new HashCountedSet<ParserRefCounted*>;
+    m_globalData->parserObjectExtraRefCounts->add(this);
 }
 
 void ParserRefCounted::deref()
 {
-    ASSERT(!m_globalData->newTrackedObjects || !m_globalData->newTrackedObjects->contains(this));
+    ASSERT(!m_globalData->newParserObjects || !m_globalData->newParserObjects->contains(this));
 
-    if (!m_globalData->trackedObjectExtraRefCounts) {
+    if (!m_globalData->parserObjectExtraRefCounts) {
         delete this;
         return;
     }
 
-    HashCountedSet<ParserRefCounted*>::iterator it = m_globalData->trackedObjectExtraRefCounts->find(this);
-    if (it == m_globalData->trackedObjectExtraRefCounts->end())
+    HashCountedSet<ParserRefCounted*>::iterator it = m_globalData->parserObjectExtraRefCounts->find(this);
+    if (it == m_globalData->parserObjectExtraRefCounts->end())
         delete this;
     else
-        m_globalData->trackedObjectExtraRefCounts->remove(it);
+        m_globalData->parserObjectExtraRefCounts->remove(it);
 }
 
 bool ParserRefCounted::hasOneRef()
 {
-    if (m_globalData->newTrackedObjects && m_globalData->newTrackedObjects->contains(this)) {
-        ASSERT(!m_globalData->trackedObjectExtraRefCounts || !m_globalData->trackedObjectExtraRefCounts->contains(this));
+    if (m_globalData->newParserObjects && m_globalData->newParserObjects->contains(this)) {
+        ASSERT(!m_globalData->parserObjectExtraRefCounts || !m_globalData->parserObjectExtraRefCounts->contains(this));
         return false;
     }
 
-    ASSERT(!m_globalData->newTrackedObjects || !m_globalData->newTrackedObjects->contains(this));
+    ASSERT(!m_globalData->newParserObjects || !m_globalData->newParserObjects->contains(this));
 
-    if (!m_globalData->trackedObjectExtraRefCounts)
+    if (!m_globalData->parserObjectExtraRefCounts)
         return true;
 
-    return !m_globalData->trackedObjectExtraRefCounts->contains(this);
+    return !m_globalData->parserObjectExtraRefCounts->contains(this);
 }
 
 void ParserRefCounted::deleteNewObjects(JSGlobalData* globalData)
 {
-    if (!globalData->newTrackedObjects)
+    if (!globalData->newParserObjects)
         return;
 
 #ifndef NDEBUG
-    HashSet<ParserRefCounted*>::iterator end = globalData->newTrackedObjects->end();
-    for (HashSet<ParserRefCounted*>::iterator it = globalData->newTrackedObjects->begin(); it != end; ++it)
-        ASSERT(!globalData->trackedObjectExtraRefCounts || !globalData->trackedObjectExtraRefCounts->contains(*it));
+    HashSet<ParserRefCounted*>::iterator end = globalData->newParserObjects->end();
+    for (HashSet<ParserRefCounted*>::iterator it = globalData->newParserObjects->begin(); it != end; ++it)
+        ASSERT(!globalData->parserObjectExtraRefCounts || !globalData->parserObjectExtraRefCounts->contains(*it));
 #endif
-    deleteAllValues(*globalData->newTrackedObjects);
-    delete globalData->newTrackedObjects;
-    globalData->newTrackedObjects = 0;
+    deleteAllValues(*globalData->newParserObjects);
+    delete globalData->newParserObjects;
+    globalData->newParserObjects = 0;
 }
 
 Node::Node(JSGlobalData* globalData)