# HG changeset patch # User Christian Brabandt # Date 1706980502 -3600 # Node ID 22be86ed7545024cd92f3b1a08dee08187a56506 # Parent a011b0b275acb48f89d043475bec7ad73373602d patch 9.1.0073: Looping over modifier_keys_table unnecessarily Commit: https://github.com/vim/vim/commit/0c989e4a3ae50085aa8c6bed5d6701760191bc1d Author: zeertzjq Date: Sat Feb 3 18:04:05 2024 +0100 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 Signed-off-by: Christian Brabandt diff --git a/src/misc2.c b/src/misc2.c --- a/src/misc2.c +++ b/src/misc2.c @@ -1130,11 +1130,7 @@ simplify_key(int key, int *modifiers) 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 diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -705,6 +705,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 73, +/**/ 72, /**/ 71,