Changeset 27255 in webkit for trunk/JavaScriptCore/kjs/nodes2string.cpp

Timestamp:

Oct 29, 2007, 10:36:24 PM (18 years ago)

Author:

oliver

Message:

Debranching remaining assignment nodes, and miscellaneous cleanup

Reviewed by Maciej.

Split read-modify code paths out of AssignBracketNode and AssignDotNode
Removed now unnecessary check for write-only assignment in ReadModifyLocalVarNode
and ReadModifyResolveNode evaluate methods

Leads to a 1% gain in SunSpider.

File:

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

trunk/JavaScriptCore/kjs/nodes2string.cpp

@@ -674 +674 @@
 }
 
-void AssignBracketNode::streamTo(SourceStream& s) const
+void ReadModifyBracketNode::streamTo(SourceStream& s) const
 {
     s << PrecCall << m_base << '[' << m_subscript << "] "
@@ -680 +680 @@
 }
 
-void AssignDotNode::streamTo(SourceStream& s) const
+void AssignBracketNode::streamTo(SourceStream& s) const
+{
+    s << PrecCall << m_base << '[' << m_subscript << "] = " << PrecAssignment << m_right;
+}
+
+void ReadModifyDotNode::streamTo(SourceStream& s) const
 {
     s << DotExpr << PrecCall << m_base << "." << m_ident << ' '
         << operatorString(m_oper) << ' ' << PrecAssignment << m_right;
+}
+
+void AssignDotNode::streamTo(SourceStream& s) const
+{
+    s << DotExpr << PrecCall << m_base << "." << m_ident << " = " << PrecAssignment << m_right;
 }