Lgtm

http://codereview.chromium.org/88022/diff/1/2
File src/array.js (right):

http://codereview.chromium.org/88022/diff/1/2#newcode891
Line 891: find_initial: if (%_ArgumentsLength() < 2) {
I think the code style is

  find_initial:
   if (%_...)

http://codereview.chromium.org/88022/diff/1/4
File test/mjsunit/array-reduce.js (right):

http://codereview.chromium.org/88022/diff/1/4#newcode29
Line 29: * @fileoverview Test reduce and reduceRight
This is an excellent test!

http://codereview.chromium.org/88022

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

Reply via email to