Refactor `button & ~MOD_MASK' as `STRIP_BUTTON_MODIFIERS(button)'.

This refactors all instances of bitwise-ANDs with `~MOD_MASK'. There is
a handful of more complex instances I left unchanged (in cube.c, midend.c,
and twiddle.c), since those AND with `~MOD_MASK | MOD_NUM_KEYPAD' or
similar. I don't think it's worth writing a macro for those cases.

Also document this new macro's usage in devel.but.

1 files changed, 1 insertions, 1 deletions

diff --git a/palisade.c b/palisade.c
index 2f3545c..811204b 100644
--- a/palisade.c
+++ b/palisade.c
@@ -912,7 +912,7 @@ static char *interpret_move(const game_state *state, game_ui *ui,
     int w = state->shared->params.w, h = state->shared->params.h;
     bool control = button & MOD_CTRL, shift = button & MOD_SHFT;
 
-    button &= ~MOD_MASK;
+    button = STRIP_BUTTON_MODIFIERS(button);
 
     if (button == LEFT_BUTTON || button == RIGHT_BUTTON) {
         int gx = FROMCOORD(x), gy = FROMCOORD(y), possible = BORDER_MASK;