https://chromiumcodereview.appspot.com/9307083/diff/2001/src/date.js File src/date.js (right):
https://chromiumcodereview.appspot.com/9307083/diff/2001/src/date.js#newcode674 src/date.js:674: // SET_DATE_LOCAL(this, t); On 2012/03/08 08:12:42, ulan wrote:
Commented code here and below.
Done. https://chromiumcodereview.appspot.com/9307083/diff/2001/src/objects-printer.cc File src/objects-printer.cc (right): https://chromiumcodereview.appspot.com/9307083/diff/2001/src/objects-printer.cc#newcode679 src/objects-printer.cc:679: weekdays[weekday()->IsSmi()? Smi::cast(weekday())->value() + 1 : -1], On 2012/03/08 08:12:42, ulan wrote:
if weekday is not a Smi, the index should be 0 not -1
Done. https://chromiumcodereview.appspot.com/9307083/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
