Changeset 45106 in webkit for trunk/JavaScriptCore/parser/NodeConstructors.h

Timestamp:

Jun 24, 2009, 3:01:34 PM (16 years ago)

Author:

[email protected]

Message:

<rdar://problem/6940519> REGRESSION (Safari 4 Public Beta - TOT): google.com/adplanner shows blank page instead of site details in "basic research'

Reviewed by Darin Adler.

The problem was caused by the page returned with a function using a
var declaration list containing around ~3000 variables. The solution
to this is to flatten the comma expression representation and make
codegen comma expressions and initializer lists iterative rather than
recursive.

File:

• trunk/JavaScriptCore/parser/NodeConstructors.h (modified) (1 diff)

trunk/JavaScriptCore/parser/NodeConstructors.h

@@ -660 +660 @@
     inline CommaNode::CommaNode(JSGlobalData* globalData, ExpressionNode* expr1, ExpressionNode* expr2)
         : ExpressionNode(globalData)
-        , m_expr1(expr1)
-        , m_expr2(expr2)
-    {
+    {
+        m_expressions.append(expr1);
+        m_expressions.append(expr2);
     }