commit 0c989e4a3ae50085aa8c6bed5d6701760191bc1d
author zeertzjq <zeertzjq@outlook.com> Sat Feb 03 18:04:05 2024 +0100
committer Christian Brabandt <cb@256bit.org> Sat Feb 03 18:05:38 2024 +0100
tree cb7f1e541840db281e2b49fb93a9178cfbcf8c31
parent 609370392a7b88dc2db38d01577509575216ff36

patch 9.1.0073: Looping over modifier_keys_table unnecessarily

Problem:  Looping over modifier_keys_table[] unnecessarily with only
          MOD_MASK_ALT or MOD_MASK_CMD, as modifier_keys_table[] only
          contains MOD_MASK_SHIFT and MOD_MASK_CTRL, and the loop won't
          do anything.
Solution: Remove MOD_MASK_ALT and MOD_MASK_CMD from the condition.
          (zeertzjq)

closes: #13963

Signed-off-by: zeertzjq <zeertzjq@outlook.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>

src/misc2.c

diff --git a/src/misc2.c b/src/misc2.c
index 57ee287..611e021 100644
--- a/src/misc2.c
+++ b/src/misc2.c
@@ -1130,11 +1130,7 @@
     int	    key0;
     int	    key1;
 
-    if (!(*modifiers & (MOD_MASK_SHIFT | MOD_MASK_CTRL | MOD_MASK_ALT
-#ifdef FEAT_GUI_GTK
-	    | MOD_MASK_CMD
-#endif
-    )))
+    if (!(*modifiers & (MOD_MASK_SHIFT | MOD_MASK_CTRL)))
 	return key;
 
     // TAB is a special case