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

Timestamp:

Aug 16, 2002, 12:43:56 AM (23 years ago)

Author:

mjs

Message:

Phase 1 of optimization to stop allocating references through the
collector. This step clearly splits evaluating to a reference and
evaluating to a value, and moves all of the reference-specific
operations from Value to Reference. A special ConstReference class
helps out for the one case where you need special reference
operations if the result is a reference, and not otherwise.

Also, Reference now inherits privately from Value, and there is a
new ReferenceList class that inherits privately from List, so the
uses of Reference and Value are now completely orthogonal. This
means that as the next step, their implementations can be
completely disentangled.

This step has no actual performance impact.

• kjs/collector.cpp: (Collector::collect):
• kjs/nodes.cpp: (Node::evaluateReference): (ResolveNode::evaluate): (ResolveNode::evaluateReference): (ElementNode::evaluate): (PropertyValueNode::evaluate): (AccessorNode1::evaluate): (AccessorNode1::evaluateReference): (AccessorNode2::evaluate): (AccessorNode2::evaluateReference): (ArgumentListNode::evaluateList): (NewExprNode::evaluate): (FunctionCallNode::evaluate): (PostfixNode::evaluate): (DeleteNode::evaluate): (VoidNode::evaluate): (TypeOfNode::evaluate): (PrefixNode::evaluate): (UnaryPlusNode::evaluate): (NegateNode::evaluate): (BitwiseNotNode::evaluate): (LogicalNotNode::evaluate): (MultNode::evaluate): (AddNode::evaluate): (ShiftNode::evaluate): (RelationalNode::evaluate): (EqualNode::evaluate): (BitOperNode::evaluate): (BinaryLogicalNode::evaluate): (ConditionalNode::evaluate): (AssignNode::evaluate): (CommaNode::evaluate): (VarDeclNode::evaluate): (ExprStatementNode::execute): (IfNode::execute): (DoWhileNode::execute): (WhileNode::execute): (ForNode::execute): (ForInNode::execute): (ReturnNode::execute): (WithNode::execute): (CaseClauseNode::evaluate): (SwitchNode::execute): (ThrowNode::execute):
• kjs/nodes.h:
• kjs/types.cpp: (ConstReference::ConstReference):
• kjs/types.h:
• kjs/value.h:

File:

• trunk/JavaScriptCore/kjs/object.h (modified) (5 diffs)

trunk/JavaScriptCore/kjs/object.h

@@ -34 +34 @@
 #include "value.h"
 #include "types.h"
+#include "reference_list.h"
 
 namespace KJS {
@@ -42 +43 @@
   class HashEntry;
   class ListImp;
+  class ReferenceList;
 
   // ECMA 262-3 8.6.1
@@ -331 +333 @@
      * @return A List of References to properties of the object.
      **/
-    List propList(ExecState *exec, bool recursive = true);
+    ReferenceList propList(ExecState *exec, bool recursive = true);
 
     /**
@@ -561 +563 @@
     void setScope(const List &s);
 
-    List propList(ExecState *exec, bool recursive = true);
+    ReferenceList propList(ExecState *exec, bool recursive = true);
 
     Value internalValue() const;
@@ -689 +691 @@
     { imp()->setScope(s); }
 
-  inline List Object::propList(ExecState *exec, bool recursive)
+  inline ReferenceList Object::propList(ExecState *exec, bool recursive)
     { return imp()->propList(exec,recursive); }