Apply tizen coding rule

Change-Id: I9204f8b2072316eceba2bce21dd01a8f71b5eed3

diff --git a/src/metadata_extractor.c b/src/metadata_extractor.c
index c08e706..83ab272 100755 (executable)
--- 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 (executable)
--- 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;
 }