From f7d509882ef1d5a54cb2b15e871cd70e4af3c5ed Mon Sep 17 00:00:00 2001 From: Jeremy Kemper Date: Fri, 6 Feb 2009 12:18:24 -0800 Subject: Prefer tap to returning --- activeresource/test/setter_trap.rb | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'activeresource/test') diff --git a/activeresource/test/setter_trap.rb b/activeresource/test/setter_trap.rb index 9a899b0fa0..7cfd9ca111 100644 --- a/activeresource/test/setter_trap.rb +++ b/activeresource/test/setter_trap.rb @@ -1,9 +1,8 @@ class SetterTrap < ActiveSupport::BasicObject class << self def rollback_sets(obj) - returning yield(setter_trap = new(obj)) do - setter_trap.rollback_sets - end + trapped = new(obj) + yield(trapped).tap { trapped.rollback_sets } end end -- cgit v1.2.3