Changeset 2851 in webkit for trunk/JavaScriptCore/kjs/interpreter.cpp

Timestamp:

Nov 24, 2002, 2:37:44 PM (23 years ago)

Author:

darin

Message:

JavaScriptCore:

• changed ScopeChain to not ref each item in the chain, and use marking instead; gains 1% on JavaScript iBench

• kjs/context.h: Return chain by reference.
• kjs/internal.cpp: (ContextImp::mark): Mark the scope chain.
• kjs/interpreter.cpp: (Context::scopeChain): Return chain by reference.
• kjs/interpreter.h: Make some Context methods inline.
• kjs/nodes.cpp: (ThisNode::evaluate): Get at ContextImp directly. (ResolveNode::evaluateReference): Ditto. (VarDeclNode::evaluate): Ditto. (VarDeclNode::processVarDecls): Ditto. (FuncDeclNode::processFuncDecl): Pass ScopeChain directly to avoid copying. (FuncExprNode::evaluate): Ditto.
• kjs/object.cpp: Make scope and setScope inline.
• kjs/object.h: Make scope return a chain by reference. Make scope and setScope both be inline. Use a normal ScopeChain instead of NoRefScopeChain since they are now one and the same.
• kjs/scope_chain.cpp: Remove all the code to ref and deref objects. Merge NoRefScopeChain in with ScopeChain since they both work this way now.
• kjs/scope_chain.h: Remove NoRefScopeChain and simplify the ref counts. Make more functions inline.

WebCore:

• force-js-clean-timestamp: Touch for ScopeChain change.

File:

• trunk/JavaScriptCore/kjs/interpreter.cpp (modified) (1 diff)

trunk/JavaScriptCore/kjs/interpreter.cpp

@@ -42 +42 @@
 // ------------------------------ Context --------------------------------------
 
-Context::Context(ContextImp *c)
-{
-  rep = c;
-}
-
-Context::Context(const Context &c)
-{
-  rep = c.rep;
-}
-
-Context& Context::operator=(const Context &c)
-{
-  rep = c.rep;
-  return *this;
-}
-
-Context::~Context()
-{
-}
-
-bool Context::isNull() const
-{
-  return (rep == 0);
-}
-
-ContextImp *Context::imp() const
-{
-  return rep;
-}
-
-const ScopeChain Context::scopeChain() const
+const ScopeChain &Context::scopeChain() const
 {
   return rep->scopeChain();