Tap that object

tap_or_no_tap.rb

# To tap
def activate_person(person_id)
  Person.find(person_id).tap do |person|
    person.activate!
  end
end

# or not to tap
def activate_person(person_id)
  person = Person.find(person_id)
  person.activate!
  person
end

# that is the question

@glv @pjb3 A few months back @brixen saw me using #tap and lamented on how it's not ideal because it creates a new scope when a perfectly suitable one already exists. From what I remember he said it's not worth extending the stack for no good reason. Why make the VM do more work when the context is already set up already? @brixen, did I get that right?