Replace error to MM_UTIL_ERROR_XXX 66/48866/3 accepted/tizen/mobile/20151001.224845 accepted/tizen/tv/20151001.224856 accepted/tizen/wearable/20151001.224909 submit/tizen/20151001.095659 tizen_3.0.m2.a1_mobile_release tizen_3.0.m2.a1_tv_release
authorJi Yong Min <jiyong.min@samsung.com>
Thu, 1 Oct 2015 05:34:54 +0000 (14:34 +0900)
committerJi Yong Min <jiyong.min@samsung.com>
Thu, 1 Oct 2015 07:28:05 +0000 (16:28 +0900)
Change-Id: Ia573cd065edac92c0165656168eb05a9e06d4e56

CMakeLists.txt
packaging/capi-media-image-util.spec
src/image_util.c

index 086820a..a0a8936 100755 (executable)
@@ -24,8 +24,8 @@ SET(service "media")
 SET(submodule "image-util")
 
 # for package file
-SET(dependents "dlog mmutil-jpeg mmutil-imgp capi-base-common capi-media-tool mm-common")
-SET(pc_dependents "dlog capi-base-common capi-media-tool mm-common")
+SET(dependents "dlog mmutil-jpeg mmutil-imgp capi-base-common capi-media-tool")
+SET(pc_dependents "dlog capi-base-common capi-media-tool")
 
 SET(fw_name "${project_prefix}-${service}-${submodule}")
 
index 659afa8..ca1b72b 100755 (executable)
@@ -1,13 +1,12 @@
 Name:       capi-media-image-util
 Summary:    A Image Utility library in Tizen Native API
-Version:    0.1.4
+Version:    0.1.5
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
 Source0:    %{name}-%{version}.tar.gz
 Source1001:     capi-media-image-util.manifest
 BuildRequires:  pkgconfig(dlog)
-BuildRequires:  pkgconfig(mm-common)
 BuildRequires:  pkgconfig(mmutil-jpeg)
 BuildRequires:  pkgconfig(mmutil-imgp)
 BuildRequires:  pkgconfig(capi-base-common)
index ab8a046..802a8cd 100755 (executable)
@@ -22,7 +22,6 @@
 #include <image_util_private.h>
 #include <stdio.h>
 #include <stdlib.h>
-#include <mm_error.h>
 
 #define IMAGE_UTIL_STRING_VALID(str)   \
        ((str != NULL && strlen(str) > 0) ? true : false)
@@ -79,36 +78,27 @@ static int _convert_image_util_error_code(const char *func, int code)
        int ret = IMAGE_UTIL_ERROR_INVALID_OPERATION;
        char *errorstr = NULL;
        switch (code) {
-               case MM_ERROR_NONE:
+               case MM_UTIL_ERROR_NONE:
                        ret = IMAGE_UTIL_ERROR_NONE;
                        errorstr = strdup("ERROR_NONE");
                        break;
-               case MM_ERROR_IMAGE_FILEOPEN:
-               case IMAGE_UTIL_ERROR_NO_SUCH_FILE:
+               case MM_UTIL_ERROR_NO_SUCH_FILE:
                        ret = IMAGE_UTIL_ERROR_NO_SUCH_FILE;
                        errorstr = strdup("NO_SUCH_FILE");
                        break;
-
-               case MM_ERROR_IMAGE_INTERNAL:
-                       ret = IMAGE_UTIL_ERROR_INVALID_OPERATION;
-                       errorstr = strdup("INVALID_OPERATION");
-                       break;
-               case IMAGE_UTIL_ERROR_INVALID_PARAMETER:
-               case MM_ERROR_IMAGEHANDLE_NOT_INITIALIZED:
-               case MM_ERROR_NO_DECODED_DATA:
-               case MM_ERROR_IMAGE_INVALID_VALUE:
+               case MM_UTIL_ERROR_INVALID_PARAMETER:
                        ret = IMAGE_UTIL_ERROR_INVALID_PARAMETER;
                        errorstr = strdup("INVALID_PARAMETER");
                        break;
-               case MM_ERROR_IMAGE_NOT_SUPPORT_FORMAT:
-               case MM_ERROR_IMAGE_DEVICE_NOT_SUPPORT:
+               case MM_UTIL_ERROR_NOT_SUPPORTED_FORMAT:
                        ret = IMAGE_UTIL_ERROR_NOT_SUPPORTED_FORMAT;
                        errorstr = strdup("NOT_SUPPORTED_FORMAT");
                        break;
-               case MM_ERROR_IMAGE_NO_FREE_SPACE:
+               case MM_UTIL_ERROR_OUT_OF_MEMORY:
                        ret = IMAGE_UTIL_ERROR_OUT_OF_MEMORY;
                        errorstr = strdup("OUT_OF_MEMORY");
                        break;
+               case MM_UTIL_ERROR_INVALID_OPERATION:
                default:
                        ret = IMAGE_UTIL_ERROR_INVALID_OPERATION;
                        errorstr = strdup("INVALID_OPERATION");
@@ -123,15 +113,20 @@ static int _convert_image_util_error_code(const char *func, int code)
 static image_util_error_e _image_util_error_convert(int error)
 {
        switch (error) {
-               case MM_ERROR_NONE:
+               case MM_UTIL_ERROR_NONE:
                        image_util_debug("Error None");
                        return IMAGE_UTIL_ERROR_NONE;
-               case MM_ERROR_IMAGE_INVALID_VALUE:
+               case MM_UTIL_ERROR_INVALID_PARAMETER:
                        image_util_error("INVALID_PARAMETER(0x%08x)", IMAGE_UTIL_ERROR_INVALID_PARAMETER);
                        return IMAGE_UTIL_ERROR_INVALID_PARAMETER;
-               case MM_ERROR_IMAGE_NOT_SUPPORT_FORMAT:
+               case MM_UTIL_ERROR_NOT_SUPPORTED_FORMAT:
                        image_util_error("NOT_SUPPORTED_FORMAT(0x%08x)", IMAGE_UTIL_ERROR_NOT_SUPPORTED_FORMAT);
                        return IMAGE_UTIL_ERROR_NOT_SUPPORTED_FORMAT;
+               case MM_UTIL_ERROR_OUT_OF_MEMORY:
+                       image_util_error("OUT_OF_MEMORY(0x%08x)", IMAGE_UTIL_ERROR_OUT_OF_MEMORY);
+                       return IMAGE_UTIL_ERROR_OUT_OF_MEMORY;
+                       break;
+               case MM_UTIL_ERROR_INVALID_OPERATION:
                default:
                        image_util_error("INVALID_OPERATION(0x%08x)", error);
                        return IMAGE_UTIL_ERROR_INVALID_OPERATION;
@@ -153,16 +148,18 @@ static void _image_util_transform_completed_cb(media_packet_h *dst, int error, v
 
 static int _image_util_create_transform_handle(transformation_s *handle)
 {
-       int ret = MM_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
        MMHandleType image_h;
 
-       ret = mm_util_create(&image_h);
-
-       image_util_retvm_if((ret != MM_ERROR_NONE), ret, "Error in mm_util_create");
+       err = mm_util_create(&image_h);
+       if (err != MM_UTIL_ERROR_NONE) {
+               image_util_error("Error - mm_util_create");
+               return _convert_image_util_error_code(__func__, err);
+       }
 
        handle->image_h = image_h;
 
-       return ret;
+       return _convert_image_util_error_code(__func__, err);
 }
 
 static bool _image_util_check_resolution(int width, int height)
@@ -184,8 +181,8 @@ int image_util_foreach_supported_jpeg_colorspace(image_util_supported_jpeg_color
 {
        int i = 0;
 
-       if (callback == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
+       image_util_retvm_if((callback == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "callback is null");
+
        /* RGB has higher precedence than YUV */
        for (i = sizeof(_convert_encode_colorspace_tbl) / sizeof(int) - 1; i >= 0; i--) {
                if (_convert_encode_colorspace_tbl[i] != -1)
@@ -199,7 +196,7 @@ int image_util_foreach_supported_jpeg_colorspace(image_util_supported_jpeg_color
 
 int image_util_transform_create(transformation_h *handle)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
 
        image_util_debug("image_util_create");
 
@@ -217,21 +214,21 @@ int image_util_transform_create(transformation_h *handle)
        _handle->set_rotate = false;
        _handle->set_crop  = false;
 
-       ret = _image_util_create_transform_handle(_handle);
-       if (ret != MM_ERROR_NONE) {
-               image_util_error("INVALID_OPERATION");
+       err = _image_util_create_transform_handle(_handle);
+       if (err != MM_UTIL_ERROR_NONE) {
+               image_util_error("Error - create transform handle");
                IMAGE_UTIL_SAFE_FREE(_handle);
-               return IMAGE_UTIL_ERROR_INVALID_OPERATION;
+               return _convert_image_util_error_code(__func__, err);
        }
 
        *handle = (transformation_h)_handle;
 
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int  image_util_transform_set_hardware_acceleration(transformation_h handle, bool mode)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
        transformation_s *_handle = (transformation_s *)handle;
 
        image_util_debug("Set hardware_acceleration %d", mode);
@@ -241,77 +238,85 @@ int  image_util_transform_set_hardware_acceleration(transformation_h handle, boo
        image_util_retvm_if((mode == true), IMAGE_UTIL_ERROR_NOT_SUPPORTED, "hardware acceleration is not supported");
 #endif
 
-       ret = mm_util_set_hardware_acceleration(_handle->image_h, mode);
-       image_util_retvm_if((ret != MM_ERROR_NONE), IMAGE_UTIL_ERROR_INVALID_OPERATION, "Error - Set hardware_acceleration");
+       err = mm_util_set_hardware_acceleration(_handle->image_h, mode);
+       if (err != MM_UTIL_ERROR_NONE) {
+               image_util_error("Error - Set hardware_acceleration");
+               return _convert_image_util_error_code(__func__, err);
+       }
 
        image_util_debug("Set hardware_acceleration %d", mode);
        _handle->hardware_acceleration = mode;
 
-       return IMAGE_UTIL_ERROR_NONE;
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_transform_set_colorspace(transformation_h handle, image_util_colorspace_e colorspace)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
        transformation_s *_handle = (transformation_s *)handle;
 
        image_util_debug("Set colorspace_convert_info [%d]", colorspace);
 
        image_util_retvm_if((_handle == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid Handle");
 
-       ret = mm_util_set_colorspace_convert(_handle->image_h, colorspace);
-       image_util_retvm_if((ret != MM_ERROR_NONE), IMAGE_UTIL_ERROR_INVALID_OPERATION, "IMAGE_UTIL_ERROR_INVALID_OPERATION(0x%08x)", IMAGE_UTIL_ERROR_INVALID_OPERATION);
+       err = mm_util_set_colorspace_convert(_handle->image_h, colorspace);
+       if (err != MM_UTIL_ERROR_NONE) {
+               image_util_error("Error - Set colorspace convert");
+               return _convert_image_util_error_code(__func__, err);
+       }
 
        _handle->colorspace = colorspace;
        _handle->set_convert = true;
 
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_transform_set_resolution(transformation_h handle, unsigned int width, unsigned int height)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
        transformation_s *_handle = (transformation_s *)handle;
 
        image_util_debug("Set resize_info w[%d] h[%d]", width, height);
 
        image_util_retvm_if((_handle == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid Handle");
        image_util_retvm_if((_handle->set_crop), IMAGE_UTIL_ERROR_INVALID_OPERATION, "Crop and Resize can't do at the same time");
+       image_util_retvm_if((_image_util_check_resolution(width, height) == false), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid resolution");
 
-       if (_image_util_check_resolution(width, height)) {
-               ret = mm_util_set_resolution(_handle->image_h, width, height);
-               image_util_retvm_if((ret != MM_ERROR_NONE), IMAGE_UTIL_ERROR_INVALID_OPERATION, "IMAGE_UTIL_ERROR_INVALID_OPERATION(0x%08x)", IMAGE_UTIL_ERROR_INVALID_OPERATION);
-               _handle->width = width;
-               _handle->height = height;
-               _handle->set_resize  = true;
-       } else {
-               image_util_error("INVALID_PARAMETER");
-               return IMAGE_UTIL_ERROR_INVALID_PARAMETER;
+       err = mm_util_set_resolution(_handle->image_h, width, height);
+       if (err != MM_UTIL_ERROR_NONE) {
+               image_util_error("Error - Set resolution");
+               return _convert_image_util_error_code(__func__, err);
        }
+       _handle->width = width;
+       _handle->height = height;
+       _handle->set_resize  = true;
 
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_transform_set_rotation(transformation_h handle, image_util_rotation_e rotation)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
        transformation_s *_handle = (transformation_s *)handle;
 
        image_util_debug("Set rotate_info [%d]", rotation);
 
        image_util_retvm_if((_handle == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid Handle");
 
-       ret = mm_util_set_rotation(_handle->image_h, rotation);
-       image_util_retvm_if((ret != MM_ERROR_NONE), IMAGE_UTIL_ERROR_INVALID_OPERATION, "IMAGE_UTIL_ERROR_INVALID_OPERATION(0x%08x)", IMAGE_UTIL_ERROR_INVALID_OPERATION);
+       err = mm_util_set_rotation(_handle->image_h, rotation);
+       if (err != MM_UTIL_ERROR_NONE) {
+               image_util_error("Error - Set rotation");
+               return _convert_image_util_error_code(__func__, err);
+       }
        _handle->rotation = rotation;
        _handle->set_rotate = true;
 
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_transform_set_crop_area(transformation_h handle, unsigned int start_x, unsigned int start_y, unsigned int end_x, unsigned int end_y)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
        transformation_s *_handle = (transformation_s *)handle;
        int dest_width;
        int dest_height;
@@ -324,20 +329,19 @@ int image_util_transform_set_crop_area(transformation_h handle, unsigned int sta
 
        image_util_debug("Set crop_info x[%d] y[%d] w[%d] h[%d]", start_x, start_y, dest_width, dest_height);
 
-       if (_image_util_check_resolution(dest_width, dest_height)) {
-               ret = mm_util_set_crop_area(_handle->image_h, start_x, start_y, end_x, end_y);
-               image_util_retvm_if((ret != MM_ERROR_NONE), IMAGE_UTIL_ERROR_INVALID_OPERATION, "IMAGE_UTIL_ERROR_INVALID_OPERATION(0x%08x)", IMAGE_UTIL_ERROR_INVALID_OPERATION);
-               _handle->start_x = start_x;
-               _handle->start_y = start_y;
-               _handle->end_x = end_x;
-               _handle->end_y = end_y;
-               _handle->set_crop  = true;
-       } else {
-               image_util_error("INVALID_PARAMETER");
-               return IMAGE_UTIL_ERROR_INVALID_PARAMETER;
+       image_util_retvm_if((_image_util_check_resolution(dest_width, dest_height) == false), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid dest resolution");
+       err = mm_util_set_crop_area(_handle->image_h, start_x, start_y, end_x, end_y);
+       if (err != MM_UTIL_ERROR_NONE) {
+               image_util_error("Error - Set crop area");
+               return _convert_image_util_error_code(__func__, err);
        }
+       _handle->start_x = start_x;
+       _handle->start_y = start_y;
+       _handle->end_x = end_x;
+       _handle->end_y = end_y;
+       _handle->set_crop  = true;
 
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_transform_get_colorspace(transformation_h handle, image_util_colorspace_e *colorspace)
@@ -451,40 +455,35 @@ int image_util_transform_get_crop_area(transformation_h handle, unsigned int *st
 
 int image_util_transform_run(transformation_h handle, media_packet_h src, image_util_transform_completed_cb completed_cb, void *user_data)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
        transformation_s *_handle = (transformation_s *)handle;
 
        image_util_debug("image_util_transform");
 
        image_util_retvm_if((_handle == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid Handle");
+       image_util_retvm_if((completed_cb == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid callback");
+       image_util_retvm_if((src == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid source");
+       image_util_retvm_if((!_handle->set_convert && !_handle->set_resize && !_handle->set_rotate && !_handle->set_crop), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid transform");
 
-       if (completed_cb) {
-               if (_handle->_util_cb != NULL) {
-                       IMAGE_UTIL_SAFE_FREE(_handle->_util_cb);
-                       _handle->_util_cb = NULL;
-               }
-               _handle->_util_cb = (image_util_cb_s *)calloc(1, sizeof(image_util_cb_s));
-               image_util_retvm_if((_handle->_util_cb == NULL), IMAGE_UTIL_ERROR_OUT_OF_MEMORY, "Out of memory");
-
-               _handle->_util_cb->user_data = user_data;
-               _handle->_util_cb->image_processing_completed_cb = completed_cb;
-       } else {
-               image_util_error("INVALID_PARAMETER[completed_cb](0x%08x)", IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-               return IMAGE_UTIL_ERROR_INVALID_PARAMETER;
+       if (_handle->_util_cb != NULL) {
+               IMAGE_UTIL_SAFE_FREE(_handle->_util_cb);
+               _handle->_util_cb = NULL;
        }
+       _handle->_util_cb = (image_util_cb_s *)calloc(1, sizeof(image_util_cb_s));
+       image_util_retvm_if((_handle->_util_cb == NULL), IMAGE_UTIL_ERROR_OUT_OF_MEMORY, "Out of memory");
 
-       if (_handle->_util_cb && src && (_handle->set_convert || _handle->set_resize || _handle->set_rotate || _handle->set_crop)) {
-               ret = mm_util_transform(_handle->image_h, src, (mm_util_completed_callback)_image_util_transform_completed_cb, (void *)_handle->_util_cb);
-       } else {
-               image_util_error("INVALID_PARAMETER[transform] (0x%08x)", IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-               return IMAGE_UTIL_ERROR_INVALID_PARAMETER;
+       _handle->_util_cb->user_data = user_data;
+       _handle->_util_cb->image_processing_completed_cb = completed_cb;
+
+       if (_handle->_util_cb) {
+               err = mm_util_transform(_handle->image_h, src, (mm_util_completed_callback)_image_util_transform_completed_cb, (void *)_handle->_util_cb);
        }
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_transform_destroy(transformation_h handle)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
        transformation_s *_handle = (transformation_s *)handle;
 
        image_util_debug("image_util_destroy");
@@ -494,268 +493,244 @@ int image_util_transform_destroy(transformation_h handle)
                return IMAGE_UTIL_ERROR_INVALID_PARAMETER;
        }
 
-       ret = mm_util_destroy(_handle->image_h);
+       err = mm_util_destroy(_handle->image_h);
 
        IMAGE_UTIL_SAFE_FREE(_handle->_util_cb);
        IMAGE_UTIL_SAFE_FREE(_handle);
 
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_convert_colorspace(unsigned char *dest, image_util_colorspace_e dest_colorspace, const unsigned char *src,  int width, int height, image_util_colorspace_e src_colorspace)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
-
-       if (dest == NULL || src == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (dest_colorspace < 0 || dest_colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (src_colorspace < 0 || src_colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
+       int err = MM_UTIL_ERROR_NONE;
 
+       image_util_retvm_if((dest == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "dest is null");
+       image_util_retvm_if((src == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "src is null");
+       image_util_retvm_if((dest_colorspace < 0 || dest_colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid dst_colorspace");
+       image_util_retvm_if((src_colorspace < 0 || src_colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid src_colorspace");
 
-       ret = mm_util_convert_colorspace(src, width, height, _convert_colorspace_tbl[src_colorspace], dest, _convert_colorspace_tbl[dest_colorspace]);
+       err = mm_util_convert_colorspace(src, width, height, _convert_colorspace_tbl[src_colorspace], dest, _convert_colorspace_tbl[dest_colorspace]);
 
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 
 int image_util_calculate_buffer_size(int width, int height, image_util_colorspace_e colorspace , unsigned int *size)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
 
-       if (colorspace < 0 || width <= 0 || height <= 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int) || size == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
+       image_util_retvm_if((colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid colorspace");
+       image_util_retvm_if((width <= 0 || height <= 0), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid width or Invalid height");
+       image_util_retvm_if((size == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "size is null");
 
-       ret = mm_util_get_image_size(_convert_colorspace_tbl[colorspace], width, height, size);
-       return _convert_image_util_error_code(__func__, ret);
+       err = mm_util_get_image_size(_convert_colorspace_tbl[colorspace], width, height, size);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_resize(unsigned char *dest, int *dest_width, int *dest_height, const unsigned char *src, int src_width, int src_height, image_util_colorspace_e colorspace)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
-
-       if (dest == NULL || src == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (dest_width == NULL || dest_height == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
+       int err = MM_UTIL_ERROR_NONE;
 
-       if (*dest_width <= 0 || *dest_height <= 0)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
+       image_util_retvm_if((dest == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "dest is null");
+       image_util_retvm_if((src == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "src is null");
+       image_util_retvm_if((colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid colorspace");
+       image_util_retvm_if((dest_width == NULL || dest_height == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "dest_width or dest_height is null");
+       image_util_retvm_if((*dest_width <= 0 || *dest_height <= 0), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid dest_width or Invalid dest_height");
 
        unsigned int dest_w, dest_h;
        dest_w = *dest_width;
        dest_h = *dest_height;
-       ret = mm_util_resize_image(src, src_width, src_height, _convert_colorspace_tbl[colorspace], dest, &dest_w, &dest_h);
-       if (ret == 0) {
-               *dest_width = dest_w;
-               *dest_height = dest_h;
+       err = mm_util_resize_image(src, src_width, src_height, _convert_colorspace_tbl[colorspace], dest, &dest_w, &dest_h);
+       if (err == MM_UTIL_ERROR_NONE) {
+               *dest_width = (int)dest_w;
+               *dest_height = (int)dest_h;
        }
 
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_rotate(unsigned char *dest, int *dest_width, int *dest_height, image_util_rotation_e dest_rotation, const unsigned char *src, int src_width, int src_height, image_util_colorspace_e colorspace)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
 
-       if (dest == NULL || src == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (dest_rotation < 0 || dest_rotation > IMAGE_UTIL_ROTATION_FLIP_VERT)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (dest_width == NULL || dest_height == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
+       image_util_retvm_if((dest == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "dest is null");
+       image_util_retvm_if((src == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "src is null");
+       image_util_retvm_if((colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid colorspace");
+       image_util_retvm_if((dest_rotation < 0 || dest_rotation > IMAGE_UTIL_ROTATION_FLIP_VERT), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid rotation");
+       image_util_retvm_if((dest_width == NULL || dest_height == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "dest_width or dest_height is null");
 
        unsigned int dest_w, dest_h;
-       ret = mm_util_rotate_image(src, src_width, src_height, _convert_colorspace_tbl[colorspace], dest, &dest_w, &dest_h, dest_rotation);
-       if (ret == 0) {
-               *dest_width = dest_w;
-               *dest_height = dest_h;
+       err = mm_util_rotate_image(src, src_width, src_height, _convert_colorspace_tbl[colorspace], dest, &dest_w, &dest_h, dest_rotation);
+       if (err == MM_UTIL_ERROR_NONE) {
+               *dest_width = (int)dest_w;
+               *dest_height = (int)dest_h;
        }
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_crop(unsigned char *dest, int x, int y, int *width, int *height, const unsigned char *src, int src_width, int src_height, image_util_colorspace_e colorspace)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
 
-       if (dest == NULL || src == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (width == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (src_width <= x  || src_height <= y || src_width < x + *width || src_height < y + *height)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
+       image_util_retvm_if((dest == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "dest is null");
+       image_util_retvm_if((src == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "src is null");
+       image_util_retvm_if((colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid colorspace");
+       image_util_retvm_if((width == NULL || height == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "width or height is null");
+       image_util_retvm_if((src_width <= x  || src_height <= y || src_width < x + *width || src_height < y + *height), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid crop area");
 
        unsigned int dest_w, dest_h;
        dest_w = *width;
        dest_h = *height;
-       ret = mm_util_crop_image(src, src_width, src_height, _convert_colorspace_tbl[colorspace], x, y, &dest_w, &dest_h, dest);
-       if (ret == 0) {
-               *width = dest_w;
-               *height = dest_h;
+       err = mm_util_crop_image(src, src_width, src_height, _convert_colorspace_tbl[colorspace], x, y, &dest_w, &dest_h, dest);
+       if (err == MM_UTIL_ERROR_NONE) {
+               *width = (int)dest_w;
+               *height = (int)dest_h;
        }
 
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_decode_jpeg(const char *path, image_util_colorspace_e colorspace, unsigned char **image_buffer, int *width, int *height, unsigned int *size)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
 
-       if (path == NULL || image_buffer == NULL || size == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (strlen(path) == 0)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_NO_SUCH_FILE);
-       if (colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (_convert_encode_colorspace_tbl[colorspace] == -1)
-               return _convert_image_util_error_code(__func__, MM_ERROR_IMAGE_NOT_SUPPORT_FORMAT);
+       image_util_retvm_if((path == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "path is null");
+       image_util_retvm_if((image_buffer == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "image_buffer is null");
+       image_util_retvm_if((size == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "size is null");
+       image_util_retvm_if((strlen(path) == 0), IMAGE_UTIL_ERROR_NO_SUCH_FILE, "Invalid path");
+       image_util_retvm_if((colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid colorspace");
+       image_util_retvm_if((_convert_encode_colorspace_tbl[colorspace] == -1), IMAGE_UTIL_ERROR_NOT_SUPPORTED_FORMAT, "not supported format");
 
        mm_util_jpeg_yuv_data decoded;
        memset(&decoded, 0, sizeof(mm_util_jpeg_yuv_data));
 
-       ret = mm_util_decode_from_jpeg_file(&decoded, path, _convert_encode_colorspace_tbl[colorspace]);
-       if (ret == 0) {
+       err = mm_util_decode_from_jpeg_file(&decoded, path, _convert_encode_colorspace_tbl[colorspace]);
+       if (err == MM_UTIL_ERROR_NONE) {
                *image_buffer = decoded.data;
                if (width)
                        *width = decoded.width;
                if (height)
                        *height = decoded.height;
                if (size)
-                       *size = decoded.size;
+                       *size = (unsigned int)decoded.size;
        }
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_decode_jpeg_from_memory(const unsigned char *jpeg_buffer, int jpeg_size, image_util_colorspace_e colorspace, unsigned char **image_buffer, int *width, int *height, unsigned int *size)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
 
-       if (jpeg_buffer == NULL || image_buffer == NULL || size == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (_convert_encode_colorspace_tbl[colorspace] == -1)
-               return _convert_image_util_error_code(__func__, MM_ERROR_IMAGE_NOT_SUPPORT_FORMAT);
+       image_util_retvm_if((jpeg_buffer == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "jpeg_buffer is null");
+       image_util_retvm_if((image_buffer == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "image_buffer is null");
+       image_util_retvm_if((size == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "size is null");
+       image_util_retvm_if((colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid colorspace");
+       image_util_retvm_if((_convert_encode_colorspace_tbl[colorspace] == -1), IMAGE_UTIL_ERROR_NOT_SUPPORTED_FORMAT, "not supported format");
 
        mm_util_jpeg_yuv_data decoded;
        memset(&decoded, 0, sizeof(mm_util_jpeg_yuv_data));
 
-       ret = mm_util_decode_from_jpeg_memory(&decoded, (void *)jpeg_buffer, jpeg_size, _convert_encode_colorspace_tbl[colorspace]);
+       err = mm_util_decode_from_jpeg_memory(&decoded, (void *)jpeg_buffer, jpeg_size, _convert_encode_colorspace_tbl[colorspace]);
 
-       if (ret == 0) {
+       if (err == MM_UTIL_ERROR_NONE) {
                *image_buffer = decoded.data;
                if (width)
                        *width = decoded.width;
                if (height)
                        *height = decoded.height;
                if (size)
-                       *size = decoded.size;
+                       *size = (unsigned int)decoded.size;
        }
 
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_decode_jpeg_with_downscale(const char *path, image_util_colorspace_e colorspace, image_util_scale_e downscale, unsigned char **image_buffer, int *width, int *height, unsigned int *size)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
 
-       if (path == NULL || image_buffer == NULL || size == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (strlen(path) == 0)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_NO_SUCH_FILE);
-       if (colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (_convert_encode_colorspace_tbl[colorspace] == -1)
-               return _convert_image_util_error_code(__func__, MM_ERROR_IMAGE_NOT_SUPPORT_FORMAT);
-       if (downscale < 0 || downscale >= sizeof(_convert_decode_scale_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
+       image_util_retvm_if((path == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "path is null");
+       image_util_retvm_if((image_buffer == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "image_buffer is null");
+       image_util_retvm_if((size == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "size is null");
+       image_util_retvm_if((strlen(path) == 0), IMAGE_UTIL_ERROR_NO_SUCH_FILE, "Invalid path");
+       image_util_retvm_if((colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid colorspace");
+       image_util_retvm_if((_convert_encode_colorspace_tbl[colorspace] == -1), IMAGE_UTIL_ERROR_NOT_SUPPORTED_FORMAT, "not supported format");
+       image_util_retvm_if((downscale < 0 || downscale >= sizeof(_convert_decode_scale_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "downsacle is invalid");
 
        mm_util_jpeg_yuv_data decoded;
        memset(&decoded, 0, sizeof(mm_util_jpeg_yuv_data));
 
-       ret = mm_util_decode_from_jpeg_file_with_downscale(&decoded, path, _convert_encode_colorspace_tbl[colorspace], _convert_decode_scale_tbl[downscale]);
-       if (ret == 0) {
+       err = mm_util_decode_from_jpeg_file_with_downscale(&decoded, path, _convert_encode_colorspace_tbl[colorspace], _convert_decode_scale_tbl[downscale]);
+       if (err == MM_UTIL_ERROR_NONE) {
                *image_buffer = decoded.data;
                if (width)
                        *width = decoded.width;
                if (height)
                        *height = decoded.height;
                if (size)
-                       *size = decoded.size;
+                       *size = (unsigned int)decoded.size;
        }
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_decode_jpeg_from_memory_with_downscale(const unsigned char *jpeg_buffer, int jpeg_size, image_util_colorspace_e colorspace, image_util_scale_e downscale, unsigned char **image_buffer, int *width, int *height, unsigned int *size)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
 
-       if (jpeg_buffer == NULL || image_buffer == NULL || size == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (_convert_encode_colorspace_tbl[colorspace] == -1)
-               return _convert_image_util_error_code(__func__, MM_ERROR_IMAGE_NOT_SUPPORT_FORMAT);
-       if (downscale < 0 || downscale >= sizeof(_convert_decode_scale_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
+       image_util_retvm_if((jpeg_buffer == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "jpeg_buffer is null");
+       image_util_retvm_if((image_buffer == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "image_buffer is null");
+       image_util_retvm_if((size == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "size is null");
+       image_util_retvm_if((colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid colorspace");
+       image_util_retvm_if((_convert_encode_colorspace_tbl[colorspace] == -1), IMAGE_UTIL_ERROR_NOT_SUPPORTED_FORMAT, "not supported format");
+       image_util_retvm_if((downscale < 0 || downscale >= sizeof(_convert_decode_scale_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "downsacle is invalid");
 
        mm_util_jpeg_yuv_data decoded;
        memset(&decoded, 0, sizeof(mm_util_jpeg_yuv_data));
 
-       ret = mm_util_decode_from_jpeg_memory_with_downscale(&decoded, (void *)jpeg_buffer, jpeg_size, _convert_encode_colorspace_tbl[colorspace], _convert_decode_scale_tbl[downscale]);
+       err = mm_util_decode_from_jpeg_memory_with_downscale(&decoded, (void *)jpeg_buffer, jpeg_size, _convert_encode_colorspace_tbl[colorspace], _convert_decode_scale_tbl[downscale]);
 
-       if (ret == 0) {
+       if (err == MM_UTIL_ERROR_NONE) {
                *image_buffer = decoded.data;
                if (width)
                        *width = decoded.width;
                if (height)
                        *height = decoded.height;
                if (size)
-                       *size = decoded.size;
+                       *size = (unsigned int)decoded.size;
        }
 
-       return _convert_image_util_error_code(__func__, ret);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_encode_jpeg(const unsigned char *buffer, int width, int height, image_util_colorspace_e colorspace,  int quality, const char *path)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
+       int err = MM_UTIL_ERROR_NONE;
+
+       image_util_retvm_if((path == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "path is null");
+       image_util_retvm_if((buffer == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "buffer is null");
+       image_util_retvm_if((strlen(path) == 0), IMAGE_UTIL_ERROR_NO_SUCH_FILE, "Invalid path");
+       image_util_retvm_if((colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid colorspace");
+       image_util_retvm_if((_convert_encode_colorspace_tbl[colorspace] == -1), IMAGE_UTIL_ERROR_NOT_SUPPORTED_FORMAT, "not supported format");
 
-       if (path == NULL || buffer == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (strlen(path) == 0)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_NO_SUCH_FILE);
-       if (colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (_convert_encode_colorspace_tbl[colorspace] == -1)
-               return _convert_image_util_error_code(__func__, MM_ERROR_IMAGE_NOT_SUPPORT_FORMAT);
-
-       ret = mm_util_jpeg_encode_to_file(path, (void *)buffer, width, height, _convert_encode_colorspace_tbl[colorspace], quality);
-       return _convert_image_util_error_code(__func__, ret);
+       err = mm_util_jpeg_encode_to_file(path, (void *)buffer, width, height, _convert_encode_colorspace_tbl[colorspace], quality);
+       return _convert_image_util_error_code(__func__, err);
 }
 
 int image_util_encode_jpeg_to_memory(const unsigned char *image_buffer, int width, int height, image_util_colorspace_e colorspace, int quality,  unsigned char **jpeg_buffer, unsigned int *jpeg_size)
 {
-       int ret = IMAGE_UTIL_ERROR_NONE;
-       int isize;
+       int err = MM_UTIL_ERROR_NONE;
+       int isize = 0;
 
-       if (jpeg_buffer == NULL || image_buffer == NULL || jpeg_size == NULL)
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int))
-               return _convert_image_util_error_code(__func__, IMAGE_UTIL_ERROR_INVALID_PARAMETER);
-       if (_convert_encode_colorspace_tbl[colorspace] == -1)
-               return _convert_image_util_error_code(__func__, MM_ERROR_IMAGE_NOT_SUPPORT_FORMAT);
+       image_util_retvm_if((jpeg_buffer == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "jpeg_buffer is null");
+       image_util_retvm_if((image_buffer == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "image_buffer is null");
+       image_util_retvm_if((jpeg_size == NULL), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "jpeg_size is null");
+       image_util_retvm_if((colorspace < 0 || colorspace >= sizeof(_convert_colorspace_tbl) / sizeof(int)), IMAGE_UTIL_ERROR_INVALID_PARAMETER, "Invalid colorspace");
+       image_util_retvm_if((_convert_encode_colorspace_tbl[colorspace] == -1), IMAGE_UTIL_ERROR_NOT_SUPPORTED_FORMAT, "not supported format");
 
-       ret = mm_util_jpeg_encode_to_memory((void **)jpeg_buffer, &isize, (void *)image_buffer, width, height, _convert_encode_colorspace_tbl[colorspace], quality);
-       if (ret == 0)
-               *jpeg_size = isize;
+       err = mm_util_jpeg_encode_to_memory((void **)jpeg_buffer, &isize, (void *)image_buffer, width, height, _convert_encode_colorspace_tbl[colorspace], quality);
+       if (err == MM_UTIL_ERROR_NONE)
+               *jpeg_size = (unsigned int)isize;
 
-       return _convert_image_util_error_code(__func__, ret);
-}
\ No newline at end of file
+       return _convert_image_util_error_code(__func__, err);
+}