are you talking about modifying the string in place and then returning a boolean? As a general rule I think it's better to avoid modifying data received through arguments and instead leave functions side-effect free, but I'm fine doing that if you prefer.