Bug 21910

Summary: include/X11/Xutil.h: IsModifierKey misses some modifiers
Product: xorg Reporter: Andreas Wettstein <wettstae>
Component: Lib/XlibAssignee: Xorg Project Team <xorg-team>
Status: RESOLVED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: major    
Priority: medium CC: jeremyhu, mail
Version: gitKeywords: patch
Hardware: All   
OS: All   
Whiteboard: 2011BRB_Reviewed
i915 platform: i915 features:
Attachments:
Description Flags
patch to fix Xutil.h none

Description Andreas Wettstein 2009-05-24 10:05:44 UTC
The header file include/X11/Xutil.h defines a macro IsModifierKey, which does not account for the modifiers ISO_Level5_Shift, ISO_Level5_Latch, and ISO_Level5_Lock.

To fix, it should be enough to replace XK_ISO_Last_Group_Lock by ISO_Level5_Lock in the macro's definition.
Comment 1 Bodo Graumann 2011-07-15 13:23:13 UTC
Created attachment 49162 [details] [review]
patch to fix Xutil.h

This patch fixes the reported issue.
Comment 2 Bodo Graumann 2011-07-15 13:25:07 UTC
This bug should be fixed to let all compose-combinations work again for the neo keyboard layout.
see: http://wiki.neo-layout.org/ticket/277
Comment 3 Jeremy Huddleston Sequoia 2011-10-08 19:47:16 UTC
Can you please send your patch to xorg-devel for review?
Comment 4 Jeremy Huddleston Sequoia 2011-10-14 17:02:33 UTC
commit 50e15379331c436851abb194251cde96999e67b3
Author: Bodo Graumann <mail@bodograumann.de>
Date:   Mon Oct 10 16:40:52 2011 +0200

    libX11: Fixing modifier key range in Xutil.h (Bug #21910)
    
    IsModifierKey, defined in include/X11/Xutil.h, is a macro determining,
    which keys are regarded as modifiers. The constants ISO_Level5_Shift,
    ISO_Level5_Latch and ISO_Level5_Lock where excluded previously, leaving
    some Neo2 modifiers functionless in combination with compose.
    This patch adjusts the range to include the correct, full range of
    modifier constants.
    
    Neo2 Bug 277 <http://wiki.neo-layout.org/ticket/277>
    
    X.Org Bug 21910 <http://bugs.freedesktop.org/show_bug.cgi?id=21910>
    
    Signed-off-by: Bodo Graumann <mail@bodograumann.de>
    Reviewed-by: Daniel Stone <daniel@fooishbar.org>

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.