Remove '\n' characters when print log

[Version] 0.1.43
[Issue Type] Code cleanup

Change-Id: Ic6668c11d83dc944fd95b0138bf8c91c3e0f635c

diff --git a/include/mediademuxer_util.h b/include/mediademuxer_util.h
index 173e0d57a091c9e62d29a67c14b4c009cab83c1a..e3ab9e8b6c486d510c0f387939eab3f039a80876 100644 (file)
--- a/include/mediademuxer_util.h
+++ b/include/mediademuxer_util.h
@@ -46,8 +46,8 @@ extern "C" {
 #define MD_V(fmt, arg...)
 #define MD_F(fmt, arg...)
 #else
-#define MD_FENTER();     LOGI("function:[%s] ENTER\n", __func__);
-#define MD_FLEAVE();     LOGI("function [%s] LEAVE\n", __func__);
+#define MD_FENTER();     LOGI("function:[%s] ENTER", __func__);
+#define MD_FLEAVE();     LOGI("function [%s] LEAVE", __func__);
 #define MD_C             LOGE  /* MMF_DEBUG_LEVEL_0 */
 #define MD_E             LOGE  /* MMF_DEBUG_LEVEL_1 */
 #define MD_W             LOGW  /* MMF_DEBUG_LEVEL_2 */

diff --git a/include/port_gst/mediademuxer_port_gst.h b/include/port_gst/mediademuxer_port_gst.h
index 0442ba79ff0007cd5573362a58e707921fcdcb3e..fabc45febc0ef933c8dc03ce8aa9bf2de8d67e86 100644 (file)
--- a/include/port_gst/mediademuxer_port_gst.h
+++ b/include/port_gst/mediademuxer_port_gst.h
@@ -35,18 +35,18 @@ extern "C" {
 
 #define MEDIADEMUXER_SET_STATE(x_element, x_state, error) \
        do { \
-               MD_I("setting state [%s:%d] to [%s]\n", #x_state, x_state, GST_ELEMENT_NAME(x_element)); \
+               MD_I("setting state [%s:%d] to [%s]", #x_state, x_state, GST_ELEMENT_NAME(x_element)); \
                if (GST_STATE_CHANGE_FAILURE == gst_element_set_state(x_element, x_state)) { \
-                       MD_E("failed to set state %s to %s\n", #x_state, GST_ELEMENT_NAME(x_element)); \
+                       MD_E("failed to set state %s to %s", #x_state, GST_ELEMENT_NAME(x_element)); \
                        goto error; \
                } \
        } while (0)
 
 #define MEDIADEMUXER_SYNC_STATE_WITH_PARENT(x_element, error) \
        do { \
-               MD_I("Sync state with parent [%s]\n", GST_ELEMENT_NAME(x_element)); \
+               MD_I("Sync state with parent [%s]", GST_ELEMENT_NAME(x_element)); \
                if (GST_STATE_CHANGE_FAILURE == gst_element_sync_state_with_parent(x_element)) { \
-                       MD_E("failed to sync %s state with parent\n", GST_ELEMENT_NAME(x_element)); \
+                       MD_E("failed to sync %s state with parent", GST_ELEMENT_NAME(x_element)); \
                        goto error; \
                } \
        } while (0)
@@ -54,7 +54,7 @@ extern "C" {
 #define MEDIADEMUXER_LINK_PAD(srcpad, sinkpad, error) \
        do { \
                if (GST_PAD_LINK_OK != gst_pad_link(srcpad, sinkpad)) { \
-                       MD_E("failed to linkpad\n"); \
+                       MD_E("failed to linkpad"); \
                        goto error; \
                } \
        } while (0)

diff --git a/packaging/capi-mediademuxer.spec b/packaging/capi-mediademuxer.spec
index 43d89d438e7a3215bda31c70101eb8da9ba56139..87ead857571379d44f1ae36dc8749bd51208d7e5 100644 (file)
--- a/packaging/capi-mediademuxer.spec
+++ b/packaging/capi-mediademuxer.spec
@@ -1,6 +1,6 @@
 Name:       capi-mediademuxer
 Summary:    A Media Demuxer library in Tizen Native API
-Version:    0.1.42
+Version:    0.1.43
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0

diff --git a/src/mediademuxer.c b/src/mediademuxer.c
index e2e7ae8fa7bbd6db63ed0413e9ea79c00be81ab3..08452fd1f35df942e29b9dd91bf27e5194f035d7 100644 (file)
--- a/src/mediademuxer.c
+++ b/src/mediademuxer.c
@@ -35,7 +35,7 @@ static gboolean _mediademuxer_eos_cb(int track_num, void *user_data);
 
 int mediademuxer_create(mediademuxer_h *demuxer)
 {
-       MD_I("mediademuxer_create\n");
+       MD_I("mediademuxer_create");
        mediademuxer_error_e ret;
        DEMUXER_INSTANCE_CHECK(demuxer);
 
@@ -49,7 +49,7 @@ int mediademuxer_create(mediademuxer_h *demuxer)
                }
                handle->demux_state = MEDIADEMUXER_STATE_NONE;
        } else {
-               MD_E("Already created the instance\n");
+               MD_E("Already created the instance");
                return MEDIADEMUXER_ERROR_INVALID_OPERATION;
        }
 
@@ -77,7 +77,7 @@ int mediademuxer_create(mediademuxer_h *demuxer)
 
 int mediademuxer_set_data_source(mediademuxer_h demuxer, const char *path)
 {
-       MD_I("mediademuxer_set_data_source\n");
+       MD_I("mediademuxer_set_data_source");
        mediademuxer_error_e ret = MEDIADEMUXER_ERROR_NONE;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
@@ -107,7 +107,7 @@ int mediademuxer_set_data_source(mediademuxer_h demuxer, const char *path)
 
 int mediademuxer_prepare(mediademuxer_h demuxer)
 {
-       MD_I("mediademuxer_prepare\n");
+       MD_I("mediademuxer_prepare");
        mediademuxer_error_e ret = MEDIADEMUXER_ERROR_NONE;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
@@ -125,7 +125,7 @@ int mediademuxer_prepare(mediademuxer_h demuxer)
                if (handle->demux_state != MEDIADEMUXER_STATE_IDLE)
                        return MEDIADEMUXER_ERROR_INVALID_STATE;
                if (handle->eos_cb == NULL)
-                       MD_E("EOS callback is not set\n");
+                       MD_E("EOS callback is not set");
                MD_E("[CoreAPI][%s] DEMUXER_ERROR_INVALID_OPERATION(0x%08x)",
                        __FUNCTION__, MEDIADEMUXER_ERROR_INVALID_OPERATION);
                return MEDIADEMUXER_ERROR_INVALID_OPERATION;
@@ -135,7 +135,7 @@ int mediademuxer_prepare(mediademuxer_h demuxer)
 
 int mediademuxer_get_track_count(mediademuxer_h demuxer, int *count)
 {
-       MD_I("mediademuxer_get_track_count\n");
+       MD_I("mediademuxer_get_track_count");
        mediademuxer_error_e ret;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
@@ -159,7 +159,7 @@ int mediademuxer_get_track_count(mediademuxer_h demuxer, int *count)
 
 int mediademuxer_select_track(mediademuxer_h demuxer, int track_index)
 {
-       MD_I("mediademuxer_select_track\n");
+       MD_I("mediademuxer_select_track");
        mediademuxer_error_e ret;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
@@ -183,7 +183,7 @@ int mediademuxer_select_track(mediademuxer_h demuxer, int track_index)
 
 int mediademuxer_start(mediademuxer_h demuxer)
 {
-       MD_I("mediademuxer_start\n");
+       MD_I("mediademuxer_start");
        mediademuxer_error_e ret;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
@@ -210,13 +210,13 @@ int mediademuxer_start(mediademuxer_h demuxer)
 int mediademuxer_get_track_info(mediademuxer_h demuxer, int track_index,
                                                        media_format_h *format)
 {
-       MD_I("mediademuxer_get_track_info\n");
+       MD_I("mediademuxer_get_track_info");
        mediademuxer_error_e ret;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
        handle = (mediademuxer_s *)demuxer;
        if (track_index < 0 || format == NULL) {
-               MD_E("Invalid input parameters\n");
+               MD_E("Invalid input parameters");
                return MEDIADEMUXER_ERROR_INVALID_PARAMETER;
        }
        if (handle && (handle->demux_state == MEDIADEMUXER_STATE_READY
@@ -240,7 +240,7 @@ int mediademuxer_get_track_info(mediademuxer_h demuxer, int track_index,
 int mediademuxer_read_sample(mediademuxer_h demuxer, int track_index,
                                                        media_packet_h *outbuf)
 {
-       MD_I("mediademuxer_read_sample\n");
+       MD_I("mediademuxer_read_sample");
        mediademuxer_error_e ret;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
@@ -266,7 +266,7 @@ int mediademuxer_read_sample(mediademuxer_h demuxer, int track_index,
 
 int mediademuxer_seek(mediademuxer_h demuxer, int64_t pos)
 {
-       MD_I("mediademuxer_seek\n");
+       MD_I("mediademuxer_seek");
        mediademuxer_error_e ret;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
@@ -290,7 +290,7 @@ int mediademuxer_seek(mediademuxer_h demuxer, int64_t pos)
 
 int mediademuxer_unselect_track(mediademuxer_h demuxer, int track_index)
 {
-       MD_I("mediademuxer_unselect_track\n");
+       MD_I("mediademuxer_unselect_track");
        mediademuxer_error_e ret;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
@@ -315,7 +315,7 @@ int mediademuxer_unselect_track(mediademuxer_h demuxer, int track_index)
 
 int mediademuxer_stop(mediademuxer_h demuxer)
 {
-       MD_I("mediademuxer_stop\n");
+       MD_I("mediademuxer_stop");
        mediademuxer_error_e ret;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
@@ -340,7 +340,7 @@ int mediademuxer_stop(mediademuxer_h demuxer)
 
 int mediademuxer_unprepare(mediademuxer_h demuxer)
 {
-       MD_I("mediademuxer_unprepare\n");
+       MD_I("mediademuxer_unprepare");
        mediademuxer_error_e ret;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
@@ -365,7 +365,7 @@ int mediademuxer_unprepare(mediademuxer_h demuxer)
 
 int mediademuxer_destroy(mediademuxer_h demuxer)
 {
-       MD_I("mediademuxer_destroy\n");
+       MD_I("mediademuxer_destroy");
        mediademuxer_error_e ret;
        DEMUXER_INSTANCE_CHECK(demuxer);
        mediademuxer_s *handle;
@@ -428,14 +428,14 @@ int mediademuxer_unset_error_cb(mediademuxer_h demuxer)
                return MEDIADEMUXER_ERROR_INVALID_STATE;
        handle->error_cb = NULL;
        handle->error_cb_userdata = NULL;
-       MD_I("mediademuxer_unset_error_cb\n");
+       MD_I("mediademuxer_unset_error_cb");
        return MEDIADEMUXER_ERROR_NONE;
 }
 //LCOV_EXCL_START
 static gboolean _mediademuxer_error_cb(mediademuxer_error_e error, void *user_data)
 {
        if (user_data == NULL) {
-               MD_I("_mediademuxer_error_cb: ERROR %d to report. But call back is not set\n", error);
+               MD_I("_mediademuxer_error_cb: ERROR %d to report. But call back is not set", error);
                return 0;
        }
        mediademuxer_s * handle = (mediademuxer_s *)user_data;
@@ -444,7 +444,7 @@ static gboolean _mediademuxer_error_cb(mediademuxer_error_e error, void *user_da
        if (handle->error_cb)
                ((mediademuxer_error_cb)handle->error_cb)(error, handle->error_cb_userdata);
        else
-               MD_I("_mediademuxer_error_cb: ERROR %d to report. But call back is not set\n", error);
+               MD_I("_mediademuxer_error_cb: ERROR %d to report. But call back is not set", error);
        return 0;
 }
 //LCOV_EXCL_STOP
@@ -471,14 +471,14 @@ int mediademuxer_unset_eos_cb(mediademuxer_h demuxer)
                return MEDIADEMUXER_ERROR_INVALID_STATE;
        handle->eos_cb = NULL;
        handle->eos_cb_userdata = NULL;
-       MD_I("mediademuxer_unset_eos_cb\n");
+       MD_I("mediademuxer_unset_eos_cb");
        return MEDIADEMUXER_ERROR_NONE;
 }
 
 static gboolean _mediademuxer_eos_cb(int track_num, void *user_data)
 {
        if (user_data == NULL) {
-               MD_I("_mediademuxer_eos_cb: EOS to report. But call back is not set\n");
+               MD_I("_mediademuxer_eos_cb: EOS to report. But call back is not set");
                return 0;
        }
        mediademuxer_s *handle = (mediademuxer_s *)user_data;
@@ -487,6 +487,6 @@ static gboolean _mediademuxer_eos_cb(int track_num, void *user_data)
        if (handle->eos_cb)
                ((mediademuxer_eos_cb)handle->eos_cb)(track_num, handle->eos_cb_userdata);
        else
-               MD_I("_mediademuxer_eos_cb: EOS %d to report. But call back is not set\n", track_num);
+               MD_I("_mediademuxer_eos_cb: EOS %d to report. But call back is not set", track_num);
        return 0;
 }

diff --git a/src/mediademuxer_ini.c b/src/mediademuxer_ini.c
index 43916835d576494423f4584d61c9d03f0301cf11..74bdeafbc35f71a0c7708ac287fbba6a88083983 100644 (file)
--- a/src/mediademuxer_ini.c
+++ b/src/mediademuxer_ini.c
@@ -77,7 +77,7 @@ static void _md_ini_check_ini_status(void);
                while (token) { \
                        index = atoi(token); \
                        if (index < 0 || index > x_list_max -1) { \
-                               MD_W("%d is not valid index\n", index); \
+                               MD_W("%d is not valid index", index); \
                        } else { \
                                x_list[index] = TRUE; \
                        } \
@@ -98,7 +98,7 @@ static void _md_ini_check_ini_status(void);
                token = strtok_r(temp_arr, delimiters, &usr_ptr); \
                while (token) { \
                        if (index > x_list_max -1) { \
-                               MD_E("%d is not valid index\n", index); \
+                               MD_E("%d is not valid index", index); \
                                break; \
                        } else { \
                                value = atoi(token); \
@@ -120,15 +120,15 @@ int md_ini_load(md_ini_t *ini)
        /* if no file exists. create one with set of default values */
        if (!dict) {
 #ifdef MEDIADEMUXER_DEFAULT_INI
-               MD_L("No inifile found. demuxer will create default inifile.\n");
+               MD_L("No inifile found. demuxer will create default inifile.");
                if (FALSE == _generate_default_ini()) {
-                       MD_W("Creating default inifile failed. MediaDemuxer will use default values.\n");
+                       MD_W("Creating default inifile failed. MediaDemuxer will use default values.");
                } else {
                        /* load default ini */
                        dict = iniparser_load(MEDIADEMUXER_INI_DEFAULT_PATH);
                }
 #else
-               MD_L("No ini file found. \n");
+               MD_L("No ini file found. ");
                return MD_ERROR_FILE_NOT_FOUND;
 #endif
        }
@@ -141,7 +141,7 @@ int md_ini_load(md_ini_t *ini)
                MEDIADEMUXER_INI_GET_STRING(dict, ini->port_name, "port_in_use:mediademuxer_port", DEFAULT_PORT);
        } else {                /* if dict is not available just fill the structure with default value */
 
-               MD_W("failed to load ini. using hardcoded default\n");
+               MD_W("failed to load ini. using hardcoded default");
                strncpy(ini->port_name, DEFAULT_PORT, MEDIADEMUXER_INI_MAX_STRLEN - 1);
        }
 
@@ -152,22 +152,22 @@ int md_ini_load(md_ini_t *ini)
        else if (0 == strcmp(ini->port_name, "CUSTOM_PORT"))
                ini->port_type = CUSTOM_PORT;
        else {
-               MD_E("Invalid port is set to [%s] [%d]\n", ini->port_name,
+               MD_E("Invalid port is set to [%s] [%d]", ini->port_name,
                     ini->port_type);
                iniparser_freedict(dict);
                return MD_CORRUPTED_INI;
        }
-       MD_L("The port is set to [%s] [%d]\n", ini->port_name, ini->port_type);
+       MD_L("The port is set to [%s] [%d]", ini->port_name, ini->port_type);
 
        /* free dict as we got our own structure */
        iniparser_freedict(dict);
 
        /* dump structure */
-       MD_L("demuxer settings -----------------------------------\n");
+       MD_L("demuxer settings -----------------------------------");
 
        /* general */
-       MD_L("port_name: %s\n", ini->port_name);
-       MD_L("port_type : %d\n", ini->port_type);
+       MD_L("port_name: %s", ini->port_name);
+       MD_L("port_type : %d", ini->port_type);
 
        return MD_ERROR_NONE;
 }
@@ -177,10 +177,10 @@ static void _md_ini_check_ini_status(void)
        struct stat ini_buff;
 
        if (g_stat(MEDIADEMUXER_INI_DEFAULT_PATH, &ini_buff) < 0) {
-               MD_W("failed to get demuxer ini status\n");
+               MD_W("failed to get demuxer ini status");
        } else {
                if (ini_buff.st_size < 5) {
-                       MD_W("demuxer.ini file size=%d, Corrupted! So, Removed\n", (int)ini_buff.st_size);
+                       MD_W("demuxer.ini file size=%d, Corrupted! So, Removed", (int)ini_buff.st_size);
 
                        if (g_remove(MEDIADEMUXER_INI_DEFAULT_PATH) == -1)
                                MD_E("failed to delete corrupted ini");

diff --git a/src/mediademuxer_port.c b/src/mediademuxer_port.c
index f753b40ad0d780c0025bb98a2dd215dbce259a5f..b07883f6f0b2b5cee8da75a1e951652caa7bb664 100644 (file)
--- a/src/mediademuxer_port.c
+++ b/src/mediademuxer_port.c
@@ -62,13 +62,13 @@ int md_create(MMHandleType *demuxer)
        md_handle_t *new_demuxer = NULL;
        MEDIADEMUXER_FENTER();
        new_demuxer = (md_handle_t *)g_malloc0(sizeof(md_handle_t));
-       MD_I("md_create allocating new_demuxer %p:\n", new_demuxer);
+       MD_I("md_create allocating new_demuxer %p:", new_demuxer);
 
        /* alloc ops structure */
        pOps = (media_port_demuxer_ops *)g_malloc0(sizeof(media_port_demuxer_ops));
 
        new_demuxer->demuxer_ops = pOps;
-       MD_I("md_create allocating new_demuxer->demuxer_ops %p:\n",
+       MD_I("md_create allocating new_demuxer->demuxer_ops %p:",
             new_demuxer->demuxer_ops);
        pOps->n_size = sizeof(media_port_demuxer_ops);
 
@@ -109,25 +109,25 @@ int __md_util_exist_file_path(const char *file_path)
                int ret_err = 0;
                ret_err = strerror_r(errno, buf, MAX_ERR_LEN);
                if (ret_err == 0)
-                       MD_E("failed to open file by %s (%d)\n", buf, errno);
+                       MD_E("failed to open file by %s (%d)", buf, errno);
                else
-                       MD_E("File not found, strerror_r() failed with errno (%d)\n", errno);
+                       MD_E("File not found, strerror_r() failed with errno (%d)", errno);
                MEDIADEMUXER_FLEAVE();
                return MD_ERROR_FILE_NOT_FOUND;
        }
 
        if (fstat(fd, &stat_results) < 0) {
-               MD_E("failed to get file status\n");
+               MD_E("failed to get file status");
                close(fd);
                MEDIADEMUXER_FLEAVE();
                return MD_ERROR_FILE_NOT_FOUND;
        } else if (stat_results.st_size == 0) {
-               MD_E("file size is zero\n");
+               MD_E("file size is zero");
                close(fd);
                MEDIADEMUXER_FLEAVE();
                return MD_ERROR_FILE_NOT_FOUND;
        } else {
-               MD_E("file size : %lld bytes\n", (long long)stat_results.st_size);
+               MD_E("file size : %lld bytes", (long long)stat_results.st_size);
        }
        close(fd);
        MEDIADEMUXER_FLEAVE();
@@ -147,7 +147,7 @@ bool __md_util_is_sdp_file(const char *path)
        g_strstrip(uri);
        /* strlen(".sdp") == 4 */
        if (strlen(uri) <= 4) {
-               MD_W("path is too short.\n");
+               MD_W("path is too short.");
                return ret;
        }
        /* first, check extension name */
@@ -173,7 +173,7 @@ mediademuxer_src_type __md_util_media_type(char **uri)
                if (file_stat == MD_ERROR_NONE) {
                        if (__md_util_is_sdp_file(path)) {
 #ifdef TIZEN_FEATURE_STREAMING
-                               MD_L("uri is actually a file but it's sdp file. giving it to rtspsrc\n");
+                               MD_L("uri is actually a file but it's sdp file. giving it to rtspsrc");
                                return MEDIADEMUXER_SRC_RTSP;
 #else
                                MD_L("uri is actually a file but it's sdp file. didn't support streaming uri");
@@ -184,7 +184,7 @@ mediademuxer_src_type __md_util_media_type(char **uri)
                        }
                        return MD_ERROR_NONE;
                } else {
-                       MD_E("could not access %s.\n", path);
+                       MD_E("could not access %s.", path);
                        goto ERROR;
                }
 #ifdef TIZEN_FEATURE_STREAMING
@@ -213,22 +213,22 @@ mediademuxer_src_type __md_util_media_type(char **uri)
                        old_uristr = (char *)g_strdup(*uri);
                        MEDIADEMUXER_CHECK_NULL(old_uristr);
                        /* need to added 7 char for file:// + 1 for '\0'+ uri len */
-                       MD_L("allocating temp old_uristr[%p] \n", old_uristr);
+                       MD_L("allocating temp old_uristr[%p] ", old_uristr);
                        new_uristr = (char *)realloc(*uri, (7 + len_uri + 1) * sizeof(char));
                        if (!new_uristr) {
                                g_free(old_uristr);
                                old_uristr = NULL;
                                return MD_ERROR_INVALID_ARGUMENT;
                        }
-                       MD_L("reallocating uri[%p] to new_uristr[%p] \n", *uri, new_uristr);
+                       MD_L("reallocating uri[%p] to new_uristr[%p] ", *uri, new_uristr);
                        *uri = new_uristr;
                        g_snprintf(*uri, 7 + len_uri + 1, "file://%s", old_uristr);
-                       MD_L("deallocating old_uristr[%p] \n", old_uristr);
+                       MD_L("deallocating old_uristr[%p] ", old_uristr);
                        g_free(old_uristr);
                        old_uristr = NULL;
                        if (__md_util_is_sdp_file((char *)(*uri))) {
 #ifdef TIZEN_FEATURE_STREAMING
-                               MD_L("uri is actually a file but it's sdp file. giving it to rtspsrc\n");
+                               MD_L("uri is actually a file but it's sdp file. giving it to rtspsrc");
                                return MEDIADEMUXER_SRC_RTSP;
 #else
                                MD_L("uri is actually a file but it's sdp file. didn't support streaming uri");
@@ -258,18 +258,18 @@ int _md_util_parse(MMHandleType demuxer, const char *type)
        new_demuxer = (md_handle_t *)demuxer;
        media_type_string = (char *)g_strdup(type);
        MEDIADEMUXER_CHECK_NULL(media_type_string);
-       MD_L("media_type_string allocating %p\n", media_type_string);
+       MD_L("media_type_string allocating %p", media_type_string);
        /*Set media_type depending upon the header of string else consider using file protocol */
        MEDIADEMUXER_FREEIF(new_demuxer->uri_src);
        new_demuxer->uri_src_media_type = __md_util_media_type(&media_type_string);
        if (new_demuxer->uri_src_media_type != MEDIADEMUXER_SRC_INVALID) {
                g_free(new_demuxer->uri_src);
                new_demuxer->uri_src = media_type_string;
-               MD_L("uri:%s\n uri_type:%d\n", new_demuxer->uri_src,
+               MD_L("uri:%s uri_type:%d", new_demuxer->uri_src,
                     new_demuxer->uri_src_media_type);
        } else {
-               MD_E("Error while setting source\n");
-               MD_E("deallocating media_type_string %p\n", media_type_string);
+               MD_E("Error while setting source");
+               MD_E("deallocating media_type_string %p", media_type_string);
                result = MD_ERROR_INVALID_ARGUMENT;
                g_free(media_type_string);
        }

diff --git a/src/port_custom/mediademuxer_port_custom.c b/src/port_custom/mediademuxer_port_custom.c
index ebe5f5bd79a5b9c01774df225eb2dc79a970d6d8..1b1116f4f7918318a8005c1355db1caded676809 100644 (file)
--- a/src/port_custom/mediademuxer_port_custom.c
+++ b/src/port_custom/mediademuxer_port_custom.c
@@ -62,7 +62,7 @@ static int custom_demuxer_init(MMHandleType *pHandle)
 {
        int ret = MD_ERROR_NONE;
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return ret;
 }
@@ -70,7 +70,7 @@ static int custom_demuxer_init(MMHandleType *pHandle)
 static int custom_demuxer_prepare(MMHandleType pHandle, char *uri)
 {
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return 0;
 }
@@ -78,7 +78,7 @@ static int custom_demuxer_prepare(MMHandleType pHandle, char *uri)
 static int custom_demuxer_get_data_count(MMHandleType pHandle, int *count)
 {
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return 0;
 }
@@ -86,7 +86,7 @@ static int custom_demuxer_get_data_count(MMHandleType pHandle, int *count)
 static int custom_demuxer_get_track_info(MMHandleType pHandle, media_format_h *format, int track)
 {
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return 0;
 }
@@ -94,7 +94,7 @@ static int custom_demuxer_get_track_info(MMHandleType pHandle, media_format_h *f
 static int custom_demuxer_set_track(MMHandleType pHandle, int track)
 {
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return 0;
 }
@@ -102,7 +102,7 @@ static int custom_demuxer_set_track(MMHandleType pHandle, int track)
 static int custom_demuxer_get_data(MMHandleType pHandle, char *buffer)
 {
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return 0;
 }

diff --git a/src/port_ffmpeg/mediademuxer_port_ffmpeg.c b/src/port_ffmpeg/mediademuxer_port_ffmpeg.c
index 7b2e0bc0a4de53e53293049fe5ced4ab5177e686..2ceea5381f27044008679f8c31ee28ae3830dbdb 100644 (file)
--- a/src/port_ffmpeg/mediademuxer_port_ffmpeg.c
+++ b/src/port_ffmpeg/mediademuxer_port_ffmpeg.c
@@ -62,7 +62,7 @@ static int ffmpeg_demuxer_init(MMHandleType *pHandle)
 {
        int ret = MD_ERROR_NONE;
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return ret;
 }
@@ -71,7 +71,7 @@ static int ffmpeg_demuxer_prepare(MMHandleType pHandle, char *uri)
 {
        int ret = MD_ERROR_NONE;
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return ret;
 }
@@ -80,7 +80,7 @@ static int ffmpeg_demuxer_get_data_count(MMHandleType pHandle, int *count)
 {
        int ret = MD_ERROR_NONE;
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return ret;
 }
@@ -89,7 +89,7 @@ static int ffmpeg_demuxer_get_track_info(MMHandleType pHandle, media_format_h *f
 {
        int ret = MD_ERROR_NONE;
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return ret;
 }
@@ -98,7 +98,7 @@ static int ffmpeg_demuxer_set_track(MMHandleType pHandle, int track)
 {
        int ret = MD_ERROR_NONE;
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return ret;
 }
@@ -107,7 +107,7 @@ static int ffmpeg_demuxer_get_data(MMHandleType pHandle, char *buffer)
 {
        int ret = MD_ERROR_NONE;
        MEDIADEMUXER_FENTER();
-       MD_E("%s:exit: Not implemented\n", __func__);
+       MD_E("%s:exit: Not implemented", __func__);
        MEDIADEMUXER_FLEAVE();
        return ret;
 }

diff --git a/src/port_gst/mediademuxer_port_gst.c b/src/port_gst/mediademuxer_port_gst.c
index 156eef9ab3247161dadc7f74e3d6579db426f41e..653c6754fa70f9503a7e9005e58853696a1ca159 100644 (file)
--- a/src/port_gst/mediademuxer_port_gst.c
+++ b/src/port_gst/mediademuxer_port_gst.c
@@ -174,7 +174,7 @@ static gint __gst_transform_gsterror(mdgst_handle_t *gst_handle, GstMessage *mes
        if (!klass)
                goto INTERNAL_ERROR;
 
-       MD_I("error code=%d, msg=%s, src element=%s, class=%s\n",
+       MD_I("error code=%d, msg=%s, src element=%s, class=%s",
                        error->code, error->message, src_element_name, klass);
 
        switch (error->code) {
@@ -255,11 +255,11 @@ static int __gst_demuxer_init(MMHandleType *pHandle)
        mdgst_handle_t *new_mediademuxer = NULL;
        new_mediademuxer = (mdgst_handle_t *)g_try_malloc0(sizeof(mdgst_handle_t));
        if (!new_mediademuxer) {
-               MD_E("Cannot allocate memory for demuxer\n");
+               MD_E("Cannot allocate memory for demuxer");
                ret = MD_ERROR;
                goto ERROR;
        }
-       MD_I("__gst_demuxer_init allocating new_mediademuxer:%p\n", new_mediademuxer);
+       MD_I("__gst_demuxer_init allocating new_mediademuxer:%p", new_mediademuxer);
        new_mediademuxer->is_prepared = false;
        (new_mediademuxer->info).num_audio_track = 0;
        (new_mediademuxer->info).num_video_track = 0;
@@ -309,7 +309,7 @@ static gboolean __gst_bus_call(GstBus *bus, GstMessage *msg, gpointer data)
                                        (ret, gst_handle->user_data[_GST_EVENT_TYPE_ERROR]);
                                        MD_W("Error : (ox%08x)", ret);
                        }
-                       MD_E("Error: %s\n", error->message);
+                       MD_E("Error: %s", error->message);
                        g_error_free(error);
                }
                break;
@@ -329,14 +329,14 @@ static void __gst_no_more_pad(GstElement *element, gpointer data)
        MEDIADEMUXER_CHECK_NULL_VOID(gst_handle);
        gst_handle->selected_tracks =
            (bool *) g_malloc0_n(sizeof(bool), gst_handle->total_tracks);
-       MD_I("Allocating %p to core->selected_tracks \n", gst_handle->selected_tracks);
-       MD_I("Number of video tracks are %d\n", head_track->num_video_track);
-       MD_I("Number of audio tracks are %d\n", head_track->num_audio_track);
-       MD_I("Number of subtitle tracks are %d\n",
+       MD_I("Allocating %p to core->selected_tracks ", gst_handle->selected_tracks);
+       MD_I("Number of video tracks are %d", head_track->num_video_track);
+       MD_I("Number of audio tracks are %d", head_track->num_audio_track);
+       MD_I("Number of subtitle tracks are %d",
             head_track->num_subtitle_track);
-       MD_I("Number of other tracks are %d\n", head_track->num_other_track);
+       MD_I("Number of other tracks are %d", head_track->num_other_track);
        while (head) {
-               MD_I("track caps[%s]\n", head->caps_string);
+               MD_I("track caps[%s]", head->caps_string);
                head = head->next;
        }
        gst_handle->is_prepared = true;
@@ -352,29 +352,29 @@ static void __gst_free_stuct(track **head)
        while (temp) {
 
                if (temp->pad) {
-                       MD_I("unref GST_PAD %p\n", temp->pad);
+                       MD_I("unref GST_PAD %p", temp->pad);
                        gst_object_unref(temp->pad);
                }
                if (temp->caps) {
-                       MD_I("unref GST_PAD caps %p\n", temp->caps);
+                       MD_I("unref GST_PAD caps %p", temp->caps);
                        gst_caps_unref(temp->caps);
                }
                if (temp->caps_string) {
-                       MD_I("deallocate GST_PAD caps_string  %p\n",
+                       MD_I("deallocate GST_PAD caps_string  %p",
                             temp->caps_string);
                        g_free(temp->caps_string);
                }
                if (temp->format) {
-                       MD_I("unref media_format  %p\n", temp->format);
+                       MD_I("unref media_format  %p", temp->format);
                        media_format_unref(temp->format);
                }
                if (temp->next) {
                        track *next = temp->next;
-                       MD_I("deallocate memory %p\n", temp);
+                       MD_I("deallocate memory %p", temp);
                        g_free(temp);
                        temp = next;
                } else {
-                       MD_I("deallocate memory %p\n", temp);
+                       MD_I("deallocate memory %p", temp);
                        g_free(temp);
                        temp = NULL;
                        *head = NULL;
@@ -542,14 +542,14 @@ static int __gst_add_track_info(GstPad *pad, GstCaps *caps, track **head,
                        }
                        id3_sinkpad = gst_element_get_static_pad(id3tag, "sink");
                        if (!id3_sinkpad) {
-                               MD_E("fail to get id3demux sink pad.\n");
+                               MD_E("fail to get id3demux sink pad.");
                                goto ERROR;
                        }
                        MEDIADEMUXER_LINK_PAD(queue_src_pad, id3_sinkpad, ERROR);
                        MEDIADEMUXER_SYNC_STATE_WITH_PARENT(id3tag, ERROR);
                        id3_srcpad = gst_element_get_static_pad(id3tag, "src");
                        if (!id3_srcpad) {
-                               MD_E("fail to get id3demux src pad.\n");
+                               MD_E("fail to get id3demux src pad.");
                                goto ERROR;
                        }
                        parse_element = gst_element_factory_make("mpegaudioparse", NULL);
@@ -658,7 +658,7 @@ ERROR:
 static void __gst_on_pad_added(GstElement *element, GstPad *pad, gpointer data)
 {
        MEDIADEMUXER_FENTER();
-       MD_I("Dynamic pad created, linking demuxer/decoder\n");
+       MD_I("Dynamic pad created, linking demuxer/decoder");
        track *tmp = NULL;
        mdgst_handle_t *gst_handle = (mdgst_handle_t *)data;
        track_info *head_track = &(gst_handle->info);
@@ -685,7 +685,7 @@ static void __gst_on_pad_added(GstElement *element, GstPad *pad, gpointer data)
        tmp = head_track->head;
 
        if (!tmp) {
-               MD_I("track is NULL\n");
+               MD_I("track is NULL");
                MEDIADEMUXER_FLEAVE();
                return;
        }
@@ -694,22 +694,22 @@ static void __gst_on_pad_added(GstElement *element, GstPad *pad, gpointer data)
                tmp = tmp->next;
 
        if (!tmp->caps_string) {
-               MD_I("track caps_string is NULL\n");
+               MD_I("track caps_string is NULL");
                MEDIADEMUXER_FLEAVE();
                return;
        }
 
        if (tmp->caps_string[0] == 'v') {
-               MD_I("found Video Pad\n");
+               MD_I("found Video Pad");
                (head_track->num_video_track)++;
        } else if (tmp->caps_string[0] == 'a') {
-               MD_I("found Audio Pad\n");
+               MD_I("found Audio Pad");
                (head_track->num_audio_track)++;
        } else if (tmp->caps_string[0] == 's') {
-               MD_I("found subtitle(or Text) Pad\n");
+               MD_I("found subtitle(or Text) Pad");
                (head_track->num_subtitle_track)++;
        } else {
-               MD_W("found Pad, caps: %s\n", tmp->caps_string);
+               MD_W("found Pad, caps: %s", tmp->caps_string);
                (head_track->num_other_track)++;
        }
        MEDIADEMUXER_FLEAVE();
@@ -726,7 +726,7 @@ static int __gst_create_audio_only_pipeline(gpointer data,  GstCaps *caps)
 
        pad = gst_element_get_static_pad(gst_handle->typefind, "src");
        if (!pad) {
-               MD_E("fail to get typefind src pad.\n");
+               MD_E("fail to get typefind src pad.");
                goto ERROR;
        }
 
@@ -780,7 +780,7 @@ static void __gst_cb_typefind(GstElement *tf, guint probability,
                goto ERROR;
        }
 
-       MD_I("Media type %s found, probability %d%%\n", type, probability);
+       MD_I("Media type %s found, probability %d%%", type, probability);
        if (strstr(type, "quicktime") || strstr(type, "audio/x-m4a") ||
                strstr(type, "x-3gp") || strstr(type, "ogg") ||
                strstr(type, "flv") || strstr(type, "x-msvideo")) {
@@ -796,7 +796,7 @@ static void __gst_cb_typefind(GstElement *tf, guint probability,
 
                if (!gst_handle->demux) {
                        gst_handle->is_valid_container = false;
-                       MD_E("factory not able to create qtdemux\n");
+                       MD_E("factory not able to create qtdemux");
                        goto ERROR;
                }
                g_signal_connect(gst_handle->demux, "pad-added",
@@ -814,17 +814,17 @@ static void __gst_cb_typefind(GstElement *tf, guint probability,
 
                pad = gst_element_get_static_pad(gst_handle->typefind, "src");
                if (!pad) {
-                       MD_E("fail to get typefind src pad.\n");
+                       MD_E("fail to get typefind src pad.");
                        goto ERROR;
                }
                demuxer_pad = gst_element_get_static_pad(gst_handle->demux, "sink");
                if (!demuxer_pad) {
-                       MD_E("fail to get qtdemux sink pad.\n");
+                       MD_E("fail to get qtdemux sink pad.");
                        goto ERROR;
                }
                fake_pad = gst_element_get_static_pad(gst_handle->fakesink, "sink");
                if (!fake_pad) {
-                       MD_E("fail to get fakesink sink pad.\n");
+                       MD_E("fail to get fakesink sink pad.");
                        goto ERROR;
                }
                gst_pad_unlink(pad, fake_pad);
@@ -846,15 +846,15 @@ static void __gst_cb_typefind(GstElement *tf, guint probability,
                   || (strstr(type, "application/x-id3"))
                   || (strstr(type, "audio/x-amr-nb-sh"))
                   || (strstr(type, "audio/x-amr-wb-sh"))) {
-               MD_I("Audio only format is found\n");
+               MD_I("Audio only format is found");
                pad = gst_element_get_static_pad(gst_handle->typefind, "src");
                if (!pad) {
-                       MD_E("fail to get typefind src pad.\n");
+                       MD_E("fail to get typefind src pad.");
                        goto ERROR;
                }
                fake_pad = gst_element_get_static_pad(gst_handle->fakesink, "sink");
                if (!fake_pad) {
-                       MD_E("fail to get fakesink sink pad.\n");
+                       MD_E("fail to get fakesink sink pad.");
                        goto ERROR;
                }
                gst_pad_unlink(pad, fake_pad);
@@ -866,7 +866,7 @@ static void __gst_cb_typefind(GstElement *tf, guint probability,
                __gst_create_audio_only_pipeline(data, caps);
        } else {
                gst_handle->is_valid_container = false;
-               MD_E("Not supported container %s\n", type);
+               MD_E("Not supported container %s", type);
        }
 
        g_free(type);
@@ -918,14 +918,14 @@ static int __gst_create_pipeline(mdgst_handle_t *gst_handle, char *uri)
 #ifdef TIZEN_FEATURE_STREAMING
                gst_handle->filesrc  = gst_element_factory_make("souphttpsrc", NULL);
                remote_streaming = 1;
-               MD_I("Source is http stream. \n");
+               MD_I("Source is http stream. ");
 #else
-               MD_I("Source is http stream. Didn't support streaming\n");
+               MD_I("Source is http stream. Didn't support streaming");
                goto ERROR;
 #endif
        } else {
                gst_handle->filesrc = gst_element_factory_make("filesrc", NULL);
-               MD_I("Source is file stream \n");
+               MD_I("Source is file stream ");
        }
        if (!gst_handle->filesrc) {
                MD_E("filesrc creation failed");
@@ -977,7 +977,7 @@ static int __gst_create_pipeline(mdgst_handle_t *gst_handle, char *uri)
 
        if (gst_element_get_state(gst_handle->pipeline, &element_state, &element_pending_state, 1 * GST_SECOND)
                == GST_STATE_CHANGE_FAILURE) {
-               MD_E(" [%s] state : %s   pending : %s \n",
+               MD_E(" [%s] state : %s   pending : %s ",
                        GST_ELEMENT_NAME(gst_handle->pipeline),
                        gst_element_state_get_name(element_state),
                        gst_element_state_get_name(element_pending_state));
@@ -1028,7 +1028,7 @@ static int __gst_demuxer_prepare(MMHandleType pHandle, char *uri)
        MD_I("__gst_demuxer_prepare Creating pipeline %p", new_mediademuxer);
        ret = __gst_create_pipeline(new_mediademuxer, uri);
        if (ret != MD_ERROR_NONE) {
-               MD_E("__gst_create_pipeline() failed. returned %d\n", ret);
+               MD_E("__gst_create_pipeline() failed. returned %d", ret);
                goto ERROR;
        }
        MEDIADEMUXER_FLEAVE();
@@ -1105,11 +1105,11 @@ static int __gst_unlink_unselected_track(track *temp, int index)
        }
        queue_sink_pad = gst_element_get_static_pad(temp->queue, "sink");
        if (!queue_sink_pad) {
-               MD_E("queue_sink_pad of appsink not available\n");
+               MD_E("queue_sink_pad of appsink not available");
                return MD_ERROR;
        }
        if (gst_pad_unlink(temp->pad, queue_sink_pad) != TRUE)
-               MD_W("demuxer is already unlinked from queue for track %d\n", index);
+               MD_W("demuxer is already unlinked from queue for track %d", index);
 
        gst_object_unref(queue_sink_pad);
        MEDIADEMUXER_FLEAVE();
@@ -1125,7 +1125,7 @@ static int __gst_demuxer_start(MMHandleType pHandle)
 
        int indx;
        for (indx = 0; indx < gst_handle->total_tracks; indx++) {
-               MD_I("track indx[%d] is marked as [%d]. (0- not selected, 1= selected)\n",
+               MD_I("track indx[%d] is marked as [%d]. (0- not selected, 1= selected)",
                        indx, gst_handle->selected_tracks[indx]);
                /*
                if (gst_handle->selected_tracks[indx] ==  false)
@@ -1133,7 +1133,7 @@ static int __gst_demuxer_start(MMHandleType pHandle)
                */
                if (gst_handle->selected_tracks[indx] != true) {
                        if (__gst_unlink_unselected_track((((mdgst_handle_t *)pHandle)->info).head, indx) != MD_ERROR_NONE) {
-                               MD_E("Failed to unlink unselected tracks\n");
+                               MD_E("Failed to unlink unselected tracks");
                                ret = MD_INTERNAL_ERROR;
                                goto ERROR;
                        }
@@ -1142,21 +1142,21 @@ static int __gst_demuxer_start(MMHandleType pHandle)
        }
 
        track_info *head_track = &(gst_handle->info);
-       MD_I("Total Audio track=%d, Video track=%d, text track=%d\n",
+       MD_I("Total Audio track=%d, Video track=%d, text track=%d",
                        head_track->num_audio_track, head_track->num_video_track,
                        head_track->num_subtitle_track);
 
        track *temp = head_track->head;
        indx = 0;
        while (temp) {
-               MD_I("Got one element %p\n", temp->appsink);
+               MD_I("Got one element %p", temp->appsink);
                if (gst_element_set_state(temp->appsink, GST_STATE_PLAYING) ==
                        GST_STATE_CHANGE_FAILURE) {
                        MD_E("Failed to set into PLAYING state");
                        ret = MD_INTERNAL_ERROR;
                        goto ERROR;
                }
-               MD_I("set the state to playing\n");
+               MD_I("set the state to playing");
                indx++;
                if (temp->next) {
                        track *next = temp->next;
@@ -1181,14 +1181,14 @@ int _set_mime_text(media_format_h format, track *head)
        GstStructure *struc = NULL;
        struc = gst_caps_get_structure(head->caps, 0);
        if (!struc) {
-               MD_E("cannot get structure from caps.\n");
+               MD_E("cannot get structure from caps.");
                goto ERROR;
        }
        if (gst_structure_has_name(struc, "text/x-raw")) {
                if (media_format_set_text_mime(format, MEDIA_FORMAT_TEXT_MP4))
                        goto ERROR;
        } else {
-               MD_I("Text mime not supported so far\n");
+               MD_I("Text mime not supported so far");
                goto ERROR;
        }
 
@@ -1212,7 +1212,7 @@ int _set_mime_video(media_format_h format, track *head)
        media_format_mimetype_e mime_type = MEDIA_FORMAT_MAX;
        struc = gst_caps_get_structure(head->caps, 0);
        if (!struc) {
-               MD_E("cannot get structure from caps.\n");
+               MD_E("cannot get structure from caps.");
                goto ERROR;
        }
        if (gst_structure_has_name(struc, "video/x-h264")) {
@@ -1222,21 +1222,21 @@ int _set_mime_video(media_format_h format, track *head)
        } else if (gst_structure_has_name(struc, "video/mpeg")) {
                mime_type = MEDIA_FORMAT_MPEG4_SP;
        } else {
-               MD_W("Video mime (%s) not supported so far\n", gst_structure_get_name(struc));
+               MD_W("Video mime (%s) not supported so far", gst_structure_get_name(struc));
                goto ERROR;
        }
        if (media_format_set_video_mime(format, mime_type)) {
-               MD_E("Unable to set video mime type (%x)\n", mime_type);
+               MD_E("Unable to set video mime type (%x)", mime_type);
                goto ERROR;
        }
        gst_structure_get_int(struc, "width", &src_width);
        gst_structure_get_int(struc, "height", &src_height);
        if (media_format_set_video_width(format, src_width)) {
-               MD_E("Unable to set video width\n");
+               MD_E("Unable to set video width");
                goto ERROR;
        }
        if (media_format_set_video_height(format, src_height)) {
-               MD_E("Unable to set video height\n");
+               MD_E("Unable to set video height");
                goto ERROR;
        }
        gst_structure_get_fraction(struc, "framerate",  &frame_rate_numerator, &frame_rate_denominator);
@@ -1247,7 +1247,7 @@ int _set_mime_video(media_format_h format, track *head)
 
        MD_I("set frame rate %d", frame_rate);
        if (media_format_set_video_frame_rate(format, frame_rate)) {
-               MD_E("Unable to set video frame rate\n");
+               MD_E("Unable to set video frame rate");
                goto ERROR;
        }
        MEDIADEMUXER_FLEAVE();
@@ -1271,7 +1271,7 @@ int _set_mime_audio(media_format_h format, track *head)
 
        struc = gst_caps_get_structure(head->caps, 0);
        if (!struc) {
-               MD_E("cannot get structure from caps.\n");
+               MD_E("cannot get structure from caps.");
                goto ERROR;
        }
 
@@ -1299,7 +1299,7 @@ int _set_mime_audio(media_format_h format, track *head)
                        if ((layer == 3) || (id3_flag == 1)) {
                                mime_type = MEDIA_FORMAT_MP3;
                        } else {
-                               MD_I("No Support for MPEG%d Layer %d media\n", mpegversion, layer);
+                               MD_I("No Support for MPEG%d Layer %d media", mpegversion, layer);
                                goto ERROR;
                        }
                }
@@ -1323,7 +1323,7 @@ int _set_mime_audio(media_format_h format, track *head)
        } else if (gst_structure_has_name(struc, "audio/x-vorbis")) {
                mime_type = MEDIA_FORMAT_VORBIS;
        } else {
-               MD_W("Audio mime (%s) not supported so far\n", gst_structure_get_name(struc));
+               MD_W("Audio mime (%s) not supported so far", gst_structure_get_name(struc));
                goto ERROR;
        }
        if (media_format_set_audio_mime(format, mime_type))
@@ -1384,13 +1384,13 @@ static int __gst_demuxer_get_track_info(MMHandleType pHandle,
        if (temp->format != NULL) {
                ret = media_format_ref(temp->format);
                if (ret != MEDIA_FORMAT_ERROR_NONE) {
-                       MD_E("Mediaformat reference count increment failed. returned %d\n", ret);
+                       MD_E("Mediaformat reference count increment failed. returned %d", ret);
                        ret = MD_INTERNAL_ERROR;
                        goto ERROR;
                }
                ret = media_format_make_writable(temp->format, format); /* copy the content */
                if (ret != MEDIA_FORMAT_ERROR_NONE) {
-                       MD_E("Mediaformat create copy failed. returned %d\n", ret);
+                       MD_E("Mediaformat create copy failed. returned %d", ret);
                        media_format_unref(temp->format);
                        ret = MD_INTERNAL_ERROR;
                        goto ERROR;
@@ -1400,35 +1400,35 @@ static int __gst_demuxer_get_track_info(MMHandleType pHandle,
        }
        ret = media_format_create(&(temp->format));
        if (ret != MEDIA_FORMAT_ERROR_NONE) {
-               MD_E("Mediaformat creation failed. returned %d\n", ret);
+               MD_E("Mediaformat creation failed. returned %d", ret);
                ret = MD_INTERNAL_ERROR;
                goto ERROR;
        }
 
-       MD_I("CAPS for selected track [%d] is [%s]\n", index, temp->caps_string);
-       MD_I("format ptr[%p]\n", temp->format);
+       MD_I("CAPS for selected track [%d] is [%s]", index, temp->caps_string);
+       MD_I("format ptr[%p]", temp->format);
        if (temp->caps_string[0] == 'a') {
-               MD_I("Setting for Audio \n");
+               MD_I("Setting for Audio ");
                _set_mime_audio(temp->format, temp);
        } else if (temp->caps_string[0] == 'v') {
-               MD_I("Setting for Video \n");
+               MD_I("Setting for Video ");
                _set_mime_video(temp->format, temp);
        } else if (temp->caps_string[0] == 't') {
-               MD_I("Setting for Subtitle\n");
+               MD_I("Setting for Subtitle");
                _set_mime_text(temp->format, temp);
        } else {
-               MD_W("Not supported so far (except audio, video and subtitle)\n");
+               MD_W("Not supported so far (except audio, video and subtitle)");
        }
 
        ret = media_format_ref(temp->format);   /* increment the ref to retain the original content */
        if (ret != MEDIA_FORMAT_ERROR_NONE) {
-               MD_E("Mediaformat reference count increment failed. returned %d\n", ret);
+               MD_E("Mediaformat reference count increment failed. returned %d", ret);
                ret = MD_INTERNAL_ERROR;
                goto ERROR;
        }
        ret = media_format_make_writable(temp->format, format); /* copy the content */
        if (ret != MEDIA_FORMAT_ERROR_NONE) {
-               MD_E("Mediaformat create copy failed. returned %d\n", ret);
+               MD_E("Mediaformat create copy failed. returned %d", ret);
                media_format_unref(temp->format);
                ret = MD_INTERNAL_ERROR;
                goto ERROR;
@@ -1451,49 +1451,49 @@ static int __gst_copy_buf_to_media_packet(media_packet_h out_pkt,
        GstMapInfo map;
 
        if (!gst_buffer_map(buffer, &map, GST_MAP_READ)) {
-               MD_E("gst_buffer_map failed\n");
+               MD_E("gst_buffer_map failed");
                ret = MD_ERROR_UNKNOWN;
                goto ERROR;
        }
        /* copy data */
        media_packet_get_buffer_size(out_pkt, &size);
-       MD_I("Media packet Buffer capacity: %"PRIu64" GST Buffer size = %"G_GSIZE_FORMAT"\n", size, map.size);
+       MD_I("Media packet Buffer capacity: %"PRIu64" GST Buffer size = %"G_GSIZE_FORMAT"", size, map.size);
        if (size < (uint64_t)map.size) {
                MD_W("Media packet Buffer capacity[%"PRIu64"] is \
-                       less than the GST Buffer size[%"G_GSIZE_FORMAT"]. Resizing...\n", size, map.size);
+                       less than the GST Buffer size[%"G_GSIZE_FORMAT"]. Resizing...", size, map.size);
                ret = media_packet_set_buffer_size(out_pkt, (uint64_t)map.size);
                media_packet_get_buffer_size(out_pkt, &size);
-               MD_I("Media packet Buffer NEW  capacity: %"PRIu64"\n", size);
+               MD_I("Media packet Buffer NEW  capacity: %"PRIu64"", size);
        }
        if (media_packet_get_buffer_data_ptr(out_pkt, &pkt_data)) {
-               MD_E("unable to get the buffer pointer from media_packet_get_buffer_data_ptr\n");
+               MD_E("unable to get the buffer pointer from media_packet_get_buffer_data_ptr");
                ret = MD_ERROR_UNKNOWN;
                goto ERROR;
        }
        memcpy((char *)pkt_data, map.data, map.size);
        if (media_packet_set_pts(out_pkt, GST_BUFFER_PTS(buffer))) {
-               MD_E("unable to set the pts\n");
+               MD_E("unable to set the pts");
                ret = MD_ERROR_UNKNOWN;
                goto ERROR;
        }
        if (media_packet_set_dts(out_pkt, GST_BUFFER_DTS(buffer))) {
-               MD_E("unable to set the dts\n");
+               MD_E("unable to set the dts");
                ret = MD_ERROR_UNKNOWN;
                goto ERROR;
        }
        if (media_packet_set_duration(out_pkt, GST_BUFFER_DURATION(buffer))) {
-               MD_E("unable to set the duration\n");
+               MD_E("unable to set the duration");
                ret = MD_ERROR_UNKNOWN;
                goto ERROR;
        }
        if (media_packet_set_buffer_size(out_pkt, gst_buffer_get_size(buffer))) {
-               MD_E("unable to set the buffer size\n");
+               MD_E("unable to set the buffer size");
                ret = MD_ERROR_UNKNOWN;
                goto ERROR;
        }
        if (!GST_BUFFER_FLAG_IS_SET(buffer, GST_BUFFER_FLAG_DELTA_UNIT)) {
                if (media_packet_set_flags(out_pkt, MEDIA_PACKET_SYNC_FRAME)) {
-                       MD_E("unable to set the buffer flag\n");
+                       MD_E("unable to set the buffer flag");
                        ret = MD_ERROR_UNKNOWN;
                        goto ERROR;
                }
@@ -1502,13 +1502,13 @@ static int __gst_copy_buf_to_media_packet(media_packet_h out_pkt,
        if (codec_data) {
                GstMapInfo codec_data_map;
                if (!gst_buffer_map(codec_data, &codec_data_map, GST_MAP_READ)) {
-                       MD_E("codec data buffer map failed\n");
+                       MD_E("codec data buffer map failed");
                        ret = MD_ERROR_UNKNOWN;
                        goto ERROR;
                }
                if (media_packet_set_codec_data(out_pkt, (void *)codec_data_map.data,
                        (unsigned int)codec_data_map.size)) {
-                       MD_E("unable to set the codec data\n");
+                       MD_E("unable to set the codec data");
                        ret = MD_ERROR_UNKNOWN;
                        gst_buffer_unmap(codec_data, &codec_data_map);
                        goto ERROR;
@@ -1532,13 +1532,13 @@ static int __gst_demuxer_create_eos_packet(media_format_h fmt, media_packet_h *o
 
        MEDIADEMUXER_FENTER();
        if (media_packet_new_alloc(fmt, NULL, NULL, &mediabuf)) {
-               MD_E("media_packet_new_alloc failed\n");
+               MD_E("media_packet_new_alloc failed");
                ret = MD_ERROR;
                goto ERROR;
        }
 
        if (media_packet_set_flags(mediabuf, MEDIA_PACKET_END_OF_STREAM)) {
-               MD_E("unable to set EOS flag\n");
+               MD_E("unable to set EOS flag");
                ret = MD_ERROR_UNKNOWN;
                goto ERROR;
        }
@@ -1568,7 +1568,7 @@ static int __gst_demuxer_read_sample(MMHandleType pHandle,
 
        track *atrack = demuxer->info.head;
        if ((demuxer->selected_tracks)[track_indx] == false) {
-               MD_E("Track Not selected\n");
+               MD_E("Track Not selected");
                ret = MD_ERROR;
                goto ERROR;
        }
@@ -1579,7 +1579,7 @@ static int __gst_demuxer_read_sample(MMHandleType pHandle,
                        track *next = atrack->next;
                        atrack = next;
                } else {
-                       MD_E("Invalid track Index\n");
+                       MD_E("Invalid track Index");
                        ret = MD_ERROR_INVALID_ARGUMENT;
                        goto ERROR;
                }
@@ -1587,12 +1587,12 @@ static int __gst_demuxer_read_sample(MMHandleType pHandle,
        }
 
        if (!atrack) {
-               MD_E("atrack is NULL\n");
+               MD_E("atrack is NULL");
                goto ERROR;
        }
 
        if (indx != track_indx) {
-               MD_E("Invalid track Index\n");
+               MD_E("Invalid track Index");
                ret = MD_ERROR_INVALID_ARGUMENT;
                goto ERROR;
        }
@@ -1602,7 +1602,7 @@ static int __gst_demuxer_read_sample(MMHandleType pHandle,
        sample = gst_app_sink_pull_sample((GstAppSink *) sink);
        if (sample == NULL) {
                if (gst_app_sink_is_eos((GstAppSink *) sink)) {
-                       MD_W("End of stream (EOS) reached, triggering the eos callback\n");
+                       MD_W("End of stream (EOS) reached, triggering the eos callback");
                        ret = MD_ERROR_NONE;
                        *outbuf = NULL;
                        __gst_eos_callback(track_indx, demuxer);
@@ -1611,7 +1611,7 @@ static int __gst_demuxer_read_sample(MMHandleType pHandle,
                        __gst_demuxer_create_eos_packet(atrack->format, outbuf);
                        return ret;
                } else {
-                       MD_E("__gst_demuxer_read_sample failed\n");
+                       MD_E("__gst_demuxer_read_sample failed");
                        *outbuf = NULL;
                        ret = MD_ERROR_UNKNOWN;
                        return ret;
@@ -1619,14 +1619,14 @@ static int __gst_demuxer_read_sample(MMHandleType pHandle,
        }
 
        if (media_packet_new_alloc(atrack->format, NULL, NULL, &mediabuf)) {
-               MD_E("media_packet_new_alloc failed\n");
+               MD_E("media_packet_new_alloc failed");
                ret = MD_ERROR;
                goto ERROR;
        }
 
        GstBuffer *buffer = gst_sample_get_buffer(sample);
        if (buffer == NULL) {
-               MD_E("gst_sample_get_buffer returned NULL pointer\n");
+               MD_E("gst_sample_get_buffer returned NULL pointer");
                ret = MD_ERROR_UNKNOWN;
                goto ERROR;
        }
@@ -1682,7 +1682,7 @@ static int __gst_demuxer_seek(MMHandleType pHandle, gint64 pos1)
                                MD_E("Failed to set into PAUSED state");
                                goto ERROR;
                        }
-                       MD_I("set the state to paused\n");
+                       MD_I("set the state to paused");
                }
                indx++;
                if (temp_track->next) {
@@ -1699,7 +1699,7 @@ static int __gst_demuxer_seek(MMHandleType pHandle, gint64 pos1)
 
        indx = 0;
        while (temp) {
-               MD_I("Got one element %p {%s}\n", temp->appsink, GST_ELEMENT_NAME(temp->appsink));
+               MD_I("Got one element %p {%s}", temp->appsink, GST_ELEMENT_NAME(temp->appsink));
                if (gst_handle->selected_tracks[indx] == true) {
                        temp->need_codec_data = TRUE;
 
@@ -1707,12 +1707,12 @@ static int __gst_demuxer_seek(MMHandleType pHandle, gint64 pos1)
                                if (!gst_element_seek(temp->appsink, rate, GST_FORMAT_TIME,
                                        GST_SEEK_FLAG_FLUSH | GST_SEEK_FLAG_KEY_UNIT | GST_SEEK_FLAG_SNAP_BEFORE,
                                        GST_SEEK_TYPE_SET, pos, GST_SEEK_TYPE_NONE, GST_CLOCK_TIME_NONE)) {
-                                       MD_E("Seek failed!\n");
+                                       MD_E("Seek failed!");
                                        goto ERROR;
                                }
                                is_seek = TRUE;
                        }
-                       MD_I("Seek success...setting appsink to playing state\n");
+                       MD_I("Seek success...setting appsink to playing state");
                        if (gst_element_set_state(temp->appsink, GST_STATE_PLAYING) == GST_STATE_CHANGE_FAILURE) {
                                MD_E("Failed to set into PLAYING state");
                                goto ERROR;
@@ -1799,7 +1799,7 @@ static void __gst_clear_struct(mdgst_handle_t *gst_handle)
        MEDIADEMUXER_FENTER();
        MEDIADEMUXER_CHECK_NULL_VOID(gst_handle);
        if (gst_handle->selected_tracks) {
-               MD_I("Deallocating gst_handle->selected_tracks %p\n",
+               MD_I("Deallocating gst_handle->selected_tracks %p",
                     gst_handle->selected_tracks);
                g_free(gst_handle->selected_tracks);
                gst_handle->selected_tracks = NULL;
@@ -1844,9 +1844,9 @@ static int __gst_demuxer_unprepare(MMHandleType pHandle)
        track *atrack = gst_handle->info.head;
        while (atrack) {
                if ((gst_handle->selected_tracks)[indx] == false) {
-                       MD_I("Track [%d] Not selected\n", indx);
+                       MD_I("Track [%d] Not selected", indx);
                } else {
-                       MD_I("Track [%d] to unset appsink...\n", indx);
+                       MD_I("Track [%d] to unset appsink...", indx);
                        __gst_demuxer_unset_track(pHandle, indx);
                }
                if (atrack->next) {
@@ -1869,7 +1869,7 @@ static int __gst_demuxer_unprepare(MMHandleType pHandle)
                gst_handle->bus_watch_id = 0;
        }
 
-       MD_I("__gst_demuxer_stop pipeline %p\n", gst_handle->pipeline);
+       MD_I("__gst_demuxer_stop pipeline %p", gst_handle->pipeline);
        if (_md_gst_destroy_pipeline(gst_handle->pipeline) != MD_ERROR_NONE) {
                ret = MD_ERROR;
                goto ERROR;
@@ -1888,7 +1888,7 @@ static int __gst_demuxer_destroy(MMHandleType pHandle)
        MEDIADEMUXER_CHECK_NULL(pHandle);
        mdgst_handle_t *new_mediademuxer = (mdgst_handle_t *)pHandle;
 
-       MD_I("__gst_demuxer_destroy deallocating new_mediademuxer:%p\n",
+       MD_I("__gst_demuxer_destroy deallocating new_mediademuxer:%p",
             new_mediademuxer);
        g_free(new_mediademuxer);
        MEDIADEMUXER_FLEAVE();
@@ -1904,18 +1904,18 @@ int __gst_set_error_cb(MMHandleType pHandle,
        mdgst_handle_t *gst_handle = (mdgst_handle_t *)pHandle;
 
        if (gst_handle->user_cb[_GST_EVENT_TYPE_ERROR]) {
-               MD_E("Already set mediademuxer_error_cb\n");
+               MD_E("Already set mediademuxer_error_cb");
                ret = MD_ERROR_INVALID_ARGUMENT;
                goto ERROR;
        } else {
                if (!callback) {
-                       MD_E("fail invalid argument (callback)\n");
+                       MD_E("fail invalid argument (callback)");
                        ret = MD_ERROR_INVALID_ARGUMENT;
                        goto ERROR;
                }
        }
 
-       MD_I("Set event handler callback(cb = %p, data = %p)\n", callback, user_data);
+       MD_I("Set event handler callback(cb = %p, data = %p)", callback, user_data);
 
        gst_handle->user_cb[_GST_EVENT_TYPE_ERROR] = (gst_error_cb)callback;
        gst_handle->user_data[_GST_EVENT_TYPE_ERROR] = user_data;
@@ -1934,18 +1934,18 @@ int __gst_set_eos_cb(MMHandleType pHandle, gst_eos_cb callback, void *user_data)
        mdgst_handle_t *gst_handle = (mdgst_handle_t *)pHandle;
 
        if (gst_handle->user_cb[_GST_EVENT_TYPE_EOS]) {
-               MD_E("Already set mediademuxer_eos_cb\n");
+               MD_E("Already set mediademuxer_eos_cb");
                ret = MD_ERROR_INVALID_ARGUMENT;
                goto ERROR;
        } else {
                if (!callback) {
-                       MD_E("fail invalid argument (callback)\n");
+                       MD_E("fail invalid argument (callback)");
                        ret = MD_ERROR_INVALID_ARGUMENT;
                        goto ERROR;
                }
        }
 
-       MD_I("Set event handler callback(cb = %p, data = %p)\n", callback, user_data);
+       MD_I("Set event handler callback(cb = %p, data = %p)", callback, user_data);
        gst_handle->user_cb[_GST_EVENT_TYPE_EOS] = (gst_eos_cb)callback;
        gst_handle->user_data[_GST_EVENT_TYPE_EOS] = user_data;
        MEDIADEMUXER_FLEAVE();