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); On 2010/07/23 08:36:23, Mads Ager wrote:
You do need a check for less than zero here, right? argc_bound can
easily be
larger than this.length.
Done. 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); On 2010/07/23 08:36:23, Mads Ager wrote:
Add another case where you bind more arguments than foo.length.
Also, space between 'f' and '='.
Done. http://codereview.chromium.org/3046010/show -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev