Changeset 2778 in webkit for trunk/JavaScriptCore/kjs/function.h

Timestamp:

Nov 20, 2002, 1:11:43 AM (23 years ago)

Author:

darin

Message:

• on the road to killing ActivationImp

• kjs/function.h: Add get/put to FunctionImp. Remove argumentsObject() from ActivationImp. Add function() to ActivationImp.
• kjs/function.cpp: (FunctionImp::FunctionImp): No arguments property. (FunctionImp::call): No need to set up the arguments property. (FunctionImp::parameterString): Remove strangeness. (FunctionImp::processParameters): Ditto. (FunctionImp::get): Added, handles arguments and length properties. (FunctionImp::put): Ditto. (FunctionImp::hasProperty): Ditto. (FunctionImp::deleteProperty): Ditto. (ActivationImp::ActivationImp): Store a function pointer so we can find it in the context.

• kjs/function_object.cpp: (FunctionObjectImp::construct): No need to set up arguments property.
• kjs/nodes.cpp: (FuncExprNode::evaluate): No need to set up length property.

• kjs/internal.h: Return ObjectImp * for activation object.

• kjs/interpreter.h: Remove stray declaration of ExecStateImp.

File:

• trunk/JavaScriptCore/kjs/function.h (modified) (2 diffs)

trunk/JavaScriptCore/kjs/function.h

@@ -39 +39 @@
     FunctionImp(ExecState *exec, const Identifier &n = Identifier::null);
     virtual ~FunctionImp();
+
+    virtual Value get(ExecState *exec, const Identifier &propertyName) const;
+    virtual void put(ExecState *exec, const Identifier &propertyName, const Value &value, int attr = None);
+    virtual bool hasProperty(ExecState *exec, const Identifier &propertyName) const;
+    virtual bool deleteProperty(ExecState *exec, const Identifier &propertyName);
 
     virtual void mark();
@@ -99 +104 @@
     ~ActivationImp();
 
-    Object argumentsObject() { return Object(arguments); }
-
     virtual const ClassInfo *classInfo() const { return &info; }
     static const ClassInfo info;
+    
+    FunctionImp *function() const { return _function; }
+    
   private:
+    FunctionImp *_function;
     ObjectImp* arguments;
   };