From 71408911750ed7101f96bbe8a94ce06757052273 Mon Sep 17 00:00:00 2001 From: Aaron Patterson Date: Wed, 25 Sep 2013 10:46:08 -0700 Subject: hash insertion order doesn't matter anymore, so only loop over the owners once --- .../lib/active_record/associations/preloader/association.rb | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'activerecord/lib') diff --git a/activerecord/lib/active_record/associations/preloader/association.rb b/activerecord/lib/active_record/associations/preloader/association.rb index bd63b40472..69b65982b3 100644 --- a/activerecord/lib/active_record/associations/preloader/association.rb +++ b/activerecord/lib/active_record/associations/preloader/association.rb @@ -73,7 +73,7 @@ module ActiveRecord owner_keys = owners_map.keys.compact # Each record may have multiple owners, and vice-versa - records_by_owner = Hash.new do |h,owner| + records_by_owner = owners.each_with_object({}) do |owner,h| h[owner] = [] end @@ -90,9 +90,7 @@ module ActiveRecord end end - owners.each_with_object(records_by_owner) do |owner,h| - h[owner] ||= [] - end + records_by_owner end def load_slices(slices) -- cgit v1.2.3