This project is archived and is in readonly mode.
inject should be replaced with map and each
Reported by Aaron Patterson | March 6th, 2010 @ 11:34 PM
Inject is pretty slow, and many times each or map are better suited for the job at hand. I've attached a couple small refactors to AR::Base.
Comments and changes to this ticket
-
Repository March 7th, 2010 @ 04:32 AM
- State changed from “new” to “committed”
(from [5a806f67591978a4f65af12ddc3595c7c356b49e]) converting inject to each and map
[#4119 state:committed]
Signed-off-by: Jeremy Kemper jeremy@bitsweat.net
http://github.com/rails/rails/commit/5a806f67591978a4f65af12ddc3595... -
Repository March 7th, 2010 @ 04:32 AM
(from [3789539916bcfbf82ee01f9aa35fca9f32049ea8]) converting inject to each and map
[#4119 state:committed]
Signed-off-by: Jeremy Kemper jeremy@bitsweat.net
http://github.com/rails/rails/commit/3789539916bcfbf82ee01f9aa35fca...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
Aaron Patterson
Attachments
Referenced by
-
4119
inject should be replaced with map and each
[#4119 state:committed]
-
4119
inject should be replaced with map and each
[#4119 state:committed]