Re: [PATCH xserver] dix: check_modmap_change() returns Success, not true

2018-08-08 Thread Keith Packard
Peter Hutterer 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

[PATCH xserver] dix: check_modmap_change() returns Success, not true

2018-08-07 Thread Peter Hutterer
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. Reported-by: Adam Jackson Signed-off-by: Peter Hutterer --- dix/inpututils.c | 2 +- 1 file changed, 1