Since we've got a pointer already, I suspect we don't want to be passing a reference or a copy. Should we change the hit_player declaration instead, to take a pointer or const pointer (as appropriate)?