On Tue, Dec 9, 2008 at 2:18 AM, Frederick Cheung
<frederick.cheung@gmail.com> wrote:> I stumbled across
http://rails.lighthouseapp.com/projects/8994/tickets/1352-enumerablegroup_by-is-on2
> recently
>
> A quick investigation reveals that the complexity with respect to the
> number of groups is indeed noticeably worse. The commit that
> introduced this (
http://github.com/rails/rails/commit/8273280a637a5e40de2c37dd1da507c749b11aab
> ) does make that particular code a lot neater, but was the
> performance regression an intended side-effect ?
Nope. Fixed; thanks for the patch!
jeremy
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Ruby on Rails: Core" group.
To post to this group, send email to rubyonrails-core@googlegroups.com
To unsubscribe from this group, send email to
rubyonrails-core+unsubscribe@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---