Changeset 28608 in webkit for trunk/JavaScriptCore/kjs/function_object.cpp

Timestamp:

Dec 10, 2007, 9:47:41 PM (17 years ago)

Author:

[email protected]

Message:

JavaScriptCore:

Reviewed by Sam Weinig.

Split this:

FunctionBodyNode

^|

ProgramNode

into this:

ScopeNode

^{| | |}

FunctionBodyNode ProgramNode EvalNode

in preparation for specializing each class more while optimizing global
variable access.

Also removed some cruft from the FunctionBodyNode interface to simplify
things.

SunSpider says this patch is a .8% speedup, which seems reasonable,
since it eliminates a few branches and adds KJS_FAST_CALL in a few
places.

Layout tests and JS tests pass. Also, this baby builds on Windows! (Qt
mileage may vary...)

WebCore:

Reviewed by Sam Weinig.

Updated for rename in JavaScriptCore.

• bridge/mac/WebCoreScriptDebugger.mm: (-[WebCoreScriptCallFrame scopeChain]): (-[WebCoreScriptCallFrame functionName]): (-[WebCoreScriptCallFrame evaluateWebScript:]):

WebKit/win:

Reviewed by Sam Weinig.

Updated for rename in JavaScriptCore.

• WebScriptCallFrame.cpp: (WebScriptCallFrame::functionName): (WebScriptCallFrame::valueByEvaluatingJavaScriptFromString):

File:

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

trunk/JavaScriptCore/kjs/function_object.cpp

@@ -232 +232 @@
               c++, i++;
           if (i == len) {
-              functionBody->addParam(Identifier(param));
+              functionBody->parameters().append(Identifier(param));
               params++;
               break;
           } else if (*c == ',') {
-              functionBody->addParam(Identifier(param));
+              functionBody->parameters().append(Identifier(param));
               params++;
               c++, i++;