From 1620fd5408a72fe4c919314ef6800f744cb45fdf Mon Sep 17 00:00:00 2001 From: Havoc Pennington Date: Wed, 21 May 2003 03:45:58 +0000 Subject: [PATCH] the mask display->ignored_modifier_mask wasn't being bound, due to "<" 2003-05-20 Havoc Pennington * src/keybindings.c (meta_change_keygrab): the mask display->ignored_modifier_mask wasn't being bound, due to "<" instead of "<=" (most people didn't notice as display->ignored_modifier_mask included Scroll_Lock). Red Hat bugzilla #91301 reported by Youssef Makki * src/display.c (meta_change_button_grab): make corresponding change for button grabs. --- ChangeLog | 11 +++++++++++ src/display.c | 4 ++-- src/keybindings.c | 4 ++-- 3 files changed, 15 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index c9ffd485f..3bea6eb19 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,14 @@ +2003-05-20 Havoc Pennington + + * src/keybindings.c (meta_change_keygrab): the mask + display->ignored_modifier_mask wasn't being bound, + due to "<" instead of "<=" (most people didn't notice + as display->ignored_modifier_mask included Scroll_Lock). + Red Hat bugzilla #91301 reported by Youssef Makki + + * src/display.c (meta_change_button_grab): make corresponding + change for button grabs. + 2003-05-20 Havoc Pennington * NEWS: update diff --git a/src/display.c b/src/display.c index 17ce17404..eceaa0427 100644 --- a/src/display.c +++ b/src/display.c @@ -2978,12 +2978,12 @@ meta_change_button_grab (MetaDisplay *display, int button, int modmask) { - int ignored_mask; + unsigned int ignored_mask; meta_error_trap_push (display); ignored_mask = 0; - while (ignored_mask < (int) display->ignored_modifier_mask) + while (ignored_mask <= display->ignored_modifier_mask) { if (ignored_mask & ~(display->ignored_modifier_mask)) { diff --git a/src/keybindings.c b/src/keybindings.c index 56ee41169..395b30b8b 100644 --- a/src/keybindings.c +++ b/src/keybindings.c @@ -925,7 +925,7 @@ meta_change_keygrab (MetaDisplay *display, unsigned int keycode, int modmask) { - int ignored_mask; + unsigned int ignored_mask; /* Grab keycode/modmask, together with * all combinations of ignored modifiers. @@ -942,7 +942,7 @@ meta_change_keygrab (MetaDisplay *display, meta_error_trap_push (display); ignored_mask = 0; - while (ignored_mask < (int) display->ignored_modifier_mask) + while (ignored_mask <= display->ignored_modifier_mask) { if (ignored_mask & ~(display->ignored_modifier_mask)) {