switch sense of removemod

This just makes more sense to me, as it's a combination
hasmod/removemod.
fix modlock logic when mod already pressed
detailed comments for each press in chorder_test
reset map in a single place
allow macros to have independent mod tracking