From: Eunhae Choi Date: Fri, 7 Dec 2018 06:03:01 +0000 (+0900) Subject: [0.6.157] fix dlog format error X-Git-Tag: accepted/tizen/unified/20181210.060101^0 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=df8f8d20b258abd0ba4d2a42055c4ee521ac3624;p=platform%2Fcore%2Fmultimedia%2Flibmm-player.git [0.6.157] fix dlog format error Change-Id: If06a2043c8faf6842111b29d692d8ca1d445bf83 --- diff --git a/packaging/libmm-player.spec b/packaging/libmm-player.spec index 1151315..dd501c4 100644 --- a/packaging/libmm-player.spec +++ b/packaging/libmm-player.spec @@ -1,6 +1,6 @@ Name: libmm-player Summary: Multimedia Framework Player Library -Version: 0.6.156 +Version: 0.6.157 Release: 0 Group: Multimedia/Libraries License: Apache-2.0 diff --git a/src/mm_player_360.c b/src/mm_player_360.c index afaf661..50f4ff7 100755 --- a/src/mm_player_360.c +++ b/src/mm_player_360.c @@ -181,7 +181,7 @@ int _mmplayer_360_get_direction_of_view(MMHandleType hplayer, float *yaw, float *yaw = M_PI * yaw_degrees / 180.0f; *pitch = M_PI * pitch_degrees / 180.0f; - LOGD("get yaw %f, pitch %f", yaw, pitch); + LOGD("get yaw %f, pitch %f", *yaw, *pitch); return MM_ERROR_NONE; } diff --git a/src/mm_player_es.c b/src/mm_player_es.c index 7df51cd..f579d6a 100644 --- a/src/mm_player_es.c +++ b/src/mm_player_es.c @@ -297,11 +297,11 @@ _mmplayer_set_media_stream_max_size(MMHandleType hplayer, MMPlayerStreamType typ if ((type < MM_PLAYER_STREAM_TYPE_AUDIO) || (type > MM_PLAYER_STREAM_TYPE_TEXT) || (max_size == 0)) { - LOGE("Invalid param type:%d, max_size:%d", type, max_size); + LOGE("Invalid param type:%d, max_size:%"G_GUINT64_FORMAT, type, max_size); return MM_ERROR_INVALID_ARGUMENT; } - LOGD("type:%s, max_size %llu", MMPLAYER_STREAM_TYPE_GET_NAME(type), max_size); + LOGD("type:%s, max_size %"G_GUINT64_FORMAT, MMPLAYER_STREAM_TYPE_GET_NAME(type), max_size); if ((element = __mmplayer_get_source_element(player, type))) { LOGD("update max_size of %s", GST_ELEMENT_NAME(element)); @@ -741,8 +741,8 @@ _mmplayer_set_audio_info(MMHandleType hplayer, media_format_h format) audio.user_info = 0; //test - LOGD("set audio player[%p] info [%p] version=%d rate=%d channel=%d", - player, audio, audio.version, audio.sample_rate, audio.channels); + LOGD("set audio player[%p] version=%d rate=%d channel=%d", + player, audio.version, audio.sample_rate, audio.channels); if (strstr(audio.mime, "audio/mpeg")) { if (audio.version == 1) { // mp3 diff --git a/src/mm_player_gst.c b/src/mm_player_gst.c index 81066be..269afa6 100644 --- a/src/mm_player_gst.c +++ b/src/mm_player_gst.c @@ -744,7 +744,7 @@ do { \ gst_sample_unref(sample);\ return FALSE;\ } \ - SECURE_LOGD("update album cover data : %p, size : %d\n", info.data, info.size);\ + SECURE_LOGD("update album cover data : %p, size : %zu", info.data, info.size);\ MMPLAYER_FREEIF(player->album_art);\ player->album_art = (gchar *)g_malloc(info.size);\ if (player->album_art) {\ @@ -754,7 +754,7 @@ do { \ msg_param.data = (void *)player->album_art;\ msg_param.size = info.size;\ MMPLAYER_POST_MSG(player, MM_MESSAGE_IMAGE_BUFFER, &msg_param);\ - SECURE_LOGD("post message image buffer data : %p, size : %d\n", info.data, info.size);\ + SECURE_LOGD("post message image buffer data : %p, size : %zu", info.data, info.size);\ } \ } \ gst_buffer_unmap(buffer, &info);\ @@ -2196,7 +2196,7 @@ __mmplayer_gst_appsrc_feed_data(GstElement *element, guint size, gpointer user_d g_object_get(G_OBJECT(element), "current-level-bytes", ¤t_level_bytes, NULL); - LOGI("type: %d, level: %llu", type, current_level_bytes); + LOGI("type: %d, level: %"G_GUINT64_FORMAT, type, current_level_bytes); MMPLAYER_MEDIA_STREAM_CALLBACK_LOCK(player); if (player->media_stream_buffer_status_cb[type]) @@ -2255,7 +2255,7 @@ __mmplayer_gst_appsrc_seek_data(GstElement * element, guint64 position, gpointer return TRUE; } - LOGD("type: %d, pos: %llu", type, position); + LOGD("type: %d, pos: %"G_GUINT64_FORMAT, type, position); MMPLAYER_MEDIA_STREAM_CALLBACK_LOCK(player); if (player->media_stream_seek_data_cb[type]) diff --git a/src/mm_player_priv.c b/src/mm_player_priv.c index da0f0a0..557b575 100644 --- a/src/mm_player_priv.c +++ b/src/mm_player_priv.c @@ -1689,7 +1689,7 @@ __mmplayer_get_property_value_for_rotation(mm_player_t* player, int display_angl break; } - LOGD("setting rotation property value : %d", value); + LOGD("setting rotation property value : %d", *value); return TRUE; } @@ -1908,7 +1908,7 @@ __mmplayer_video_param_set_display_overlay(mm_player_t* player) /* default is using wl_surface_id */ unsigned int wl_surface_id = 0; wl_surface_id = *(int*)handle; - LOGD("set video param : wl_surface_id %d %p", wl_surface_id, *(int*)handle); + LOGD("set video param : wl_surface_id %d", wl_surface_id); gst_video_overlay_set_wl_window_wl_surface_id( GST_VIDEO_OVERLAY(player->pipeline->videobin[MMPLAYER_V_SINK].gst), *(int*)handle); @@ -6663,7 +6663,7 @@ int _mmplayer_set_next_uri(MMHandleType hplayer, const char* uri, bool is_first_ player->uri_info.uri_list = g_list_append(player->uri_info.uri_list, g_strdup(original_uri)); player->uri_info.uri_idx = 0; - LOGD("add original path at first : %s(%d)", original_uri); + LOGD("add original path at first : %s", original_uri); } } @@ -7395,7 +7395,7 @@ __mmplayer_check_subtitle(mm_player_t* player) if (!subtitle_uri || !strlen(subtitle_uri)) return FALSE; - SECURE_LOGD("subtitle uri is %s[%d]", subtitle_uri, strlen(subtitle_uri)); + SECURE_LOGD("subtitle uri is %s[%zu]", subtitle_uri, strlen(subtitle_uri)); player->is_external_subtitle_present = TRUE; MMPLAYER_FLEAVE(); @@ -8771,7 +8771,7 @@ __mmplayer_update_bitrate_attrs(mm_player_t *player, MMHandleType attrs) msec_dur = GST_TIME_AS_MSECONDS(player->duration); if (msec_dur > 0) { bitrate = data_size * 8 * 1000 / msec_dur; - SECURE_LOGD("file size : %u, video bitrate = %llu", data_size, bitrate); + SECURE_LOGD("file size : %"G_GUINT64_FORMAT", video bitrate = %"G_GUINT64_FORMAT, data_size, bitrate); mm_attrs_set_int_by_name(attrs, "content_video_bitrate", bitrate); ret = TRUE; diff --git a/src/mm_player_tracks.c b/src/mm_player_tracks.c index d839282..a92e05f 100644 --- a/src/mm_player_tracks.c +++ b/src/mm_player_tracks.c @@ -420,7 +420,7 @@ static int __mmplayer_track_get_language(mm_player_t* player, MMPlayerTrackType LOGD("there is no lang info - und\n"); strncpy(*code, "und", language_code_size); } else { - LOGD("language information[%d] code: %s, len: %d \n", type, tag, strlen(tag)); + LOGD("language information[%d] code: %s, len: %zu", type, tag, strlen(tag)); strncpy(*code, tag, /*strlen(tag)*/language_code_size); g_free(tag); } diff --git a/src/mm_player_utils.c b/src/mm_player_utils.c index dacc032..c59d9a5 100644 --- a/src/mm_player_utils.c +++ b/src/mm_player_utils.c @@ -290,7 +290,7 @@ util_get_cookie_list(const char *cookies) if (!cookies || !strlen(cookies)) return NULL; - SECURE_LOGD("cookies : %d[bytes] - %s \n", strlen(cookies), cookies); + SECURE_LOGD("cookies : %zu[bytes] - %s \n", strlen(cookies), cookies); temp = g_strdup(cookies); @@ -309,7 +309,7 @@ util_get_cookie_list(const char *cookies) if (cookie_list[i]) { if (strlen(cookie_list[i])) { g_strstrip(cookie_list[i]); - SECURE_LOGD("cookie_list[%d] : %d[bytes] - %s \n", i, strlen(cookie_list[i]), cookie_list[i]); + SECURE_LOGD("cookie_list[%d] : %zu[bytes] - %s \n", i, strlen(cookie_list[i]), cookie_list[i]); } else { cookie_list[i][0] = '\0'; }