Changeset 10148 in webkit for trunk/JavaScriptCore/kjs/object.h

Timestamp:

Aug 12, 2005, 12:36:00 AM (20 years ago)

Author:

mjs

Message:

Reviewed by hyatt.

• refactor function calls, 3% speedup on JS iBench.

• kjs/grammar.y:
• kjs/nodes.cpp: (Node::throwError): Added new useful variants. (FunctionCallValueNode::evaluate): New node to handle calls on expressions that are strictly values, not references. (FunctionCallValueNode::ref): ditto (FunctionCallValueNode::deref): ditto (FunctionCallResolveNode::evaluate): New node to handle calls on identifier expressions, so that they are looked up in the scope chain. (FunctionCallResolveNode::ref): ditto (FunctionCallResolveNode::deref): ditto (FunctionCallBracketNode::evaluate): New node to handle calls on bracket dereferences, so that the expression before brackets is used as the this object. (FunctionCallBracketNode::ref): ditto (FunctionCallBracketNode::deref): ditto (FunctionCallDotNode::evaluate): New node to handle calls on dot dereferences, so that the expression before the dot is used as the this object. (FunctionCallDotNode::ref): ditto (FunctionCallDotNode::deref): ditto (dotExprNotAnObjectString): helper function to avoid global variable access. (dotExprDoesNotAllowCallsString): ditto
• kjs/nodes.h: Declared new classes.
• kjs/nodes2string.cpp: (FunctionCallValueNode::streamTo): Added - serializes the appropriate function call (FunctionCallResolveNode::streamTo): ditto (FunctionCallBracketNode::streamTo): ditto (FunctionCallParenBracketNode::streamTo): ditto (FunctionCallDotNode::streamTo): ditto (FunctionCallParenDotNode::streamTo): ditto
• kjs/object.h: (KJS::ObjectImp::isActivation): Change how activation objects are detected in the scope chain, a virtual function is cheaper than the old inheritance test.
• kjs/function.h: (KJS::ActivationImp::isActivation): Ditto.

File:

• trunk/JavaScriptCore/kjs/object.h (modified) (1 diff)

trunk/JavaScriptCore/kjs/object.h

@@ -528 +528 @@
     void restoreProperties(const SavedProperties &p) { _prop.restore(p); }
 
+    virtual bool isActivation() { return false; }
   protected:
     PropertyMap _prop;