Print warning when failed to call isf control API

Change-Id: Ic3836e6ec926582f955a5c218395fda8408c9d21
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

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 (file)
--- 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 (file)
--- 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);