updated snapshot


http://codereview.appspot.com/135054/diff/1/3
File src/com/google/caja/cajita.js (right):

http://codereview.appspot.com/135054/diff/1/3#newcode1598
Line 1598: var args = [self].concat(Array.slice(opt_args, 0));
On 2009/10/21 19:33:54, MikeSamuel wrote:
> that doesn't work, because opt_args might be an arguments object,
> so we have to call Array.slice anyway.

Really?  I thought apply could take an array like object, like
arguments, and so
calling push indirectly would push the arguments onto the end of the
single
element array.

oh, right.  arguments does works with apply.  I was thinking of
somewhere
else where I accidentally had foo.concat(args), which behaves
differently
when args is an array and when args is an arguments object.

ok.

http://codereview.appspot.com/135054

Reply via email to