Remove unnecessary dup in ActiveModel::Serializer::Associations#associate

Empact · Empact · commit 8ac1f107f43d · 2016-01-12T14:42:07.000-08:00
The `_reflections` are duped on `inherited` - no need to `dup` them
with each addition.
diff --git a/lib/active_model/serializer/associations.rb b/lib/active_model/serializer/associations.rb
@@ -76,8 +76,6 @@ def has_one(name, options = {}, &block)
         # @api private
         #
         def associate(reflection)
-          self._reflections = _reflections.dup
-
           self._reflections << reflection
         end
       end

Original file line number	Diff line number	Diff line change
`@@ -76,8 +76,6 @@ def has_one(name, options = {}, &block)`
`76`	`76`	`# @api private`
`77`	`77`	`#`
`78`	`78`	`def associate(reflection)`
`79`		`- self._reflections = _reflections.dup`
`80`		`-`
`81`	`79`	`self._reflections << reflection`
`82`	`80`	`end`
`83`	`81`	`end`