From d2405a0aab6b246842163cdb1a40d2d300b7b879 Mon Sep 17 00:00:00 2001 From: Aaron Patterson Date: Tue, 14 May 2013 16:15:57 -0700 Subject: use inject rather than multiple assignments --- activesupport/lib/active_support/callbacks.rb | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'activesupport/lib/active_support/callbacks.rb') diff --git a/activesupport/lib/active_support/callbacks.rb b/activesupport/lib/active_support/callbacks.rb index 429e4f60a4..e6e2fcfb3f 100644 --- a/activesupport/lib/active_support/callbacks.rb +++ b/activesupport/lib/active_support/callbacks.rb @@ -540,11 +540,9 @@ module ActiveSupport def compile return @callbacks if @callbacks - @callbacks = Filters::ENDING - @chain.reverse_each do |callback| - @callbacks = callback.apply(@callbacks) + @callbacks = @chain.reverse.inject(Filters::ENDING) do |chain, callback| + callback.apply chain end - @callbacks end def append(*callbacks) -- cgit v1.2.3