diff options
author | Haejeong Kim <backto.kim@samsung.com> | 2016-01-26 15:17:50 +0900 |
---|---|---|
committer | Haejeong Kim <backto.kim@samsung.com> | 2016-01-26 15:18:07 +0900 |
commit | 787685c66f7e58a92fbadb72f3d30ab5a240b2cd (patch) | |
tree | ad9d6790bb67ad14fda56f43f4c2ed695ec58aba | |
parent | 596a5838b342f20423aae12beed2d845e4692c04 (diff) | |
download | metadata-extractor-787685c66f7e58a92fbadb72f3d30ab5a240b2cd.tar.gz metadata-extractor-787685c66f7e58a92fbadb72f3d30ab5a240b2cd.tar.bz2 metadata-extractor-787685c66f7e58a92fbadb72f3d30ab5a240b2cd.zip |
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
-rwxr-xr-x | src/metadata_extractor.c | 387 | ||||
-rwxr-xr-x | test/metadata_extractor_test.c | 3 |
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; } |