From fef781cac4cb6ff6482307f3c253bdc01d4bed82 Mon Sep 17 00:00:00 2001 From: Carlos Antonio da Silva Date: Mon, 25 Jun 2012 14:38:42 -0300 Subject: Use default + merge! instead of reverse_merge Since we're not directly requiring the reverse_merge extension, we can avoid another hash creation by using default_hash.merge! instead. --- activemodel/lib/active_model/validations/acceptance.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'activemodel/lib/active_model') diff --git a/activemodel/lib/active_model/validations/acceptance.rb b/activemodel/lib/active_model/validations/acceptance.rb index c49f831cae..43651094cf 100644 --- a/activemodel/lib/active_model/validations/acceptance.rb +++ b/activemodel/lib/active_model/validations/acceptance.rb @@ -4,7 +4,7 @@ module ActiveModel module Validations class AcceptanceValidator < EachValidator #:nodoc: def initialize(options) - super(options.reverse_merge(:allow_nil => true, :accept => "1")) + super({ :allow_nil => true, :accept => "1" }.merge!(options)) end def validate_each(record, attribute, value) -- cgit v1.2.3