Apply tizen coding rulesubmit/tizen_ivi/20160217.000001submit/tizen_ivi/20160217.000000submit/tizen_common/20160218.142243submit/tizen/20160127.062708accepted/tizen/wearable/20160127.073009accepted/tizen/tv/20160127.073003accepted/tizen/mobile/20160127.072956accepted/tizen/ivi/20160218.022828

Change-Id: I9204f8b2072316eceba2bce21dd01a8f71b5eed3

2 files changed, 187 insertions, 203 deletions

diff --git a/src/metadata_extractor.c b/src/metadata_extractor.c
index c08e706..83ab272 100755
--- a/src/metadata_extractor.c
+++ b/src/metadata_extractor.c
@@ -87,42 +87,37 @@ static int __metadata_extractor_check_and_extract_meta(metadata_extractor_s *met
 		} else {
 			metadata_extractor_debug("Extract Stream Info");
 			ret = __metadata_extractor_get_stream_info(metadata, metadata->path);
-			if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+			if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 				return ret;
-			}
 		}
 	} else if (metadata_type == METADATA_TYPE_ATTR) {
-		if ((metadata) && (metadata->attr_h)) {
+		if ((metadata) && (metadata->attr_h))
 			return ret;
-		} else {
+		else {
 			metadata_extractor_debug("Extract Content attrs");
 			if (metadata->path != NULL) {
 				ret = __metadata_extractor_create_content_attrs(metadata, metadata->path);
-				if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+				if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 					return ret;
-				}
 			} else if (metadata->buffer != NULL) {
 				ret = __metadata_extractor_create_content_attrs_from_buffer(metadata, metadata->buffer, metadata->buffer_size);
-				if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+				if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 					return ret;
-				}
 			}
 		}
 	} else if (metadata_type == METADATA_TYPE_TAG) {
-		if ((metadata) && (metadata->tag_h)) {
+		if ((metadata) && (metadata->tag_h))
 			return ret;
-		} else {
+		else {
 			metadata_extractor_debug("Extract Tag attrs");
 			if (metadata->path != NULL) {
 				ret = __metadata_extractor_create_tag_attr(metadata, metadata->path);
-				if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+				if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 					return ret;
-				}
 			} else if (metadata->buffer != NULL) {
 				ret = __metadata_extractor_create_tag_attrs_from_buffer(metadata, metadata->buffer, metadata->buffer_size);
-				if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+				if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 					return ret;
-				}
 			}
 		}
 	} else {
@@ -1147,14 +1142,12 @@ int metadata_extractor_get_synclyrics(metadata_extractor_h metadata, int index,
 	}
 
 	ret = __metadata_extractor_check_and_extract_meta(_metadata, METADATA_TYPE_TAG);
-	if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+	if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 		return ret;
-	}
 
 	ret = __metadata_extractor_get_synclyrics_pair_num(_metadata, &_synclyrics_num);
-	if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+	if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 		return ret;
-	}
 
 	if ((_synclyrics_num > 0) && (index < _synclyrics_num)) {
 		ret = mm_file_get_synclyrics_info((MMHandleType)_metadata->tag_h, index, &_time_info, &_lyrics);
@@ -1168,9 +1161,8 @@ int metadata_extractor_get_synclyrics(metadata_extractor_h metadata, int index,
 				metadata_extractor_error("OUT_OF_MEMORY(0x%08x)", METADATA_EXTRACTOR_ERROR_OUT_OF_MEMORY);
 				return METADATA_EXTRACTOR_ERROR_OUT_OF_MEMORY;
 			}
-		} else {
+		} else
 			*lyrics = NULL;
-		}
 
 		*time_stamp = _time_info;
 	} else {
@@ -1201,184 +1193,183 @@ int metadata_extractor_get_metadata(metadata_extractor_h metadata, metadata_extr
 			ret = __metadata_extractor_check_and_extract_meta(_metadata, METADATA_TYPE_STREAM);
 		else
 			ret = __metadata_extractor_check_and_extract_meta(_metadata, METADATA_TYPE_ATTR);
-	} else if ((attribute >= METADATA_DURATION) && (attribute < METADATA_HAS_AUDIO)) {
+	} else if ((attribute >= METADATA_DURATION) && (attribute < METADATA_HAS_AUDIO))
 		ret = __metadata_extractor_check_and_extract_meta(_metadata, METADATA_TYPE_ATTR);
-	} else if ((attribute > METADATA_HAS_AUDIO) && (attribute <= METADATA_ROTATE)) {
+	else if ((attribute > METADATA_HAS_AUDIO) && (attribute <= METADATA_ROTATE))
 		ret = __metadata_extractor_check_and_extract_meta(_metadata, METADATA_TYPE_TAG);
-	} else {
+	else {
 		metadata_extractor_error("INVALID_PARAMETER [%d]", attribute);
 		ret = METADATA_EXTRACTOR_ERROR_INVALID_PARAMETER;
 	}
 
-	if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+	if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 		return ret;
-	}
 
 	/*metadata_extractor_debug("attribute [%d]", attribute); */
 
 	switch (attribute) {
-		case METADATA_DURATION: {
-			is_string = 0;
-			ret = __metadata_extractor_get_duration(_metadata, &i_value);
-			break;
-		}
-		case METADATA_VIDEO_BITRATE: {
-			is_string = 0;
-			ret = __metadata_extractor_get_video_bitrate(_metadata, &i_value);
-			break;
-		}
-		case METADATA_VIDEO_FPS: {
-			is_string = 0;
-			ret = __metadata_extractor_get_video_FPS(_metadata, &i_value);
-			break;
-		}
-		case METADATA_VIDEO_WIDTH: {
-			is_string = 0;
-			ret = __metadata_extractor_get_video_width(_metadata, &i_value);
-			break;
-		}
-		case METADATA_VIDEO_HEIGHT: {
-			is_string = 0;
-			ret = __metadata_extractor_get_video_height(_metadata, &i_value);
-			break;
-		}
-		case METADATA_HAS_VIDEO: {
-			is_string = 0;
-			ret = __metadata_extractor_get_video_track_count(_metadata, &i_value);
-			break;
-		}
-		case METADATA_AUDIO_BITRATE: {
-			is_string = 0;
-			ret = __metadata_extractor_get_audio_bitrate(_metadata, &i_value);
-			break;
-		}
-		case METADATA_AUDIO_CHANNELS: {
-			is_string = 0;
-			ret = __metadata_extractor_get_audio_channel(_metadata, &i_value);
-			break;
-		}
-		case METADATA_AUDIO_SAMPLERATE: {
-			is_string = 0;
-			ret = __metadata_extractor_get_audio_samplerate(_metadata, &i_value);
-			break;
-		}
-		case METADATA_AUDIO_BITPERSAMPLE: {
-			is_string = 0;
-			ret = __metadata_extractor_get_audio_bitpersample(_metadata, &i_value);
-			break;
-		}
-		case METADATA_HAS_AUDIO: {
-			is_string = 0;
-			ret = __metadata_extractor_get_audio_track_count(_metadata, &i_value);
-			break;
-		}
-		case METADATA_ARTIST: {
-			is_string = 1;
-			ret = __metadata_extractor_get_artist(_metadata, &s_value);
-			break;
-		}
-		case METADATA_TITLE: {
-			is_string = 1;
-			ret = __metadata_extractor_get_title(_metadata, &s_value);
-			break;
-		}
-		case METADATA_ALBUM: {
-			is_string = 1;
-			ret = __metadata_extractor_get_album(_metadata, &s_value);
-			break;
-		}
-		case METADATA_ALBUM_ARTIST: {
-			is_string = 1;
-			ret = __metadata_extractor_get_album_artist(_metadata, &s_value);
-			break;
-		}
-		case METADATA_GENRE: {
-			is_string = 1;
-			ret = __metadata_extractor_get_genre(_metadata, &s_value);
-			break;
-		}
-		case METADATA_AUTHOR: {
-			is_string = 1;
-			ret = __metadata_extractor_get_author(_metadata, &s_value);
-			break;
-		}
-		case METADATA_COPYRIGHT: {
-			is_string = 1;
-			ret = __metadata_extractor_get_copyright(_metadata, &s_value);
-			break;
-		}
-		case METADATA_DATE: {
-			is_string = 1;
-			ret = __metadata_extractor_get_date(_metadata, &s_value);
-			break;
-		}
-		case METADATA_DESCRIPTION: {
-			is_string = 1;
-			ret = __metadata_extractor_get_description(_metadata, &s_value);
-			break;
-		}
-		case METADATA_COMMENT: {
-			is_string = 1;
-			ret = __metadata_extractor_get_comment(_metadata, &s_value);
-			break;
-		}
-		case METADATA_TRACK_NUM: {
-			is_string = 1;
-			ret = __metadata_extractor_get_track_num(_metadata, &s_value);
-			break;
-		}
-		case METADATA_CLASSIFICATION: {
-			is_string = 1;
-			ret = __metadata_extractor_get_classification(_metadata, &s_value);
-			break;
-		}
-		case METADATA_RATING: {
-			is_string = 1;
-			ret = __metadata_extractor_get_rating(_metadata, &s_value);
-			break;
-		}
-		case METADATA_LONGITUDE: {
-			is_double = 1;
-			ret = __metadata_extractor_get_longitude(_metadata, &d_value);
-			break;
-		}
-		case METADATA_LATITUDE: {
-			is_double = 1;
-			ret = __metadata_extractor_get_latitude(_metadata, &d_value);
-			break;
-		}
-		case METADATA_ALTITUDE: {
-			is_double = 1;
-			ret = __metadata_extractor_get_altitude(_metadata, &d_value);
-			break;
-		}
-		case METADATA_CONDUCTOR: {
-			is_string = 1;
-			ret = __metadata_extractor_get_conductor(_metadata, &s_value);
-			break;
-		}
-		case METADATA_UNSYNCLYRICS: {
-			is_string = 1;
-			ret = __metadata_extractor_get_unsynclyrics(_metadata, &s_value);
-			break;
-		}
-		case METADATA_SYNCLYRICS_NUM: {
-			is_string = 0;
-			ret = __metadata_extractor_get_synclyrics_pair_num(_metadata, &i_value);
-			break;
-		}
-		case METADATA_RECDATE: {
-			is_string = 1;
-			ret = __metadata_extractor_get_recording_date(_metadata, &s_value);
-			break;
-		}
-		case METADATA_ROTATE: {
-			is_string = 1;
-			ret = __metadata_extractor_get_rotate(_metadata, &s_value);
-			break;
-		}
-		default:
-			break;
+	case METADATA_DURATION: {
+		is_string = 0;
+		ret = __metadata_extractor_get_duration(_metadata, &i_value);
+		break;
+	}
+	case METADATA_VIDEO_BITRATE: {
+		is_string = 0;
+		ret = __metadata_extractor_get_video_bitrate(_metadata, &i_value);
+		break;
+	}
+	case METADATA_VIDEO_FPS: {
+		is_string = 0;
+		ret = __metadata_extractor_get_video_FPS(_metadata, &i_value);
+		break;
+	}
+	case METADATA_VIDEO_WIDTH: {
+		is_string = 0;
+		ret = __metadata_extractor_get_video_width(_metadata, &i_value);
+		break;
+	}
+	case METADATA_VIDEO_HEIGHT: {
+		is_string = 0;
+		ret = __metadata_extractor_get_video_height(_metadata, &i_value);
+		break;
+	}
+	case METADATA_HAS_VIDEO: {
+		is_string = 0;
+		ret = __metadata_extractor_get_video_track_count(_metadata, &i_value);
+		break;
+	}
+	case METADATA_AUDIO_BITRATE: {
+		is_string = 0;
+		ret = __metadata_extractor_get_audio_bitrate(_metadata, &i_value);
+		break;
+	}
+	case METADATA_AUDIO_CHANNELS: {
+		is_string = 0;
+		ret = __metadata_extractor_get_audio_channel(_metadata, &i_value);
+		break;
+	}
+	case METADATA_AUDIO_SAMPLERATE: {
+		is_string = 0;
+		ret = __metadata_extractor_get_audio_samplerate(_metadata, &i_value);
+		break;
+	}
+	case METADATA_AUDIO_BITPERSAMPLE: {
+		is_string = 0;
+		ret = __metadata_extractor_get_audio_bitpersample(_metadata, &i_value);
+		break;
+	}
+	case METADATA_HAS_AUDIO: {
+		is_string = 0;
+		ret = __metadata_extractor_get_audio_track_count(_metadata, &i_value);
+		break;
+	}
+	case METADATA_ARTIST: {
+		is_string = 1;
+		ret = __metadata_extractor_get_artist(_metadata, &s_value);
+		break;
+	}
+	case METADATA_TITLE: {
+		is_string = 1;
+		ret = __metadata_extractor_get_title(_metadata, &s_value);
+		break;
+	}
+	case METADATA_ALBUM: {
+		is_string = 1;
+		ret = __metadata_extractor_get_album(_metadata, &s_value);
+		break;
+	}
+	case METADATA_ALBUM_ARTIST: {
+		is_string = 1;
+		ret = __metadata_extractor_get_album_artist(_metadata, &s_value);
+		break;
+	}
+	case METADATA_GENRE: {
+		is_string = 1;
+		ret = __metadata_extractor_get_genre(_metadata, &s_value);
+		break;
+	}
+	case METADATA_AUTHOR: {
+		is_string = 1;
+		ret = __metadata_extractor_get_author(_metadata, &s_value);
+		break;
+	}
+	case METADATA_COPYRIGHT: {
+		is_string = 1;
+		ret = __metadata_extractor_get_copyright(_metadata, &s_value);
+		break;
+	}
+	case METADATA_DATE: {
+		is_string = 1;
+		ret = __metadata_extractor_get_date(_metadata, &s_value);
+		break;
+	}
+	case METADATA_DESCRIPTION: {
+		is_string = 1;
+		ret = __metadata_extractor_get_description(_metadata, &s_value);
+		break;
+	}
+	case METADATA_COMMENT: {
+		is_string = 1;
+		ret = __metadata_extractor_get_comment(_metadata, &s_value);
+		break;
+	}
+	case METADATA_TRACK_NUM: {
+		is_string = 1;
+		ret = __metadata_extractor_get_track_num(_metadata, &s_value);
+		break;
+	}
+	case METADATA_CLASSIFICATION: {
+		is_string = 1;
+		ret = __metadata_extractor_get_classification(_metadata, &s_value);
+		break;
+	}
+	case METADATA_RATING: {
+		is_string = 1;
+		ret = __metadata_extractor_get_rating(_metadata, &s_value);
+		break;
+	}
+	case METADATA_LONGITUDE: {
+		is_double = 1;
+		ret = __metadata_extractor_get_longitude(_metadata, &d_value);
+		break;
+	}
+	case METADATA_LATITUDE: {
+		is_double = 1;
+		ret = __metadata_extractor_get_latitude(_metadata, &d_value);
+		break;
+	}
+	case METADATA_ALTITUDE: {
+		is_double = 1;
+		ret = __metadata_extractor_get_altitude(_metadata, &d_value);
+		break;
+	}
+	case METADATA_CONDUCTOR: {
+		is_string = 1;
+		ret = __metadata_extractor_get_conductor(_metadata, &s_value);
+		break;
+	}
+	case METADATA_UNSYNCLYRICS: {
+		is_string = 1;
+		ret = __metadata_extractor_get_unsynclyrics(_metadata, &s_value);
+		break;
+	}
+	case METADATA_SYNCLYRICS_NUM: {
+		is_string = 0;
+		ret = __metadata_extractor_get_synclyrics_pair_num(_metadata, &i_value);
+		break;
+	}
+	case METADATA_RECDATE: {
+		is_string = 1;
+		ret = __metadata_extractor_get_recording_date(_metadata, &s_value);
+		break;
+	}
+	case METADATA_ROTATE: {
+		is_string = 1;
+		ret = __metadata_extractor_get_rotate(_metadata, &s_value);
+		break;
+	}
+	default:
+		break;
 	}
 
 	if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
@@ -1399,11 +1390,10 @@ int metadata_extractor_get_metadata(metadata_extractor_h metadata, metadata_extr
 	} else {
 		char extracted_metadata[META_MAX_LEN] = {0, };
 
-		if (is_double) {
+		if (is_double)
 			snprintf(extracted_metadata, sizeof(extracted_metadata), "%f", d_value);
-		} else {
+		else
 			snprintf(extracted_metadata, sizeof(extracted_metadata), "%d", i_value);
-		}
 
 		*value = strdup(extracted_metadata);
 		if (*value == NULL) {
@@ -1429,14 +1419,12 @@ int metadata_extractor_get_artwork(metadata_extractor_h metadata, void **artwork
 	}
 
 	ret = __metadata_extractor_check_and_extract_meta(_metadata, METADATA_TYPE_TAG);
-	if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+	if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 		return ret;
-	}
 
 	ret = __metadata_extractor_get_artwork(_metadata, &_artwork, &_artwork_size);
-	if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+	if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 		return ret;
-	}
 
 	if ((_artwork_size > 0) && (_artwork != NULL)) {
 		ret = __metadata_extractor_get_artwork_mime(_metadata, &_artwork_mime);
@@ -1487,14 +1475,12 @@ int metadata_extractor_get_frame(metadata_extractor_h metadata, void **frame, in
 	}
 
 	ret = __metadata_extractor_check_and_extract_meta(_metadata, METADATA_TYPE_ATTR);
-	if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+	if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 		return ret;
-	}
 
 	ret = __metadata_extractor_get_video_thumbnail(_metadata, &_frame, &_frame_size);
-	if (ret != METADATA_EXTRACTOR_ERROR_NONE) {
+	if (ret != METADATA_EXTRACTOR_ERROR_NONE)
 		return ret;
-	}
 
 	if ((_frame_size > 0) && (_frame != NULL)) {
 		*frame = calloc(1, _frame_size);
@@ -1531,11 +1517,10 @@ int metadata_extractor_get_frame_at_time(metadata_extractor_h metadata, unsigned
 
 	metadata_extractor_debug("accurate [%d] timestamp[%d] micro_timestamp[%lld]", is_accurate, timestamp, micro_timestamp);
 
-	if (_metadata->path) {
+	if (_metadata->path)
 		ret = mm_file_get_video_frame(_metadata->path, micro_timestamp, is_accurate, (unsigned char **)&_frame, &_frame_size, &width, &height);
-	} else {
+	else
 		ret = mm_file_get_video_frame_from_memory(_metadata->buffer, _metadata->buffer_size, micro_timestamp, is_accurate, (unsigned char **)&_frame, &_frame_size, &width, &height);
-	}
 
 	if (ret != FILEINFO_ERROR_NONE) {
 		SAFE_FREE(_frame);
diff --git a/test/metadata_extractor_test.c b/test/metadata_extractor_test.c
index 140ae31..ffe52bb 100755
--- a/test/metadata_extractor_test.c
+++ b/test/metadata_extractor_test.c
@@ -46,9 +46,8 @@ static int _is_file_exist(const char *filename)
 	if (filename) {
 		const char *to_access = (strstr(filename, "file://") != NULL) ? filename + 7 : filename;
 		ret = access(to_access, R_OK);
-		if (ret != 0) {
+		if (ret != 0)
 			printf("file [%s] not found.\n", to_access);
-		}
 	}
 	return !ret;
 }