That depends on the semantics. If the function returns the Person object with a particular name (obtained from an expensive dB query) then you want the mutated

That depends on the semantics. If the function returns the Person object with a particular name (obtained from an expensive dB query) then you want the mutated Person returned, not the now stale value that was returned on the previous call to the function.