Ticket #2056 (closed enhancement: fixed)
performance enhancements
| Reported by: | greg | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | 1.2.4 |
| Component: | core | Version: | 1.2.1 |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description
This patch increases the performance of merge and filter. There are also some various other changes for minor performance increases or increased readability, and a collect function- a version of map without flattening, which would be more useful for users than the current map (not sure if it would be useful internally)
The merge change is to simply go two steps at a time instead of one. The filter saves the evaled functions. Note that two variable had to be given increased scope. If this is unacceptable, we can discuss ways to hide them.
Attachments
-
jquery.patch
(13.2 KB) -
added by greg 5 years ago.
- patch file
Change History
Changed 5 years ago by greg
-
attachment
jquery.patch
added
comment:1 Changed 5 years ago by flesler
- Status changed from new to closed
- Resolution set to fixed
Many have been already implemented. Most take >= code and they don't really add noticeable improvements.
It'd be good if you could add these changes separatedly, so it's easier to handle and commit. I'll close as fixed for those already implemented.
If you have specific changes, please add a new ticket.
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
patch file