Merge pull request #18885 from ypxing/master

remove unnecessary assignment/parameter passing in AS::Callbacks...::Around (master branch) 

activesupport/lib/active_support/callbacks.rb

@@ -291,13 +291,11 @@ def self.halting_and_conditional(callback_sequence, user_callback, user_conditio
 
             if !halted && user_conditions.all? { |c| c.call(target, value) }
               user_callback.call(target, value) {
-                env = run.call env
-                env.value
+                run.call.value
               }
-
               env
             else
-              run.call env
+              run.call
             end
           end
         end
@@ -309,11 +307,10 @@ def self.halting(callback_sequence, user_callback)
             value  = env.value
 
             if env.halted
-              run.call env
+              run.call
             else
               user_callback.call(target, value) {
-                env = run.call env
-                env.value
+                run.call.value
               }
               env
             end
@@ -328,12 +325,11 @@ def self.conditional(callback_sequence, user_callback, user_conditions)
 
             if user_conditions.all? { |c| c.call(target, value) }
               user_callback.call(target, value) {
-                env = run.call env
-                env.value
+                run.call.value
               }
               env
             else
-              run.call env
+              run.call
             end
           end
         end
@@ -342,8 +338,7 @@ def self.conditional(callback_sequence, user_callback, user_conditions)
         def self.simple(callback_sequence, user_callback)
           callback_sequence.around do |env, &run|
             user_callback.call(env.target, env.value) {
-              env = run.call env
-              env.value
+              run.call.value
             }
             env
           end