Fix length setting and this LGTM.
http://codereview.chromium.org/3046010/diff/17001/18003 File src/v8natives.js (right): http://codereview.chromium.org/3046010/diff/17001/18003#newcode1146 src/v8natives.js:1146: %FunctionSetLength(result, this.length - argc_bound); You do need a check for less than zero here, right? argc_bound can easily be larger than this.length. http://codereview.chromium.org/3046010/diff/17001/18004 File test/mjsunit/function-bind.js (right): http://codereview.chromium.org/3046010/diff/17001/18004#newcode47 test/mjsunit/function-bind.js:47: f= foo.bind(foo, 2, 2, 2); Add another case where you bind more arguments than foo.length. Also, space between 'f' and '='. http://codereview.chromium.org/3046010/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
