http://codereview.chromium.org/160580/diff/1/4
File test/mjsunit/regress/regress-416.js (right):

http://codereview.chromium.org/160580/diff/1/4#newcode32
Line 32: assertEquals((new Date(1e81)).valueOf(), Number.NaN, "new
Date(1e81)");
This seems backwards to me. The expected value should go before the
actual value and you can't really compare NaN's this way. Shouldn't it
be something like:

assertTrue(isNaN(new Date(1e81).getTime()), "new Date(1e81");

http://codereview.chromium.org/160580

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

Reply via email to