I'll revert that bit of the change for now then until we determine whether double damage is intended or not. The if statement is in the other spot in the code where the damage is initially applied, and if double damage is the intent I think that should be moved to where the rest of the damage is applied to make the intent clear (player::apply_damage() as opposed to player::deal_damage()). I could probably also eliminate many of the other comments.