Skip to content
Permalink
Browse files
Ensure we still clear the X locale modifiers even if not compiled wit…
…h ibus or fcitx support
  • Loading branch information
mdiluz committed Jul 12, 2018
1 parent b5a420c commit c3178e67ca80d905d1812265bd8459c90fdb486b
Showing 1 changed file with 0 additions and 6 deletions.
@@ -292,9 +292,7 @@ X11_InitKeyboard(_THIS)
char *prev_locale = setlocale(LC_ALL, NULL);
char *prev_xmods = X11_XSetLocaleModifiers(NULL);
const char *new_xmods = "";
#if defined(HAVE_IBUS_IBUS_H) || defined(HAVE_FCITX_FRONTEND_H)
const char *env_xmods = SDL_getenv("XMODIFIERS");
#endif
SDL_bool has_dbus_ime_support = SDL_FALSE;

if (prev_locale) {
@@ -309,16 +307,12 @@ X11_InitKeyboard(_THIS)
when it is used via XIM which causes issues. Prevent this by forcing
@im=none if XMODIFIERS contains @im=ibus. IBus can still be used via
the DBus implementation, which also has support for pre-editing. */
#ifdef HAVE_IBUS_IBUS_H
if (env_xmods && SDL_strstr(env_xmods, "@im=ibus") != NULL) {
has_dbus_ime_support = SDL_TRUE;
}
#endif
#ifdef HAVE_FCITX_FRONTEND_H
if (env_xmods && SDL_strstr(env_xmods, "@im=fcitx") != NULL) {
has_dbus_ime_support = SDL_TRUE;
}
#endif
if (has_dbus_ime_support || !xkb_repeat) {
new_xmods = "@im=none";
}

0 comments on commit c3178e6

Please sign in to comment.