2008-10-24 Cameron Zwarich <zwarich@apple.com>
authorcwzwarich@webkit.org <cwzwarich@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 24 Oct 2008 21:07:39 +0000 (21:07 +0000)
committercwzwarich@webkit.org <cwzwarich@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 24 Oct 2008 21:07:39 +0000 (21:07 +0000)
        Reviewed by Geoff Garen.

        Remove ScopeChainNode::bottom() and inline it into its only caller,
        ScopeChainnode::globalObject().

        * kjs/JSGlobalObject.h:
        (JSC::ScopeChainNode::globalObject):
        * kjs/ScopeChain.h:
        (JSC::ScopeChain::bottom):

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

JavaScriptCore/ChangeLog
JavaScriptCore/kjs/JSGlobalObject.h
JavaScriptCore/kjs/ScopeChain.h

index 673cd635e5b11b29ed4a2034d192568716e739d1..ee9525800a19bfe3ca2e74fc74ce470f359df38e 100644 (file)
@@ -1,3 +1,15 @@
+2008-10-24  Cameron Zwarich  <zwarich@apple.com>
+
+        Reviewed by Geoff Garen.
+
+        Remove ScopeChainNode::bottom() and inline it into its only caller,
+        ScopeChainnode::globalObject().
+
+        * kjs/JSGlobalObject.h:
+        (JSC::ScopeChainNode::globalObject):
+        * kjs/ScopeChain.h:
+        (JSC::ScopeChain::bottom):
+
 2008-10-24  Cameron Zwarich  <zwarich@apple.com>
 
         Reviewed by Maciej Stachowiak.
index e0e4fb60ee76ffbd561ab789fe0baad3004b0ec2..d8a072af39f825c72f6aea9526ebf9c5cfbc34bc 100644 (file)
@@ -329,7 +329,10 @@ namespace JSC {
 
     inline JSGlobalObject* ScopeChainNode::globalObject() const
     {
-        return asGlobalObject(bottom());
+        const ScopeChainNode* n = this;
+        while (n->next)
+            n = n->next;
+        return asGlobalObject(n->object);
     }
 
     inline JSValue* StructureID::prototypeForLookup(ExecState* exec)
index c460bf33e90025e09cd92ac0c1d9ccb086e19e4b..834217c9f6106ae4ae2ebd0074b8e329390661ea 100644 (file)
@@ -63,8 +63,6 @@ namespace JSC {
             return this;
         }
 
-        JSObject* bottom() const;
-
         ScopeChainNode* push(JSObject*);
         ScopeChainNode* pop();
 
@@ -98,14 +96,6 @@ namespace JSC {
         return result;
     }
 
-    inline JSObject* ScopeChainNode::bottom() const
-    {
-        const ScopeChainNode* n = this;
-        while (n->next)
-            n = n->next;
-        return n->object;
-    }
-
     inline void ScopeChainNode::release()
     {
         // This function is only called by deref(),
@@ -188,7 +178,6 @@ namespace JSC {
         ScopeChainNode* node() const { return m_node; }
 
         JSObject* top() const { return m_node->object; }
-        JSObject* bottom() const { return m_node->bottom(); }
 
         ScopeChainIterator begin() const { return m_node->begin(); }
         ScopeChainIterator end() const { return m_node->end(); }