Bug Tracker

Modify ↓

Ticket #2803 (closed enhancement: fixed)

Opened 5 years ago

Last modified 3 years ago

Faster Map

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

Description (last modified by flesler) (diff)

These changes make jQuery.map considerably faster.

Attachments

faster-map.diff Download (780 bytes) - added by flesler 5 years ago.

Change History

comment:1 Changed 5 years ago by flesler

  • Status changed from new to assigned
  • Description modified (diff)

Changed 5 years ago by flesler

comment:2 Changed 5 years ago by flesler

  • Status changed from assigned to closed
  • Resolution set to fixed

Applied at [5470].

comment:3 Changed 3 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 3 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.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View ↑

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑
 
Note: See TracTickets for help on using tickets.