One might argue that it would be better to have the function not produce side-effects (ie, not change its incoming parameter), but rather copy the list, add to it, and return it.  There's a performance hit with that, though, so I am fine with whatever you decide is best.