http://bugs.winehq.org/show_bug.cgi?id=735
------- Additional Comments From dmitry@codeweavers.com 2007-09-04 03:50 ------- Your patch detects an X11 keyboard layout by its name. In order to work on wide range of systems it requires to hard code all/most existing X11 keyboard layouts into Wine's keyboard.c. I'm not sure it's better than what we have now. Besides it won't work if somebody changes/updates/adds a new layout.
The biggest problem with X11 keyboard APIs is that it doesn't report the locale associated with a keyboard layout, i.e. it doesn't have a notion of the input locale.
Also DebugBreak should be replaced by assert(0).