Modifying original argument variables is not the best idea. This isn't a dealbreaker, but you should still call it something else.