LGTM.

https://chromiumcodereview.appspot.com/10388164/diff/1/test/mjsunit/regress/regress-2071.js
File test/mjsunit/regress/regress-2071.js (right):

https://chromiumcodereview.appspot.com/10388164/diff/1/test/mjsunit/regress/regress-2071.js#newcode35
test/mjsunit/regress/regress-2071.js:35: print(b);
nit: is this print() statement needed?

https://chromiumcodereview.appspot.com/10388164/diff/1/test/mjsunit/regress/regress-2071.js#newcode42
test/mjsunit/regress/regress-2071.js:42: for(var i = 0; i < 10000; i++)
{
nit: I think 1000 iterations is enough. Let's let the test suite run a
little faster :-)

https://chromiumcodereview.appspot.com/10388164/

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

Reply via email to