Revision: 4817
Author: [email protected]
Date: Tue Jun  8 04:20:36 2010
Log: Fix bug in mjsunit/const-eval-init causing objects to be initialized to undefined.

In mjsunit/const-eval-init - testInitSlowCaseExtension a range of objects are initialized to undefined instead of a value because the variable i is within quotes (i.e., the source becommes "a1 = i" instead of "a1 = 1".

This should have no impact on the test, I just stumbled over this on
an unrelated matter.

Review URL: http://codereview.chromium.org/2758002
http://code.google.com/p/v8/source/detail?r=4817

Modified:
 /branches/bleeding_edge/test/mjsunit/const-eval-init.js

=======================================
--- /branches/bleeding_edge/test/mjsunit/const-eval-init.js Thu Jan 22 05:53:06 2009 +++ /branches/bleeding_edge/test/mjsunit/const-eval-init.js Tue Jun 8 04:20:36 2010
@@ -91,7 +91,7 @@
   var source = "";
   // Introduce 100 properties on the context extension object to force
   // it in slow case.
-  for (var i = 0; i < 100; i++) source += ("var a" + i + " = i;");
+  for (var i = 0; i < 100; i++) source += ("var a" + i + " = " + i + ";");
source += "const x = 10; assertEquals(10, x); x = 11; assertEquals(10, x)";
   eval(source);
 }

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to