From 7b8bff630ce2d98b87ffbf4843e71ca2c97df717 Mon Sep 17 00:00:00 2001 From: Pierre Ossman Date: Mon, 21 Jul 2014 16:17:23 +0200 Subject: [PATCH] Use NoSymbol as the error code This is how the X11 API does things, so it's clearer if we use the same principle. --- vncviewer/Viewport.cxx | 10 +++++++--- vncviewer/keysym2ucs.c | 4 +++- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/vncviewer/Viewport.cxx b/vncviewer/Viewport.cxx index cd8b1bc3..7d551682 100644 --- a/vncviewer/Viewport.cxx +++ b/vncviewer/Viewport.cxx @@ -39,6 +39,10 @@ #include #endif +#ifndef NoSymbol +#define NoSymbol 0 +#endif + #include "Viewport.h" #include "CConn.h" #include "OptionsDialog.h" @@ -702,7 +706,7 @@ rdr::U32 Viewport::translateKeyEvent(int keyCode, int origKeyCode, const char *k // Unknown special key? if (keyText[0] == '\0') { vlog.error(_("Unknown FLTK key code %d (0x%04x)"), keyCode, keyCode); - return XK_VoidSymbol; + return NoSymbol; } // Look up the symbol the key produces and translate that from Unicode @@ -710,7 +714,7 @@ rdr::U32 Viewport::translateKeyEvent(int keyCode, int origKeyCode, const char *k if (fl_utf_nb_char((const unsigned char*)keyText, strlen(keyText)) != 1) { vlog.error(_("Multiple characters given for key code %d (0x%04x): '%s'"), keyCode, keyCode, keyText); - return XK_VoidSymbol; + return NoSymbol; } ucs = fl_utf8decode(keyText, NULL, NULL); @@ -754,7 +758,7 @@ void Viewport::handleKeyEvent(int keyCode, int origKeyCode, const char *keyText, } keySym = translateKeyEvent(keyCode, origKeyCode, keyText); - if (keySym == XK_VoidSymbol) + if (keySym == NoSymbol) return; #ifdef WIN32 diff --git a/vncviewer/keysym2ucs.c b/vncviewer/keysym2ucs.c index 4e3dca25..2fd58706 100644 --- a/vncviewer/keysym2ucs.c +++ b/vncviewer/keysym2ucs.c @@ -34,6 +34,8 @@ #include "keysym2ucs.h" +#define NoSymbol 0 + struct codepair { unsigned short keysym; unsigned short ucs; @@ -886,5 +888,5 @@ unsigned ucs2keysym(unsigned ucs) return ucs | 0x01000000; /* no matching keysym value found */ - return 0xFFFFFF; + return NoSymbol; } -- 2.39.5