LGTM
http://codereview.chromium.org/519061/diff/1/7 File src/array.js (right): http://codereview.chromium.org/519061/diff/1/7#newcode113 src/array.js:113: if (UseSparseVariant(array, length, is_array) && (separator.length == 0)) { I don't think the extra () around separator.length == 0 really add anything here. http://codereview.chromium.org/519061/diff/1/2 File test/mjsunit/fuzz-natives.js (right): http://codereview.chromium.org/519061/diff/1/2#newcode100 test/mjsunit/fuzz-natives.js:100: // may lead to out-of-memory. This is somehow unsatisfactory. http://codereview.chromium.org/519061
-- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
