From bffd10cb7bf652b9411dba7003d14cad64cd19e4 Mon Sep 17 00:00:00 2001 From: Gustavo Sverzut Barbieri Date: Tue, 8 Jan 2013 14:49:50 +0000 Subject: [PATCH] efl/ecore_imf: minor fixes. * removed some useless variable attribution; * removed some useless casts; * () -> (void) for function without arguments. SVN revision: 82419 --- .../ecore/immodules/ibus/ibus_module.c | 2 +- .../ecore/immodules/xim/ecore_imf_xim.c | 24 +++++++------------ 2 files changed, 10 insertions(+), 16 deletions(-) diff --git a/src/modules/ecore/immodules/ibus/ibus_module.c b/src/modules/ecore/immodules/ibus/ibus_module.c index a6e175c3ca..8ac1bf35bc 100644 --- a/src/modules/ecore/immodules/ibus/ibus_module.c +++ b/src/modules/ecore/immodules/ibus/ibus_module.c @@ -81,7 +81,7 @@ im_module_exit(void) } static Ecore_IMF_Context * -im_module_create() +im_module_create(void) { Ecore_IMF_Context *ctx = NULL; IBusIMContext *ctxd = NULL; diff --git a/src/modules/ecore/immodules/xim/ecore_imf_xim.c b/src/modules/ecore/immodules/xim/ecore_imf_xim.c index c35a70bf37..696798d213 100644 --- a/src/modules/ecore/immodules/xim/ecore_imf_xim.c +++ b/src/modules/ecore/immodules/xim/ecore_imf_xim.c @@ -198,7 +198,7 @@ _ecore_imf_context_xim_client_window_set(Ecore_IMF_Context *ctx, void *window) { DBG("ctx=%p, window=%p", ctx, window); - _ecore_imf_xim_ic_client_window_set(ctx, (Ecore_X_Window)((Ecore_Window)window)); + _ecore_imf_xim_ic_client_window_set(ctx, (Ecore_X_Window)((unsigned long)window)); } static void @@ -422,18 +422,19 @@ _ecore_imf_xim_feedback_attr_add(Eina_List **attrs, int start_pos, int end_pos) { - Ecore_IMF_Preedit_Attr *attr = NULL; - + Ecore_IMF_Preedit_Attr *attr; unsigned int start_index = _ecore_imf_xim_utf8_offset_to_index(str, start_pos); unsigned int end_index = _ecore_imf_xim_utf8_offset_to_index(str, end_pos); if (feedback & FEEDBACK_MASK) { - attr = (Ecore_IMF_Preedit_Attr *)calloc(1, sizeof(Ecore_IMF_Preedit_Attr)); + attr = calloc(1, sizeof(Ecore_IMF_Preedit_Attr)); attr->start_index = start_index; attr->end_index = end_index; - *attrs = eina_list_append(*attrs, (void *)attr); + *attrs = eina_list_append(*attrs, attr); } + else + return; if (feedback & XIMUnderline) attr->preedit_type = ECORE_IMF_PREEDIT_TYPE_SUB1; @@ -967,7 +968,7 @@ _ecore_imf_xim_preedit_draw_call(XIC xic EINA_UNUSED, if (tmp) { int tmp_len; - new_text = eina_unicode_utf8_to_unicode((const char *)tmp, &tmp_len); + new_text = eina_unicode_utf8_to_unicode(tmp, &tmp_len); free(tmp); } @@ -1407,15 +1408,8 @@ _ecore_imf_xim_instantiate_cb(Display *display, XPointer call_data EINA_UNUSED) { XIM_Im_Info *info = (XIM_Im_Info *)client_data; - XIM im = NULL; - - im = XOpenIM(display, NULL, NULL, NULL); - - if (!im) - { - fprintf(stderr, "Failed to connect to IM\n"); - return; - } + XIM im = XOpenIM(display, NULL, NULL, NULL); + EINA_SAFETY_ON_NULL_RETURN(im); info->im = im; _ecore_imf_xim_im_setup(info);