Bug Tracker

Opened 12 years ago

Closed 12 years ago

Last modified 10 years ago

#2803 closed enhancement (fixed)

Faster Map

Reported by: flesler Owned by: flesler
Priority: major Milestone: 1.2.4
Component: core Version: 1.2.3
Keywords: map Cc:
Blocked by: Blocking:

Description (last modified by flesler)

These changes make jQuery.map considerably faster.

Attachments (1)

faster-map.diff​ (780 bytes) - added by flesler 12 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 12 years ago by flesler

Description: modified (diff)
Status: new → assigned

Changed 12 years ago by flesler

Attachment: faster-map.diff​ added

comment:2 Changed 12 years ago by flesler

Resolution: → fixed
Status: assigned → closed

Applied at [5470].

comment:3 Changed 10 years ago by gregg.lind

Is the change to ret.concat.apply here redundant?

1203 return ret;

1199 return ret.concat.apply( [], ret );

comment:4 Changed 10 years ago by ajpiano

No, as we established during our discussion of this, that is how nested arrays are flattened. I filed a pull request to add a comment to the source.

Note: See TracTickets for help on using tickets.