This may just be personal style, but when I'm mutating things I like to be explicit that I'm also returning them. I could go either way with killing the return line