Removed active/inactive tracking logic from the Player class.
This is currently duplicated by the GameState class and trying to
duplicate it in both places is (a) unnecessary and (b) likely to
lead to bugs down the line