fixup! Fix build warnings

[Version] 0.12.4
[Issue Type] Warning

Change-Id: Ie402f9bfa739dfba422ecffc9993a258b5cb39f8

diff --git a/focus_server/mm_sound_mgr_focus.c b/focus_server/mm_sound_mgr_focus.c
index b2c961e..a3c162d 100644 (file)
--- a/focus_server/mm_sound_mgr_focus.c
+++ b/focus_server/mm_sound_mgr_focus.c
@@ -148,7 +148,7 @@ static int _mm_sound_mgr_focus_get_priority_from_stream_type(int *priority, cons
 
        if (priority == NULL || stream_type == NULL) {
                ret = MM_ERROR_INVALID_ARGUMENT;
-               debug_error("invalid argument, priority[0x%x], stream_type[%s], ret[0x%x]", priority, stream_type, ret);
+               debug_error("invalid argument, priority[%p], stream_type[%s], ret[0x%x]", priority, stream_type, ret);
        } else {
                for (i = 0; i < AVAIL_STREAMS_MAX; i++) {
                        if (g_stream_list.stream_types[i] &&
@@ -1524,7 +1524,7 @@ int mm_sound_mgr_focus_emergent_exit(const _mm_sound_mgr_focus_param_t *param)
                                        if (node->taken_by_id[i].pid == param_s.pid && node->taken_by_id[i].handle_id == param_s.handle_id) {
                                                /* do callback for resumption */
                                                if ((ret = _mm_sound_mgr_focus_do_callback(FOCUS_COMMAND_ACQUIRE, node, &param_s)))
-                                                       debug_error("Fail to _focus_do_callback for COMMAND ACQUIRE to node[%x], ret[0x%x]", node, ret);
+                                                       debug_error("Fail to _focus_do_callback for COMMAND ACQUIRE to node[%p], ret[0x%x]", node, ret);
                                                if (!strncmp(my_node->stream_type, node->stream_type, MAX_STREAM_TYPE_LEN))
                                                        need_to_trigger_watch_cb = false;
                                        }

diff --git a/mm_sound.c b/mm_sound.c
index 11097ad..e0a5553 100644 (file)
--- a/mm_sound.c
+++ b/mm_sound.c
@@ -598,7 +598,7 @@ static void signal_callback(GDBusConnection *conn,
                _dbus_signal_callback(signal_name, value, user_data);
        } else  {
                value_type = g_variant_get_type(parameters);
-               debug_warning("signal type is %s", value_type);
+               debug_warning("signal type is %p", value_type);
        }
 }
 

diff --git a/mm_sound_client.c b/mm_sound_client.c
index bd9f948..cda4d91 100644 (file)
--- a/mm_sound_client.c
+++ b/mm_sound_client.c
@@ -760,7 +760,7 @@ static void _mm_sound_device_connected_callback_wrapper_func(int device_id, cons
 
        debug_log("[Device %s] id(%d) type(%s) direction(%d) state(%d) name(%s) is_running(%d) vendor-id(%04x) product-id(%04x)",
                                is_connected ? "Connected" : "Disconnected", device_id, device_type, io_direction, state, name,
-                               is_running, vendor_id, product_id, is_connected);
+                               is_running, vendor_id, product_id);
 
        if (cb_data == NULL) {
                debug_warning("device connected changed callback data null");

diff --git a/packaging/libmm-sound.spec b/packaging/libmm-sound.spec
index 7fd5a1f..b342d8b 100644 (file)
--- a/packaging/libmm-sound.spec
+++ b/packaging/libmm-sound.spec
@@ -1,6 +1,6 @@
 Name:       libmm-sound
 Summary:    MMSound Package contains client lib and sound_server binary
-Version:    0.12.3
+Version:    0.12.4
 Release:    0
 Group:      System/Libraries
 License:    Apache-2.0

diff --git a/server/mm_sound_mgr_codec.c b/server/mm_sound_mgr_codec.c
index 8452e54..afb4760 100644 (file)
--- a/server/mm_sound_mgr_codec.c
+++ b/server/mm_sound_mgr_codec.c
@@ -988,7 +988,7 @@ static int _MMSoundMgrCodecStopCallback(int param)
        __mm_sound_mgr_ipc_notify_play_file_end(param);
        debug_msg("Client callback msg_type (instance) : [%d]", (int)g_slots[param].param);
 
-       debug_msg("Handle allocated handle : [0x%08X]", g_slots[param].plughandle);
+       debug_msg("Handle allocated handle : [%p]", g_slots[param].plughandle);
        err = g_plugins[g_slots[param].pluginid].Destroy(g_slots[param].plughandle);
        if (err < 0)
                debug_critical("[CODEC MGR] Fail to destroy slot number : [%d] err [0x%x]", param, err);

diff --git a/server/plugin/tone/mm_sound_plugin_codec_tone.c b/server/plugin/tone/mm_sound_plugin_codec_tone.c
index e52c171..6b31981 100644 (file)
--- a/server/plugin/tone/mm_sound_plugin_codec_tone.c
+++ b/server/plugin/tone/mm_sound_plugin_codec_tone.c
@@ -788,7 +788,7 @@ int MMSoundPlugCodecToneCreate(mmsound_codec_param_t *param, mmsound_codec_info_
                debug_error("Device Open Error 0x%x", result);
                goto Error;
        }
-       debug_log("Create audio_handle is %lu", toneInfo->handle);
+       debug_log("Create audio_handle is %d", toneInfo->handle);
 
        debug_msg("tone : %d", param->tone);
        debug_msg("repeat : %d", param->repeat_count);
@@ -830,7 +830,7 @@ Error:
        if (toneInfo) {
                if (toneInfo->handle) {
                        if (MM_ERROR_NONE != mm_sound_pa_close(toneInfo->handle))
-                               debug_error("mm_sound_pa_close() fail. handle(%lu)", toneInfo->handle);
+                               debug_error("mm_sound_pa_close() fail. handle(%d)", toneInfo->handle);
                }
                free(toneInfo);
        }
@@ -1138,7 +1138,7 @@ static void _running_tone(void *param)
 
 exit:
        if (MM_ERROR_NONE != mm_sound_pa_close(toneInfo->handle))
-               debug_error("mm_sound_pa_close() fail. handle(%lu)", toneInfo->handle);
+               debug_error("mm_sound_pa_close() fail. handle(%d)", toneInfo->handle);
 
 #if 0
        pthread_mutex_destroy(&g_control.syncker);

diff --git a/server/plugin/wav/mm_sound_plugin_codec_wave.c b/server/plugin/wav/mm_sound_plugin_codec_wave.c
index dbd81ce..0686128 100644 (file)
--- a/server/plugin/wav/mm_sound_plugin_codec_wave.c
+++ b/server/plugin/wav/mm_sound_plugin_codec_wave.c
@@ -164,7 +164,7 @@ static void _pa_stream_drain_complete_callback(pa_stream* s, int success, void *
        else
                pa_operation_unref(o);
 
-       debug_msg("Call stop callback(%p, %p) of mgr_codec", h->stop_cb, h->cb_param);
+       debug_msg("Call stop callback(%p, %d) of mgr_codec", h->stop_cb, h->cb_param);
        if (h->stop_cb)
                h->stop_cb(h->cb_param);
 }