I don't like assigning reassigning parameters of a method. I think that there is some value to be gained from always knowing that your parameters in a method are not going to change throughout the execution of the method (this includes mutating parameters with methods). Unless you see a large performance impact or readability improvements that we can get by re-assigning the input parameter, I'd like to leave this as is.