Peter Hutterer <peter.hutte...@who-t.net> writes: > Not sure what if anything calls XSetDeviceModifierMapping() but this would've > failed all the time. check_modmap_change() returns Successs but we were > treating it like a boolean. Fix this.
Yeah, and check_modmap_change_slave *does* return a boolean; I see no possibility for confusion here. I saw that check_modmap_change returns -1 for BadValue and then checked to make sure this odd value is fixed up in both ProcXSetDeviceModifierMapping and ProcSetModifierMapping so that this value doesn't get back to the client. Reviewed-by: Keith Packard <kei...@keithp.com> -- -keith
signature.asc
Description: PGP signature
_______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel