From 2635173097f32662987bfc898dd7642aa8754e22 Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Wed, 16 Aug 2017 08:09:28 +0900 Subject: [PATCH] Print warning when failed to call isf control API Change-Id: Ic3836e6ec926582f955a5c218395fda8408c9d21 Signed-off-by: Jihoon Kim --- .../input_method_setting_list_popup_view.cpp | 3 ++- im_setting_list/input_method_setting_list_ui.cpp | 16 +++++++++++++--- 2 files changed, 15 insertions(+), 4 deletions(-) diff --git a/im_setting_list/input_method_setting_list_popup_view.cpp b/im_setting_list/input_method_setting_list_popup_view.cpp index 37f94d6..551b0a6 100644 --- a/im_setting_list/input_method_setting_list_popup_view.cpp +++ b/im_setting_list/input_method_setting_list_popup_view.cpp @@ -140,7 +140,8 @@ static Eina_Bool _ime_select_idler_cb(void *data) if (!ad) return ECORE_CALLBACK_CANCEL; LOGD("set active IME\n"); - isf_control_set_active_ime(g_active_ime_info_list[selected_index].appid); + if (isf_control_set_active_ime(g_active_ime_info_list[selected_index].appid) != 0) + LOGW("Failed to set active IME : %s\n", g_active_ime_info_list[selected_index].appid); LOGD("update window selector\n"); im_setting_list_update_window_selector(ad); diff --git a/im_setting_list/input_method_setting_list_ui.cpp b/im_setting_list/input_method_setting_list_ui.cpp index 507c638..31776d3 100644 --- a/im_setting_list/input_method_setting_list_ui.cpp +++ b/im_setting_list/input_method_setting_list_ui.cpp @@ -163,7 +163,7 @@ static void im_setting_list_load_ime_info(void) } free(info); } else { - LOGD("isf_control_get_all_ime_info failed\n"); + LOGW("isf_control_get_all_ime_info failed\n"); } im_setting_list_sort_ime_info(ime_info_list_preinstall, ime_info_list_user); } @@ -207,7 +207,9 @@ im_setting_list_check_popup_ok_cb(void *data, Evas_Object *obj, void *event_info } Eina_Bool state = EINA_FALSE; state = g_gen_item_data[index].chk_status; - isf_control_set_enable_ime(g_ime_info_list[index].appid, state); + if (isf_control_set_enable_ime(g_ime_info_list[index].appid, state) != 0) + LOGW("Failed to set enable ime : %s\n", g_ime_info_list[index].appid); + evas_object_del(cb_data->popup); } @@ -227,7 +229,10 @@ im_setting_list_check_popup_cancel_cb(void *data, Evas_Object *obj, void *event_ if (isf_control_set_enable_ime(g_ime_info_list[index].appid, !state) == 0) { g_gen_item_data[index].chk_status = !state; elm_genlist_item_update(g_gen_item_data[index].gen_item); + } else { + LOGW("Failed to set enable ime : %s\n", g_ime_info_list[index].appid); } + evas_object_del(cb_data->popup); } @@ -309,7 +314,8 @@ static void im_setting_list_check_button_change_cb(void *data, Evas_Object *obj, Eina_Bool state = g_gen_item_data[index].chk_status; if (!state) { - isf_control_set_enable_ime(g_ime_info_list[index].appid, state); + if (isf_control_set_enable_ime(g_ime_info_list[index].appid, state) != 0) + LOGW("Failed to set enable ime : %s\n", g_ime_info_list[index].appid); } else { im_setting_list_show_popup(data, obj, im_setting_list_check_popup_ok_cb, im_setting_list_check_popup_cancel_cb); } @@ -340,6 +346,8 @@ im_setting_list_popup_ok_cb(void *data, Evas_Object *obj, void *event_info) elm_object_signal_emit(ck, "elm,activate,check,on", "elm"); elm_check_state_set(ck, !state); } + } else { + LOGW("Failed to set enable ime : %s\n", g_ime_info_list[index].appid); } evas_object_del(cb_data->popup); } @@ -382,6 +390,8 @@ static void im_setting_list_item_sel_cb(void *data, Evas_Object *obj, void *even elm_check_state_set(ck, !state); } g_gen_item_data[index].chk_status = !state; + } else { + LOGW("Failed to set enable ime : %s\n", g_ime_info_list[index].appid); } } else { im_setting_list_show_popup(data, obj, im_setting_list_popup_ok_cb, im_setting_list_popup_cancel_cb); -- 2.34.1