LGTM
http://codereview.chromium.org/6532087/diff/1/src/v8natives.js File src/v8natives.js (right): http://codereview.chromium.org/6532087/diff/1/src/v8natives.js#newcode95 src/v8natives.js:95: if (IS_UNDEFINED(radix) || radix == 10 || radix == 0) { Erik suggests using === here. I agree with that and we should do so below as well (the checks before %_GetCachedArrayIndex). http://codereview.chromium.org/6532087/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
