Changeset 26715 in webkit for trunk/JavaScriptCore/kjs/nodes.cpp

Timestamp:

Oct 17, 2007, 2:01:27 PM (18 years ago)

Author:

ggaren

Message:

Reviewed by Darin Adler.

Merged DeclaredFunctionImp into FunctionImp (the base class) because
the distinction between the two was unused.

Removed codeType() from FunctionImp because FunctionImp and its
subclasses all returned FunctionCode, so it was unused, practically
speaking.

Removed a different codeType() from GlobalFuncImp because it was unused.
(Perhaps it was vestigial from a time when GlobalFuncImp used to
inherit from FunctionImp.)

• bindings/runtime_method.cpp:
• bindings/runtime_method.h:
• kjs/function.cpp: (KJS::): (KJS::FunctionImp::FunctionImp): (KJS::FunctionImp::callAsFunction): (KJS::FunctionImp::construct): (KJS::FunctionImp::execute): (KJS::FunctionImp::processVarDecls):
• kjs/function.h: (KJS::FunctionImp::implementsConstruct): (KJS::FunctionImp::scope):
• kjs/function_object.cpp: (FunctionProtoFunc::callAsFunction): (FunctionObjectImp::construct):
• kjs/nodes.cpp: (FuncDeclNode::processFuncDecl): (FuncExprNode::evaluate):

File:

• trunk/JavaScriptCore/kjs/nodes.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/nodes.cpp

@@ -2497 +2497 @@
 
   // TODO: let this be an object with [[Class]] property "Function"
-  FunctionImp *func = new DeclaredFunctionImp(exec, ident, body.get(), context->scopeChain());
+  FunctionImp *func = new FunctionImp(exec, ident, body.get(), context->scopeChain());
 
   JSObject *proto = exec->lexicalInterpreter()->builtinObject()->construct(exec, List::empty());
@@ -2537 +2537 @@
   }
 
-  FunctionImp* func = new DeclaredFunctionImp(exec, ident, body.get(), context->scopeChain());
+  FunctionImp* func = new FunctionImp(exec, ident, body.get(), context->scopeChain());
   JSObject* proto = exec->lexicalInterpreter()->builtinObject()->construct(exec, List::empty());
   proto->put(exec, exec->propertyNames().constructor, func, ReadOnly | DontDelete | DontEnum);