From: Shobhit Verma <shobhit.v@samsung.com>
Date: Fri, 17 Feb 2023 11:14:41 +0000 (+0530)
Subject: [ITC][capi-media-tool][NonACR] Fixed build warnings
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=db522be022cb6aa756f6502ae4c23b8e8344757b;p=test%2Ftct%2Fnative%2Fapi.git

[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 ad989fc01..da8379e78 100755
--- 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
new mode 100755
index 4072dfb40..ad2b6dd8b
--- 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");