Improve readability of media_audio
[platform/core/api/media-content.git] / src / media_image.c
index cf344fc..16ad32a 100755 (executable)
 
 #include <media_info_private.h>
 
+#define media_content_retv_free_image_if(expr, val, p_str) do { \
+                               if (expr) {     \
+                                       LOGE(FONT_COLOR_RED"Memory allocation failure"FONT_COLOR_RESET);        \
+                                       image_meta_destroy(p_str);      \
+                                       return (val);   \
+                               }       \
+                       } while (0)
+
 
 int image_meta_destroy(image_meta_h image)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        image_meta_s *_image = (image_meta_s*)image;
+       media_content_retvm_if(_image == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "Image handle is null");
 
-       if (_image) {
-               SAFE_FREE(_image->media_id);
-               SAFE_FREE(_image->date_taken);
-               SAFE_FREE(_image->title);
-               SAFE_FREE(_image->weather);
-               SAFE_FREE(_image->burst_id);
-               SAFE_FREE(_image->exposure_time);
-               SAFE_FREE(_image->model);
-               SAFE_FREE(_image);
-
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       SAFE_FREE(_image->media_id);
+       SAFE_FREE(_image->date_taken);
+       SAFE_FREE(_image->exposure_time);
+       SAFE_FREE(_image->model);
+       SAFE_FREE(_image);
 
-       return ret;
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int image_meta_clone(image_meta_h *dst, image_meta_h src)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        image_meta_s *_src = (image_meta_s*)src;
+       media_content_retvm_if(_src == NULL, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "Source handle is null");
 
-       if (_src != NULL) {
-               image_meta_s *_dst = (image_meta_s*)calloc(1, sizeof(image_meta_s));
-               media_content_retvm_if(_dst == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, "OUT_OF_MEMORY");
-
-               if (STRING_VALID(_src->media_id)) {
-                       _dst->media_id = strdup(_src->media_id);
-                       if (_dst->media_id == NULL) {
-                               media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-                               image_meta_destroy((image_meta_h)_dst);
-                               return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-                       }
-               }
-
-               if (STRING_VALID(_src->date_taken)) {
-                       _dst->date_taken = strdup(_src->date_taken);
-                       if (_dst->date_taken == NULL) {
-                               media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-                               image_meta_destroy((image_meta_h)_dst);
-                               return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-                       }
-               }
-
-               if (STRING_VALID(_src->title)) {
-                       _dst->title = strdup(_src->title);
-                       if (_dst->title == NULL) {
-                               media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-                               image_meta_destroy((image_meta_h)_dst);
-                               return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-                       }
-               }
+       image_meta_s *_dst = (image_meta_s*)calloc(1, sizeof(image_meta_s));
+       media_content_retvm_if(_dst == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, "OUT_OF_MEMORY");
 
-               if (STRING_VALID(_src->weather)) {
-                       _dst->weather = strdup(_src->weather);
-                       if (_dst->weather == NULL) {
-                               media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-                               image_meta_destroy((image_meta_h)_dst);
-                               return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-                       }
-               }
+       if (STRING_VALID(_src->media_id)) {
+               _dst->media_id = strdup(_src->media_id);
+               media_content_retv_free_image_if(_dst->media_id == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, (image_meta_h)_dst);
+       }
 
-               if (STRING_VALID(_src->burst_id)) {
-                       _dst->burst_id = strdup(_src->burst_id);
-                       if (_dst->burst_id == NULL) {
-                               media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-                               image_meta_destroy((image_meta_h)_dst);
-                               return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-                       }
-               }
+       if (STRING_VALID(_src->date_taken)) {
+               _dst->date_taken = strdup(_src->date_taken);
+               media_content_retv_free_image_if(_dst->date_taken == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, (image_meta_h)_dst);
+       }
 
-               if (STRING_VALID(_src->exposure_time)) {
-                       _dst->exposure_time = strdup(_src->exposure_time);
-                       if (_dst->exposure_time == NULL) {
-                               media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-                               image_meta_destroy((image_meta_h)_dst);
-                               return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-                       }
-               }
+       if (STRING_VALID(_src->exposure_time)) {
+               _dst->exposure_time = strdup(_src->exposure_time);
+               media_content_retv_free_image_if(_dst->exposure_time == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, (image_meta_h)_dst);
+       }
 
-               if (STRING_VALID(_src->model)) {
-                       _dst->model = strdup(_src->model);
-                       if (_dst->model == NULL) {
-                               media_content_error("OUT_OF_MEMORY(0x%08x)", MEDIA_CONTENT_ERROR_OUT_OF_MEMORY);
-                               image_meta_destroy((image_meta_h)_dst);
-                               return MEDIA_CONTENT_ERROR_OUT_OF_MEMORY;
-                       }
-               }
+       if (STRING_VALID(_src->model)) {
+               _dst->model = strdup(_src->model);
+               media_content_retv_free_image_if(_dst->model == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, (image_meta_h)_dst);
+       }
 
-               _dst->fnumber = _src->fnumber;
-               _dst->iso = _src->iso;
-               _dst->width = _src->width;
-               _dst->height = _src->height;
-               _dst->orientation = _src->orientation;
+       _dst->fnumber = _src->fnumber;
+       _dst->iso = _src->iso;
+       _dst->width = _src->width;
+       _dst->height = _src->height;
+       _dst->orientation = _src->orientation;
 
-               *dst = (image_meta_h)_dst;
+       *dst = (image_meta_h)_dst;
 
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
-
-       return ret;
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int image_meta_get_media_id(image_meta_h image, char **media_id)
@@ -227,27 +175,6 @@ int image_meta_get_date_taken(image_meta_h image, char **date_taken)
        return ret;
 }
 
-int image_meta_get_burst_id(image_meta_h image, char **burst_id)
-{
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-       image_meta_s *_image = (image_meta_s*)image;
-
-       if (_image && burst_id) {
-               if (STRING_VALID(_image->burst_id)) {
-                       *burst_id = strdup(_image->burst_id);
-                       media_content_retvm_if(*burst_id == NULL, MEDIA_CONTENT_ERROR_OUT_OF_MEMORY, "OUT_OF_MEMORY");
-               } else {
-                       *burst_id = NULL;
-               }
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
-
-       return ret;
-}
-
 int image_meta_get_exposure_time(image_meta_h image, char **exposure_time)
 {
        int ret = MEDIA_CONTENT_ERROR_NONE;
@@ -322,29 +249,10 @@ int image_meta_get_model(image_meta_h image, char **model)
        return ret;
 }
 
-int image_meta_is_burst_shot(image_meta_h image, bool *is_burst_shot)
-{
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-       image_meta_s *_image = (image_meta_s*)image;
-
-       if (_image && is_burst_shot) {
-               if (STRING_VALID(_image->burst_id))
-                       *is_burst_shot = true;
-               else
-                       *is_burst_shot = false;
-
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               media_content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
-
-       return ret;
-}
-
 int image_meta_set_orientation(image_meta_h image, media_content_orientation_e orientation)
 {
        int ret = MEDIA_CONTENT_ERROR_NONE;
+       media_content_warn("DEPRECATION WARNING: image_meta_set_orientation() is deprecated and will be removed from next release.");
        image_meta_s *_image = (image_meta_s*)image;
 
        if (_image == NULL) {
@@ -365,17 +273,20 @@ int image_meta_set_orientation(image_meta_h image, media_content_orientation_e o
 int image_meta_update_to_db(image_meta_h image)
 {
        int ret = MEDIA_CONTENT_ERROR_NONE;
+       media_content_warn("DEPRECATION WARNING: image_meta_update_to_db() is deprecated and will be removed from next release.");
        image_meta_s *_image = (image_meta_s*)image;
        char *sql = NULL;
 
        if (_image != NULL && STRING_VALID(_image->media_id)) {
-               char storage_id[MEDIA_CONTENT_UUID_SIZE+1] = {0,};
-               memset(storage_id, 0x00, sizeof(storage_id));
-
-               ret = _media_db_get_storage_id_by_media_id(_image->media_id, storage_id);
+#ifdef _USE_TVPD_MODE
+               char *storage_id = NULL;
+               ret = _media_db_get_storage_id_by_media_id(_image->media_id, &storage_id);
                media_content_retv_if(ret != MEDIA_CONTENT_ERROR_NONE, ret);
-
-               sql = sqlite3_mprintf(UPDATE_IMAGE_META_FROM_MEDIA, storage_id, _image->orientation, _image->weather, _image->media_id);
+               sql = sqlite3_mprintf(UPDATE_IMAGE_META_FROM_MEDIA, storage_id, _image->orientation, _image->media_id);
+               SAFE_FREE(storage_id);
+#else
+               sql = sqlite3_mprintf(UPDATE_IMAGE_META_FROM_MEDIA, _image->orientation, _image->media_id);
+#endif
                ret = _content_query_sql(sql);
                SQLITE3_SAFE_FREE(sql);
        } else {
@@ -384,4 +295,4 @@ int image_meta_update_to_db(image_meta_h image)
        }
 
        return ret;
-}
\ No newline at end of file
+}