[ITC][capi-media-tool][NonACR] Fixed build warnings

Change-Id: I8cab9c564bfb0a5e5bdfa1ede03b4984e3107c2e
Signed-off-by: Shobhit Verma <shobhit.v@samsung.com>

diff --git a/src/itc/capi-media-tool/ITs-capi-media-format.c b/src/itc/capi-media-tool/ITs-capi-media-format.c
index ad989fc0121188542e19f6e1fa59c54b912ce14b..da8379e78b96b93c3ff0c736108958759f05d627 100755 (executable)
--- a/src/itc/capi-media-tool/ITs-capi-media-format.c
+++ b/src/itc/capi-media-tool/ITs-capi-media-format.c
@@ -92,6 +92,12 @@ int g_nBitDepth[] = {
        32
 };
 
+/**
+* @function            ITs_media_format_startup
+* @description         Called before each test, cretes media format handle
+* @parameter           NA
+* @return                      NA
+*/
 void ITs_media_format_startup(void)
 {
        struct stat stBuff;
@@ -111,6 +117,12 @@ void ITs_media_format_startup(void)
        return;
 }
 
+/**
+* @function            ITs_media_format_cleanup
+* @description         Called after each test
+* @parameter           NA
+* @return                      NA
+*/
 void ITs_media_format_cleanup(void)
 {
 #if DEBUG
@@ -131,7 +143,7 @@ void ITs_media_format_cleanup(void)
 //& purpose: Increase/Decrease reference count of media_format_h object
 //& type: auto
 /**
-* @testcase                    ITc_media_format_set_audio_avg_bps_p
+* @testcase                    ITc_media_format_ref_p
 * @since_tizen                 2.3
 * @author              SRID(gaurav.m2)
 * @reviewer            SRID(gupta.sanjay)
@@ -849,7 +861,7 @@ int ITc_media_format_set_get_video_frame_rate_p(void)
 int ITc_media_format_set_get_container_mime_p(void)
 {
        START_TEST;
-       media_format_h fmt_container;
+
        int nRet = -1;
 
        media_format_mimetype_e eGetMimeType;
@@ -877,13 +889,13 @@ int ITc_media_format_set_get_container_mime_p(void)
                nRet = media_format_set_container_mime(g_hfmt,eMimeType[nEnumCounter]);
                if ( nRet != MEDIA_FORMAT_ERROR_NONE )
                {
-                       FPRINTF("[Line : %d][%s] Failed , for enum = [ %s ]\\n", __LINE__, API_NAMESPACE, MediaFormatMimeTypeGetEnum(eMimeType[nEnumCounter]), MediaFormatGetError(nRet));
+                       FPRINTF("[Line : %d][%s] Failed , for enum = [ %s ] error = [%s] \\n", __LINE__, API_NAMESPACE, MediaFormatMimeTypeGetEnum(eMimeType[nEnumCounter]), MediaFormatGetError(nRet));
                        return 1;
                }
                nRet = media_format_get_container_mime(g_hfmt,&eGetMimeType);
                if ( nRet != MEDIA_FORMAT_ERROR_NONE )
                {
-                       FPRINTF("[Line : %d][%s] Failed , for enum = [ %s ]\\n", __LINE__, API_NAMESPACE, MediaFormatMimeTypeGetEnum(eGetMimeType), MediaFormatGetError(nRet));
+                       FPRINTF("[Line : %d][%s] Failed , for enum = [ %s ] error = [%s] \\n", __LINE__, API_NAMESPACE, MediaFormatMimeTypeGetEnum(eGetMimeType), MediaFormatGetError(nRet));
                        return 1;
                }
                if (eMimeType[nEnumCounter] != eGetMimeType)
@@ -917,14 +929,13 @@ int ITc_media_format_set_text_mime_set_text_type_get_text_info_p(void)
 {
     START_TEST;
 
-    media_format_h hFmtContainer = NULL;
     media_format_mimetype_e eGetMimeType;
     media_format_text_type_e eGetTextType;
     int nRet = -1;
 
 
     media_format_mimetype_e eSetMimeType[] = {
-        /* Text */                
+        /* Text */
         MEDIA_FORMAT_TEXT_MP4,                            /**< media format mime type is MP4, TEXT, (Since 3.0) */
         MEDIA_FORMAT_TEXT_3GP,                            /**< media format mime type is 3GP, TEXT, (Since 3.0) */
     };
@@ -1126,7 +1137,7 @@ int ITc_media_format_get_extra_p(void)
        {
                if (strcmp (extra, (char*)get_extra) != 0)
                {
-                       FPRINTF("[Line : %d][%s] media format received , get_extra = %s:failure\\n", __LINE__, API_NAMESPACE,get_extra);
+                       FPRINTF("[Line : %d][%s] media format set and get value mismatch.\\n", __LINE__, API_NAMESPACE);
                        return 1;
                }
        }
@@ -1157,8 +1168,8 @@ int ITc_media_format_channel_positions_to_from_mask_p(void)
        START_TEST;
 
        int nRet = MEDIA_FORMAT_ERROR_NONE;
-       uint64_t nsetChMask;
-       uint64_t ngetChMask;
+       uint64_t nsetChMask = 0;
+       uint64_t ngetChMask = 0;
        int nAudioCh = 1;
        int nChannels;
        media_format_channel_position_e *eGetPositions;

diff --git a/src/itc/capi-media-tool/ITs-capi-media-tool.c b/src/itc/capi-media-tool/ITs-capi-media-tool.c
old mode 100644 (file)
new mode 100755 (executable)
index 4072dfb..ad2b6dd
--- a/src/itc/capi-media-tool/ITs-capi-media-tool.c
+++ b/src/itc/capi-media-tool/ITs-capi-media-tool.c
@@ -48,6 +48,12 @@ static void DisposeCallbackTbmSurface(media_packet_h packet, void * userdata)
 
 }
 
+/**
+* @function            ITs_media_tools_startup
+* @description         Called before each test, cretes media format handle
+* @parameter           NA
+* @return                      NA
+*/
 void ITs_media_tools_startup(void)
 {
        struct stat stBuff;
@@ -67,6 +73,12 @@ void ITs_media_tools_startup(void)
        return;
 }
 
+/**
+* @function            ITs_media_format_cleanup
+* @description         Called after each test
+* @parameter           NA
+* @return                      NA
+*/
 void ITs_media_tools_cleanup(void)
 {
 #if DEBUG
@@ -1174,20 +1186,17 @@ int ITc_media_packet_set_get_flags_p(void)
 
        for ( nEnumCounter=0; nEnumCounter<nEnumSize; nEnumCounter++)
        {
-               char *szEnumVal = NULL;
-               char *szGetEnumVal = NULL;
-
                nRet = media_packet_set_flags(hPacket, eMediaBufferFlags[nEnumCounter]);
                if ( nRet != MEDIA_PACKET_ERROR_NONE )
                {
-                       FPRINTF("[Line : %d][%s] Failed , for enum = [ %s ]\\n", __LINE__, API_NAMESPACE, MediaBufferFlagsGetEnum(eMediaBufferFlags[nEnumCounter]), MediaPacketGetError(nRet));
+                       FPRINTF("[Line : %d][%s] Failed , for enum = [ %s ] error = [%s] \\n", __LINE__, API_NAMESPACE, MediaBufferFlagsGetEnum(eMediaBufferFlags[nEnumCounter]), MediaPacketGetError(nRet));
                        media_packet_unref(hPacket);
                        return 1;
                }
                nRet = media_packet_get_flags(hPacket, &eGetMediaBufferFlags);
                if ( nRet != MEDIA_PACKET_ERROR_NONE )
                {
-                       FPRINTF("[Line : %d][%s] Failed , for enum = [ %s ]\\n", __LINE__, API_NAMESPACE, MediaBufferFlagsGetEnum(eGetMediaBufferFlags), MediaPacketGetError(nRet));
+                       FPRINTF("[Line : %d][%s] Failed , for enum = [ %s ] error = [%s] \\n", __LINE__, API_NAMESPACE, MediaBufferFlagsGetEnum(eGetMediaBufferFlags), MediaPacketGetError(nRet));
                        media_packet_unref(hPacket);
                        return 1;
                }
@@ -1200,7 +1209,7 @@ int ITc_media_packet_set_get_flags_p(void)
                nRet = media_packet_unset_flags(hPacket, eMediaBufferFlags[nEnumCounter]);
                if ( nRet != MEDIA_PACKET_ERROR_NONE )
                {
-                       FPRINTF("[Line : %d][%s] Failed , for enum = [ %s ]\\n", __LINE__, API_NAMESPACE, MediaBufferFlagsGetEnum(eMediaBufferFlags[nEnumCounter]), MediaPacketGetError(nRet));
+                       FPRINTF("[Line : %d][%s] Failed , for enum = [ %s ]  error = [%s] \\n", __LINE__, API_NAMESPACE, MediaBufferFlagsGetEnum(eMediaBufferFlags[nEnumCounter]), MediaPacketGetError(nRet));
                        media_packet_unref(hPacket);
                        return 1;
                }
@@ -1340,7 +1349,7 @@ int ITc_media_packet_get_number_of_video_planes_p(void)
        media_packet_h hPacketTbm;
        unsigned int *nTbmFormatArray = NULL;
        unsigned int g_nTotalFormatCount = 0;
-       int nNum = -1;
+       unsigned int nNum = -1;
        int nRet = -1;
 
        CHECK_HANDLE(g_hfmt, "media_format_make_writable");