fix usage of log functions.

Assure there is not mismatch between format and passed values.

Change-Id: If1d9015cf06d1a26893817c13b36b7ed4c3db5ff

diff --git a/src/main.c b/src/main.c
index b86c977..ca6bbbf 100644 (file)
--- a/src/main.c
+++ b/src/main.c
@@ -389,7 +389,7 @@ static Eina_Bool _indicator_listen_timer_cb(void* data)
 
 
        if (!elm_win_socket_listen(win->win , INDICATOR_SERVICE_NAME, 0, EINA_FALSE)) {
-               _E("failed to elm_win_socket_listen() %x", win->win);
+               _E("failed to elm_win_socket_listen()");
                return ECORE_CALLBACK_RENEW;
        } else {
                _D("listen success");
@@ -417,7 +417,7 @@ static void _create_window(struct appdata *ad)
        }
 
        if (!elm_win_socket_listen(ad->win.win , INDICATOR_SERVICE_NAME, 0, EINA_FALSE)) {
-               _E("Failed 1st to elm_win_socket_listen() %x", ad->win.win);
+               _E("Failed 1st to elm_win_socket_listen()");
 
                if (s_info.listen_timer != NULL) {
                        ecore_timer_del(s_info.listen_timer);
@@ -455,7 +455,7 @@ static void _create_base_gui(void* data)
        ad->win.w = ad->win.port_w;
        ad->win.evas = evas_object_evas_get(ad->win.win);
 
-       _D("win_size = Original(%d, %d), Scaled(%lf, %lf)", ad->win.port_w, ad->win.h, ELM_SCALE_SIZE(ad->win.port_w), ELM_SCALE_SIZE(ad->win.h));
+       _D("win_size = Original(%d, %d), Scaled(%d, %d)", ad->win.port_w, ad->win.h, ELM_SCALE_SIZE(ad->win.port_w), ELM_SCALE_SIZE(ad->win.h));
 
        _create_layout(ad, util_get_res_file_path(EDJ_FILE), GRP_NAME);
        _create_box(&(ad->win));

diff --git a/src/modules/clock/clock.c b/src/modules/clock/clock.c
index e75dec7..60523b0 100644 (file)
--- a/src/modules/clock/clock.c
+++ b/src/modules/clock/clock.c
@@ -245,7 +245,7 @@ static Eina_Bool indicator_clock_changed_cb(void *data)
                return EINA_FALSE;
        }
 
-       _D("[CLOCK MODULE] Timer Status : %d Time: %s", clock_timer, result);
+       _D("[CLOCK MODULE] Timer Time: %s", result);
        util_part_text_emit(data, "elm.text.clock", result);
        return EINA_FALSE;
 }

diff --git a/src/modules/connection/mobile_hotspot.c b/src/modules/connection/mobile_hotspot.c
index d0847c1..f2735e7 100644 (file)
--- a/src/modules/connection/mobile_hotspot.c
+++ b/src/modules/connection/mobile_hotspot.c
@@ -168,7 +168,7 @@ static bool _teter_client_cb(tethering_client_h client, void *user_data)
        tethering_type_e type;
 
        int ret = tethering_client_get_tethering_type(client, &type);
-       retvm_if(ret != TETHERING_ERROR_NONE, false, "tethering_client_get_tethering_type failed: %d", get_error_message(ret));
+       retvm_if(ret != TETHERING_ERROR_NONE, false, "tethering_client_get_tethering_type failed: %s", get_error_message(ret));
 
        switch (type) {
        case TETHERING_TYPE_BT:
@@ -203,7 +203,7 @@ static int _tethering_data_reload(tethering_data_t *data)
                return 0;
 
        int ret = tethering_foreach_connected_clients(data->handle, TETHERING_TYPE_ALL, _teter_client_cb, data);
-       retvm_if(ret != TETHERING_ERROR_NONE, FAIL, "tethering_foreach_connected_clients failed: %d", get_error_message(ret));
+       retvm_if(ret != TETHERING_ERROR_NONE, FAIL, "tethering_foreach_connected_clients failed: %s", get_error_message(ret));
 
        return 0;
 }
@@ -229,26 +229,26 @@ static void _tethering_data_disabled(tethering_error_e result, tethering_type_e
 static int _tethering_data_init(tethering_data_t *data)
 {
        int ret = tethering_create(&data->handle);
-       retvm_if(ret != TETHERING_ERROR_NONE, FAIL, "tethering_create failed: %d", get_error_message(ret));
+       retvm_if(ret != TETHERING_ERROR_NONE, FAIL, "tethering_create failed: %s", get_error_message(ret));
 
        ret = tethering_set_connection_state_changed_cb(data->handle, TETHERING_TYPE_ALL, _tethering_data_connection_state_changed, data);
        if (ret != TETHERING_ERROR_NONE) {
                tethering_destroy(data->handle);
-               _E("tethering_set_connection_state_changed_cb failed: %d", get_error_message(ret));
+               _E("tethering_set_connection_state_changed_cb failed: %s", get_error_message(ret));
                return FAIL;
        }
 
        ret = tethering_set_enabled_cb(data->handle, TETHERING_TYPE_ALL, _tethering_data_enabled_changed, data);
        if (ret != TETHERING_ERROR_NONE) {
                tethering_destroy(data->handle);
-               _E("tethering_set_enabled_cb failed: %d", get_error_message(ret));
+               _E("tethering_set_enabled_cb failed: %s", get_error_message(ret));
                return FAIL;
        }
 
        ret = tethering_set_disabled_cb(data->handle, TETHERING_TYPE_ALL, _tethering_data_disabled, data);
        if (ret != TETHERING_ERROR_NONE) {
                tethering_destroy(data->handle);
-               _E("tethering_set_disabled_cb failed: %d", get_error_message(ret));
+               _E("tethering_set_disabled_cb failed: %s", get_error_message(ret));
                return FAIL;
        }
 

diff --git a/src/ticker.c b/src/ticker.c
index 3903abd..a8522ea 100644 (file)
--- a/src/ticker.c
+++ b/src/ticker.c
@@ -118,7 +118,7 @@ static void _request_to_delete_noti(notification_h noti)
 
        int applist = NOTIFICATION_DISPLAY_APP_ALL;
        int ret = notification_get_display_applist(noti, &applist);
-       retm_if(ret != NOTIFICATION_ERROR_NONE, false, "notification_get_display_applist failed: %s", get_error_message(ret));
+       retm_if(ret != NOTIFICATION_ERROR_NONE, "notification_get_display_applist failed: %s", get_error_message(ret));
 
        if (applist == NOTIFICATION_DISPLAY_APP_TICKER) {
                ret = notification_delete(noti);
@@ -126,7 +126,7 @@ static void _request_to_delete_noti(notification_h noti)
                        _E("Could not delete notification");
        } else {
                ret = notification_set_display_applist(noti, applist & ~NOTIFICATION_DISPLAY_APP_TICKER);
-               retm_if(ret != NOTIFICATION_ERROR_NONE, "notification_set_display_applist failed: %s", ret, get_error_message(ret));
+               retm_if(ret != NOTIFICATION_ERROR_NONE, "notification_set_display_applist failed: %s", get_error_message(ret));
 
                ret = notification_update(noti);
                retm_if(ret != NOTIFICATION_ERROR_NONE, "notification_update failed: %s", get_error_message(ret));