Fix defects from static analysis

Change-Id: I9358f33ff843d2860ad021b3bc33bea8224c5aae

diff --git a/src/vc_elm_efl_dump.c b/src/vc_elm_efl_dump.c
index 35e6cb3..2e7dd12 100644 (file)
--- a/src/vc_elm_efl_dump.c
+++ b/src/vc_elm_efl_dump.c
@@ -411,7 +411,7 @@ static Eina_Bool _has_accessible_name(Object_Info *object_info)
                        }
                }
                if (NULL != text) {
-                       VC_ELM_LOG_DUMP("####### [DEBUG] widget_type(s), text(%s)", widget_type, text);
+                       VC_ELM_LOG_DUMP("####### [DEBUG] widget_type(%s), text(%s)", widget_type, text);
                        return EINA_TRUE;
                }
                parent_info = ea_object_dump_parent_widget_data_get(parent_info);

diff --git a/src/vc_elm_widget_wrapper.c b/src/vc_elm_widget_wrapper.c
index cbc307d..b0b3fc8 100644 (file)
--- a/src/vc_elm_widget_wrapper.c
+++ b/src/vc_elm_widget_wrapper.c
@@ -35,7 +35,7 @@ struct __voice_control_wrapper_s {
 
 static struct __voice_control_wrapper_s vcw = {0, 0, 0, 0, 0, 0, 0, 0};
 static Ecore_Timer *g_initialize_timer = NULL;
-#define INITIALIZE_INTERVAL_TIME 300
+#define INITIALIZE_INTERVAL_TIME 300.0
 
 static vc_h g_vc_w;
 
@@ -99,9 +99,9 @@ static Eina_Bool __vc_elm_widget_wrapper_initialize_timer_cb(void *data)
        int ret = -1;
        if (0 != (ret = vc_widget_initialize(&g_vc_w))) {
                VC_ELM_LOG_ERR("Fail to vc_widet_initialize, ret(%d)", ret);
-               if (INITIALIZE_INTERVAL_TIME != (int)ecore_timer_interval_get(g_initialize_timer) * 1000) {
-                       VC_ELM_LOG_INFO("Set initializing timer to %d ms", INITIALIZE_INTERVAL_TIME);
-                       ecore_timer_interval_set(g_initialize_timer, (double)(INITIALIZE_INTERVAL_TIME / 1000));
+               if ((int)INITIALIZE_INTERVAL_TIME != (int)ecore_timer_interval_get(g_initialize_timer) * 1000) {
+                       VC_ELM_LOG_INFO("Set initializing timer to %f ms", INITIALIZE_INTERVAL_TIME);
+                       ecore_timer_interval_set(g_initialize_timer, INITIALIZE_INTERVAL_TIME / 1000);
                }
                return EINA_TRUE;
        }