From 5e1f15892f5225670aca2ab212eb4ea66ae8f766 Mon Sep 17 00:00:00 2001 From: "Ryan C. Gordon" Date: Thu, 21 Aug 2014 23:15:17 -0400 Subject: [PATCH] Adjusted code style slightly for SDL_ibus.c --- src/core/linux/SDL_ibus.c | 193 +++++++++++++++++++------------------- 1 file changed, 98 insertions(+), 95 deletions(-) diff --git a/src/core/linux/SDL_ibus.c b/src/core/linux/SDL_ibus.c index 2a538b1b15d0e..0bb0f562fcbb6 100644 --- a/src/core/linux/SDL_ibus.c +++ b/src/core/linux/SDL_ibus.c @@ -54,14 +54,14 @@ IBus_ModState(void) SDL_Keymod sdl_mods = SDL_GetModState(); /* Not sure about MOD3, MOD4 and HYPER mappings */ - if(sdl_mods & KMOD_LSHIFT) ibus_mods |= IBUS_SHIFT_MASK; - if(sdl_mods & KMOD_CAPS) ibus_mods |= IBUS_LOCK_MASK; - if(sdl_mods & KMOD_LCTRL) ibus_mods |= IBUS_CONTROL_MASK; - if(sdl_mods & KMOD_LALT) ibus_mods |= IBUS_MOD1_MASK; - if(sdl_mods & KMOD_NUM) ibus_mods |= IBUS_MOD2_MASK; - if(sdl_mods & KMOD_MODE) ibus_mods |= IBUS_MOD5_MASK; - if(sdl_mods & KMOD_LGUI) ibus_mods |= IBUS_SUPER_MASK; - if(sdl_mods & KMOD_RGUI) ibus_mods |= IBUS_META_MASK; + if (sdl_mods & KMOD_LSHIFT) ibus_mods |= IBUS_SHIFT_MASK; + if (sdl_mods & KMOD_CAPS) ibus_mods |= IBUS_LOCK_MASK; + if (sdl_mods & KMOD_LCTRL) ibus_mods |= IBUS_CONTROL_MASK; + if (sdl_mods & KMOD_LALT) ibus_mods |= IBUS_MOD1_MASK; + if (sdl_mods & KMOD_NUM) ibus_mods |= IBUS_MOD2_MASK; + if (sdl_mods & KMOD_MODE) ibus_mods |= IBUS_MOD5_MASK; + if (sdl_mods & KMOD_LGUI) ibus_mods |= IBUS_SUPER_MASK; + if (sdl_mods & KMOD_RGUI) ibus_mods |= IBUS_META_MASK; return ibus_mods; } @@ -73,32 +73,32 @@ IBus_GetVariantText(DBusConnection *conn, DBusMessageIter *iter, SDL_DBusContext const char *text = NULL; DBusMessageIter sub1, sub2; - if(dbus->message_iter_get_arg_type(iter) != DBUS_TYPE_VARIANT){ + if (dbus->message_iter_get_arg_type(iter) != DBUS_TYPE_VARIANT) { return NULL; } dbus->message_iter_recurse(iter, &sub1); - if(dbus->message_iter_get_arg_type(&sub1) != DBUS_TYPE_STRUCT){ + if (dbus->message_iter_get_arg_type(&sub1) != DBUS_TYPE_STRUCT) { return NULL; } dbus->message_iter_recurse(&sub1, &sub2); - if(dbus->message_iter_get_arg_type(&sub2) != DBUS_TYPE_STRING){ + if (dbus->message_iter_get_arg_type(&sub2) != DBUS_TYPE_STRING) { return NULL; } const char *struct_id = NULL; dbus->message_iter_get_basic(&sub2, &struct_id); - if(!struct_id || SDL_strncmp(struct_id, "IBusText", sizeof("IBusText")) != 0){ + if (!struct_id || SDL_strncmp(struct_id, "IBusText", sizeof("IBusText")) != 0) { return NULL; } dbus->message_iter_next(&sub2); dbus->message_iter_next(&sub2); - if(dbus->message_iter_get_arg_type(&sub2) != DBUS_TYPE_STRING){ + if (dbus->message_iter_get_arg_type(&sub2) != DBUS_TYPE_STRING) { return NULL; } @@ -113,8 +113,8 @@ IBus_utf8_strlen(const char *str) size_t utf8_len = 0; const char *p; - for(p = str; *p; ++p){ - if(!((*p & 0x80) && !(*p & 0x40))){ + for (p = str; *p; ++p) { + if (!((*p & 0x80) && !(*p & 0x40))) { ++utf8_len; } } @@ -127,16 +127,17 @@ IBus_MessageFilter(DBusConnection *conn, DBusMessage *msg, void *user_data) { SDL_DBusContext *dbus = (SDL_DBusContext *)user_data; - if(dbus->message_is_signal(msg, IBUS_INPUT_INTERFACE, "CommitText")){ + if (dbus->message_is_signal(msg, IBUS_INPUT_INTERFACE, "CommitText")) { DBusMessageIter iter; + dbus->message_iter_init(msg, &iter); const char *text = IBus_GetVariantText(conn, &iter, dbus); - if(text && *text){ + if (text && *text) { char buf[SDL_TEXTEDITINGEVENT_TEXT_SIZE]; size_t text_bytes = SDL_strlen(text), i = 0; - while(i < text_bytes){ + while (i < text_bytes) { size_t sz = SDL_utf8strlcpy(buf, text+i, sizeof(buf)); SDL_SendKeyboardText(buf); @@ -147,17 +148,17 @@ IBus_MessageFilter(DBusConnection *conn, DBusMessage *msg, void *user_data) return DBUS_HANDLER_RESULT_HANDLED; } - if(dbus->message_is_signal(msg, IBUS_INPUT_INTERFACE, "UpdatePreeditText")){ + if (dbus->message_is_signal(msg, IBUS_INPUT_INTERFACE, "UpdatePreeditText")) { DBusMessageIter iter; dbus->message_iter_init(msg, &iter); const char *text = IBus_GetVariantText(conn, &iter, dbus); - if(text && *text){ + if (text && *text) { char buf[SDL_TEXTEDITINGEVENT_TEXT_SIZE]; size_t text_bytes = SDL_strlen(text), i = 0; size_t cursor = 0; - while(i < text_bytes){ + while (i < text_bytes) { size_t sz = SDL_utf8strlcpy(buf, text+i, sizeof(buf)); size_t chars = IBus_utf8_strlen(buf); @@ -173,9 +174,9 @@ IBus_MessageFilter(DBusConnection *conn, DBusMessage *msg, void *user_data) return DBUS_HANDLER_RESULT_HANDLED; } - if(dbus->message_is_signal(msg, IBUS_INPUT_INTERFACE, "HidePreeditText")){ - SDL_SendEditingText("", 0, 0); - return DBUS_HANDLER_RESULT_HANDLED; + if (dbus->message_is_signal(msg, IBUS_INPUT_INTERFACE, "HidePreeditText")) { + SDL_SendEditingText("", 0, 0); + return DBUS_HANDLER_RESULT_HANDLED; } return DBUS_HANDLER_RESULT_NOT_YET_HANDLED; @@ -185,18 +186,18 @@ static char * IBus_ReadAddressFromFile(const char *file_path) { FILE *addr_file = fopen(file_path, "r"); - if(!addr_file){ + if (!addr_file) { return NULL; } char addr_buf[1024]; SDL_bool success = SDL_FALSE; - while(fgets(addr_buf, sizeof(addr_buf), addr_file)){ - if(SDL_strncmp(addr_buf, "IBUS_ADDRESS=", sizeof("IBUS_ADDRESS=")-1) == 0){ + while (fgets(addr_buf, sizeof(addr_buf), addr_file)) { + if (SDL_strncmp(addr_buf, "IBUS_ADDRESS=", sizeof("IBUS_ADDRESS=")-1) == 0) { size_t sz = SDL_strlen(addr_buf); - if(addr_buf[sz-1] == '\n') addr_buf[sz-1] = 0; - if(addr_buf[sz-2] == '\r') addr_buf[sz-2] = 0; + if (addr_buf[sz-1] == '\n') addr_buf[sz-1] = 0; + if (addr_buf[sz-2] == '\r') addr_buf[sz-2] = 0; success = SDL_TRUE; break; } @@ -204,7 +205,7 @@ IBus_ReadAddressFromFile(const char *file_path) fclose(addr_file); - if(success){ + if (success) { return SDL_strdup(addr_buf + (sizeof("IBUS_ADDRESS=") - 1)); } else { return NULL; @@ -214,19 +215,19 @@ IBus_ReadAddressFromFile(const char *file_path) static char * IBus_GetDBusAddressFilename(void) { - if(ibus_addr_file){ + if (ibus_addr_file) { return SDL_strdup(ibus_addr_file); } SDL_DBusContext *dbus = SDL_DBus_GetContext(); - if(!dbus){ + if (!dbus) { return NULL; } /* Use this environment variable if it exists. */ const char *addr = SDL_getenv("IBUS_ADDRESS"); - if(addr && *addr){ + if (addr && *addr) { return SDL_strdup(addr); } @@ -235,7 +236,7 @@ IBus_GetDBusAddressFilename(void) const char *disp_env = SDL_getenv("DISPLAY"); char *display = NULL; - if(!disp_env || !*disp_env){ + if (!disp_env || !*disp_env) { display = SDL_strdup(":0.0"); } else { display = SDL_strdup(disp_env); @@ -245,7 +246,7 @@ IBus_GetDBusAddressFilename(void) char *disp_num = SDL_strrchr(display, ':'), *screen_num = SDL_strrchr(display, '.'); - if(!disp_num){ + if (!disp_num) { SDL_free(display); return NULL; } @@ -253,11 +254,11 @@ IBus_GetDBusAddressFilename(void) *disp_num = 0; disp_num++; - if(screen_num){ + if (screen_num) { *screen_num = 0; } - if(!*host){ + if (!*host) { host = "unix"; } @@ -265,11 +266,11 @@ IBus_GetDBusAddressFilename(void) SDL_memset(config_dir, 0, sizeof(config_dir)); const char *conf_env = SDL_getenv("XDG_CONFIG_HOME"); - if(conf_env && *conf_env){ + if (conf_env && *conf_env) { SDL_strlcpy(config_dir, conf_env, sizeof(config_dir)); } else { const char *home_env = SDL_getenv("HOME"); - if(!home_env || !*home_env){ + if (!home_env || !*home_env) { SDL_free(display); return NULL; } @@ -296,15 +297,15 @@ IBus_SetCapabilities(void *data, const char *name, const char *old_val, { SDL_DBusContext *dbus = SDL_DBus_GetContext(); - if(IBus_CheckConnection(dbus)){ + if (IBus_CheckConnection(dbus)) { DBusMessage *msg = dbus->message_new_method_call(IBUS_SERVICE, input_ctx_path, IBUS_INPUT_INTERFACE, "SetCapabilities"); - if(msg){ + if (msg) { Uint32 caps = IBUS_CAP_FOCUS; - if(!(internal_editing && *internal_editing == '1')){ + if (!(internal_editing && *internal_editing == '1')) { caps |= IBUS_CAP_PREEDIT_TEXT; } @@ -313,8 +314,8 @@ IBus_SetCapabilities(void *data, const char *name, const char *old_val, DBUS_TYPE_INVALID); } - if(msg){ - if(dbus->connection_send(ibus_conn, msg, NULL)){ + if (msg) { + if (dbus->connection_send(ibus_conn, msg, NULL)) { dbus->connection_flush(ibus_conn); } dbus->message_unref(msg); @@ -331,13 +332,13 @@ IBus_SetupConnection(SDL_DBusContext *dbus, const char* addr) ibus_conn = dbus->connection_open_private(addr, NULL); - if(!ibus_conn){ + if (!ibus_conn) { return SDL_FALSE; } dbus->connection_flush(ibus_conn); - if(!dbus->bus_register(ibus_conn, NULL)){ + if (!dbus->bus_register(ibus_conn, NULL)) { ibus_conn = NULL; return SDL_FALSE; } @@ -348,22 +349,22 @@ IBus_SetupConnection(SDL_DBusContext *dbus, const char* addr) IBUS_PATH, IBUS_INTERFACE, "CreateInputContext"); - if(msg){ + if (msg) { const char *client_name = "SDL2_Application"; dbus->message_append_args(msg, DBUS_TYPE_STRING, &client_name, DBUS_TYPE_INVALID); } - if(msg){ + if (msg) { DBusMessage *reply; reply = dbus->connection_send_with_reply_and_block(ibus_conn, msg, 1000, NULL); - if(reply){ - if(dbus->message_get_args(reply, NULL, + if (reply) { + if (dbus->message_get_args(reply, NULL, DBUS_TYPE_OBJECT_PATH, &path, - DBUS_TYPE_INVALID)){ - if(input_ctx_path){ + DBUS_TYPE_INVALID)) { + if (input_ctx_path) { SDL_free(input_ctx_path); } input_ctx_path = SDL_strdup(path); @@ -374,7 +375,7 @@ IBus_SetupConnection(SDL_DBusContext *dbus, const char* addr) dbus->message_unref(msg); } - if(result){ + if (result) { SDL_AddHintCallback(SDL_HINT_IME_INTERNAL_EDITING, &IBus_SetCapabilities, NULL); dbus->bus_add_match(ibus_conn, "type='signal',interface='org.freedesktop.IBus.InputContext'", NULL); @@ -391,27 +392,27 @@ IBus_SetupConnection(SDL_DBusContext *dbus, const char* addr) static SDL_bool IBus_CheckConnection(SDL_DBusContext *dbus) { - if(!dbus) return SDL_FALSE; + if (!dbus) return SDL_FALSE; - if(ibus_conn && dbus->connection_get_is_connected(ibus_conn)){ + if (ibus_conn && dbus->connection_get_is_connected(ibus_conn)) { return SDL_TRUE; } - if(inotify_fd > 0 && inotify_wd > 0){ + if (inotify_fd > 0 && inotify_wd > 0) { char buf[1024]; ssize_t readsize = read(inotify_fd, buf, sizeof(buf)); - if(readsize > 0){ + if (readsize > 0) { char *p; SDL_bool file_updated = SDL_FALSE; - for(p = buf; p < buf + readsize; /**/){ + for (p = buf; p < buf + readsize; /**/) { struct inotify_event *event = (struct inotify_event*) p; - if(event->len > 0){ + if (event->len > 0) { char *addr_file_no_path = SDL_strrchr(ibus_addr_file, '/'); - if(!addr_file_no_path) return SDL_FALSE; + if (!addr_file_no_path) return SDL_FALSE; - if(SDL_strcmp(addr_file_no_path + 1, event->name) == 0){ + if (SDL_strcmp(addr_file_no_path + 1, event->name) == 0) { file_updated = SDL_TRUE; break; } @@ -420,9 +421,9 @@ IBus_CheckConnection(SDL_DBusContext *dbus) p += sizeof(struct inotify_event) + event->len; } - if(file_updated){ + if (file_updated) { char *addr = IBus_ReadAddressFromFile(ibus_addr_file); - if(addr){ + if (addr) { SDL_bool result = IBus_SetupConnection(dbus, addr); SDL_free(addr); return result; @@ -440,9 +441,9 @@ SDL_IBus_Init(void) SDL_bool result = SDL_FALSE; SDL_DBusContext *dbus = SDL_DBus_GetContext(); - if(dbus){ + if (dbus) { char *addr_file = IBus_GetDBusAddressFilename(); - if(!addr_file){ + if (!addr_file) { return SDL_FALSE; } @@ -450,13 +451,13 @@ SDL_IBus_Init(void) char *addr = IBus_ReadAddressFromFile(addr_file); - if(inotify_fd < 0){ + if (inotify_fd < 0) { inotify_fd = inotify_init(); fcntl(inotify_fd, F_SETFL, O_NONBLOCK); } char *addr_file_dir = SDL_strrchr(addr_file, '/'); - if(addr_file_dir){ + if (addr_file_dir) { *addr_file_dir = 0; } @@ -473,24 +474,24 @@ SDL_IBus_Init(void) void SDL_IBus_Quit(void) { - if(input_ctx_path){ + if (input_ctx_path) { SDL_free(input_ctx_path); input_ctx_path = NULL; } - if(ibus_addr_file){ + if (ibus_addr_file) { SDL_free(ibus_addr_file); ibus_addr_file = NULL; } SDL_DBusContext *dbus = SDL_DBus_GetContext(); - if(dbus && ibus_conn){ + if (dbus && ibus_conn) { dbus->connection_close(ibus_conn); dbus->connection_unref(ibus_conn); } - if(inotify_fd > 0 && inotify_wd > 0){ + if (inotify_fd > 0 && inotify_wd > 0) { inotify_rm_watch(inotify_fd, inotify_wd); inotify_wd = -1; } @@ -505,13 +506,13 @@ IBus_SimpleMessage(const char *method) { SDL_DBusContext *dbus = SDL_DBus_GetContext(); - if(IBus_CheckConnection(dbus)){ + if (IBus_CheckConnection(dbus)) { DBusMessage *msg = dbus->message_new_method_call(IBUS_SERVICE, input_ctx_path, IBUS_INPUT_INTERFACE, method); - if(msg){ - if(dbus->connection_send(ibus_conn, msg, NULL)){ + if (msg) { + if (dbus->connection_send(ibus_conn, msg, NULL)) { dbus->connection_flush(ibus_conn); } dbus->message_unref(msg); @@ -538,12 +539,12 @@ SDL_IBus_ProcessKeyEvent(Uint32 keysym, Uint32 keycode) SDL_bool result = SDL_FALSE; SDL_DBusContext *dbus = SDL_DBus_GetContext(); - if(IBus_CheckConnection(dbus)){ + if (IBus_CheckConnection(dbus)) { DBusMessage *msg = dbus->message_new_method_call(IBUS_SERVICE, input_ctx_path, IBUS_INPUT_INTERFACE, "ProcessKeyEvent"); - if(msg){ + if (msg) { Uint32 mods = IBus_ModState(); dbus->message_append_args(msg, DBUS_TYPE_UINT32, &keysym, @@ -552,14 +553,14 @@ SDL_IBus_ProcessKeyEvent(Uint32 keysym, Uint32 keycode) DBUS_TYPE_INVALID); } - if(msg){ + if (msg) { DBusMessage *reply; reply = dbus->connection_send_with_reply_and_block(ibus_conn, msg, 300, NULL); - if(reply){ - if(!dbus->message_get_args(reply, NULL, + if (reply) { + if (!dbus->message_get_args(reply, NULL, DBUS_TYPE_BOOLEAN, &result, - DBUS_TYPE_INVALID)){ + DBUS_TYPE_INVALID)) { result = SDL_FALSE; } dbus->message_unref(reply); @@ -577,35 +578,37 @@ SDL_IBus_ProcessKeyEvent(Uint32 keysym, Uint32 keycode) void SDL_IBus_UpdateTextRect(SDL_Rect *rect) { - if(rect){ + if (rect) { SDL_memcpy(&ibus_cursor_rect, rect, sizeof(ibus_cursor_rect)); } SDL_Window *focused_win = SDL_GetKeyboardFocus(); - if(!focused_win) return; - + if (!focused_win) { + return; + } + SDL_SysWMinfo info; SDL_VERSION(&info.version); - if(!SDL_GetWindowWMInfo(focused_win, &info)) return; + if (!SDL_GetWindowWMInfo(focused_win, &info)) { + return; + } int x = 0, y = 0; SDL_GetWindowPosition(focused_win, &x, &y); #if SDL_VIDEO_DRIVER_X11 - if(info.subsystem == SDL_SYSWM_X11){ - SDL_DisplayData *displaydata = - (SDL_DisplayData *) SDL_GetDisplayForWindow(focused_win)->driverdata; + if (info.subsystem == SDL_SYSWM_X11) { + SDL_DisplayData *displaydata = (SDL_DisplayData *) SDL_GetDisplayForWindow(focused_win)->driverdata; Display *x_disp = info.info.x11.display; Window x_win = info.info.x11.window; int x_screen = displaydata->screen; Window unused; - X11_XTranslateCoordinates(x_disp, x_win, RootWindow(x_disp, x_screen), - 0, 0, &x, &y, &unused); + X11_XTranslateCoordinates(x_disp, x_win, RootWindow(x_disp, x_screen), 0, 0, &x, &y, &unused); } #endif @@ -614,12 +617,12 @@ SDL_IBus_UpdateTextRect(SDL_Rect *rect) SDL_DBusContext *dbus = SDL_DBus_GetContext(); - if(IBus_CheckConnection(dbus)){ + if (IBus_CheckConnection(dbus)) { DBusMessage *msg = dbus->message_new_method_call(IBUS_SERVICE, input_ctx_path, IBUS_INPUT_INTERFACE, "SetCursorLocation"); - if(msg){ + if (msg) { dbus->message_append_args(msg, DBUS_TYPE_INT32, &x, DBUS_TYPE_INT32, &y, @@ -628,8 +631,8 @@ SDL_IBus_UpdateTextRect(SDL_Rect *rect) DBUS_TYPE_INVALID); } - if(msg){ - if(dbus->connection_send(ibus_conn, msg, NULL)){ + if (msg) { + if (dbus->connection_send(ibus_conn, msg, NULL)) { dbus->connection_flush(ibus_conn); } dbus->message_unref(msg); @@ -642,10 +645,10 @@ SDL_IBus_PumpEvents(void) { SDL_DBusContext *dbus = SDL_DBus_GetContext(); - if(IBus_CheckConnection(dbus)){ + if (IBus_CheckConnection(dbus)) { dbus->connection_read_write(ibus_conn, 0); - while(dbus->connection_dispatch(ibus_conn) == DBUS_DISPATCH_DATA_REMAINS){ + while (dbus->connection_dispatch(ibus_conn) == DBUS_DISPATCH_DATA_REMAINS) { /* Do nothing, actual work happens in IBus_MessageFilter */ } }