From 5de69c22b0ff037f648a740a7c01869e78587df2 Mon Sep 17 00:00:00 2001 From: Franklin Wei Date: Sun, 21 Jul 2024 18:06:37 -0400 Subject: 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. --- filling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'filling.c') diff --git a/filling.c b/filling.c index 80a06dd..ccae7e2 100644 --- a/filling.c +++ b/filling.c @@ -1474,7 +1474,7 @@ static char *interpret_move(const game_state *state, game_ui *ui, assert(ui); assert(ds); - button &= ~MOD_MASK; + button = STRIP_BUTTON_MODIFIERS(button); if (button == LEFT_BUTTON || button == LEFT_DRAG) { /* A left-click anywhere will clear the current selection. */ -- cgit v1.1