Aliases should be to .call

[jdunck]

You have a bunch of aliases of the form:
var slice = Array.prototype.slice

In most, if not all, cases, you then use that as slice.call.

If it's truly done for performance, your aliases should be to
var slicecall = Array.prototype.slice.call

That'll save one more attr lookup in a ton of loops.

[documentcloud]

That's a nice thought, but I'm afraid that pulling off a reference to Function#call isn't going to get you the original method -- it's just going to get you a broken version of call.

slice = Array.prototype.slice ;
sliceCall = Array.prototype.slice.call;

slice.call([1, 2, 3, 4, 5], 2)
=> [3, 4, 5]

sliceCall([1, 2, 3, 4, 5], 2)
=> TypeError: Type error

Closing the ticket.

[jdunck]

Well, I was thinking
sliceCall = _.bind(slice.call, slice)
but I guess that doesn't save anything since .apply still has to be run in the func returned by .bind. :-/

[documentcloud]

Sure, but an extra function invocation is slower than a property access in any case: