From 7fdfabd753a39dbdcfcfa65ec33d0192448119d8 Mon Sep 17 00:00:00 2001 From: Daniel Stone Date: Thu, 6 Apr 2017 15:52:10 +0100 Subject: keycodes: Ignore high keycodes Rather than throwing a fatal error when a keycode definition exceeds the declared maximum (i.e. 255), just ignore the definition and continue. This allows xkeyboard-config to start shipping datasets including high keycodes, which will work in xkbcommon as it ignores explicit range declarations. Signed-off-by: Daniel Stone Reviewed-by: Ran Benita Reviewed-by: Peter Hutterer Reported-by: Christian Kellner --- keycodes.c | 28 ++++++++++++++++++++-------- 1 file changed, 20 insertions(+), 8 deletions(-) (limited to 'keycodes.c') diff --git a/keycodes.c b/keycodes.c index 22d9eae..d5ac762 100644 --- a/keycodes.c +++ b/keycodes.c @@ -31,6 +31,8 @@ #include "misc.h" #include "alias.h" +static Bool high_keycode_warned; + char * longText(unsigned long val, unsigned format) { @@ -330,10 +332,15 @@ AddKeyName(KeyNamesInfo * info, if ((kc < info->effectiveMin) || (kc > info->effectiveMax)) { - ERROR2("Illegal keycode %d for name <%s>\n", kc, name); - ACTION2("Must be in the range %d-%d inclusive\n", - info->effectiveMin, info->effectiveMax); - return False; + if (!high_keycode_warned) + { + WARN2("Unsupported high keycode %d for name <%s> ignored\n", + kc, name); + ACTION2("X11 cannot support keycodes above 255.\n"); + ACTION2("This warning only shows for the first high keycode.\n"); + high_keycode_warned = True; + } + return True; } if (kc < info->computedMin) info->computedMin = kc; @@ -589,10 +596,15 @@ HandleKeycodeDef(KeycodeDef * stmt, unsigned merge, KeyNamesInfo * info) code = result.ival; if ((code < info->effectiveMin) || (code > info->effectiveMax)) { - ERROR2("Illegal keycode %d for name <%s>\n", code, stmt->name); - ACTION2("Must be in the range %d-%d inclusive\n", - info->effectiveMin, info->effectiveMax); - return 0; + if (!high_keycode_warned) + { + WARN2("Unsupported high keycode %d for name <%s> ignored\n", + code, stmt->name); + ACTION2("X11 cannot support keycodes above 255.\n"); + ACTION2("This warning only shows for the first high keycode.\n"); + high_keycode_warned = True; + } + return 1; } if (stmt->merge != MergeDefault) { -- cgit v1.2.3