apply content_retip_if_fail() for invalid parameters
[platform/core/api/media-content.git] / src / media_pvr.c
index fee2cfb..5125b31 100755 (executable)
 
 int media_pvr_get_media_count_from_db(filter_h filter, int *media_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
-
-       if (media_count) {
-               ret = _media_db_get_group_item_count(NULL, filter, MEDIA_GROUP_PVR, media_count);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(media_count);
 
-       return ret;
+       return _media_db_get_group_item_count(NULL, filter, MEDIA_GROUP_PVR, media_count);
 }
 
 int media_pvr_foreach_media_from_db(filter_h filter, media_pvr_cb callback, void *user_data)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
+       content_retip_if_fail(callback);
 
-       if (callback != NULL) {
-               ret = _media_db_get_pvr(filter, callback, user_data);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
-
-       return ret;
+       return _media_db_get_pvr(filter, callback, user_data);
 }
 
 int media_pvr_destroy(media_pvr_h pvr)
 {
        int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
-       if (_pvr) {
-               g_free(_pvr->media_id);
-               g_free(_pvr->storage_id);
-               g_free(_pvr->channel_name);
-               g_free(_pvr->channel_num);
-               g_free(_pvr->program_title);
-               g_free(_pvr->program_crid);
-               g_free(_pvr->guidance);
-               g_free(_pvr->synopsis);
-               g_free(_pvr->genre);
-               g_free(_pvr->language);
-               g_free(_pvr->path);
-               g_free(_pvr->modified_month);
-               g_free(_pvr->private_data);
-               g_free(_pvr);
-
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
 
-       return ret;
+       content_retip_if_fail(pvr);
+
+       g_free(_pvr->media_id);
+       g_free(_pvr->storage_id);
+       g_free(_pvr->channel_name);
+       g_free(_pvr->channel_num);
+       g_free(_pvr->program_title);
+       g_free(_pvr->program_crid);
+       g_free(_pvr->guidance);
+       g_free(_pvr->synopsis);
+       g_free(_pvr->genre);
+       g_free(_pvr->language);
+       g_free(_pvr->path);
+       g_free(_pvr->modified_month);
+       g_free(_pvr->private_data);
+       g_free(_pvr);
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_clone(media_pvr_h *dst, media_pvr_h src)
@@ -145,10 +127,8 @@ int media_pvr_get_pvr_from_db(const char *media_id, media_pvr_h *pvr)
        char *select_query = NULL;
        sqlite3_stmt *stmt = NULL;
 
-       if (!STRING_VALID(media_id) || (pvr == NULL)) {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               return MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(STRING_VALID(media_id));
+       content_retip_if_fail(pvr);
 
        select_query = sqlite3_mprintf(SELECT_PVR_FROM_PVR, media_id);
 
@@ -308,482 +288,362 @@ int media_pvr_get_storage_id(media_pvr_h pvr, char **storage_id)
 
 int media_pvr_get_size(media_pvr_h pvr, unsigned long long *size)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && size) {
-               *size = _pvr->size;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(size);
 
-       return ret;
+       *size = _pvr->size;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_timezone(media_pvr_h pvr, int *timezone)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && timezone) {
-               *timezone = _pvr->timezone;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(timezone);
 
-       return ret;
+       *timezone = _pvr->timezone;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_ptc(media_pvr_h pvr, int *ptc)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && ptc) {
-               *ptc = _pvr->ptc;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(ptc);
 
-       return ret;
+       *ptc = _pvr->ptc;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_major(media_pvr_h pvr, int *major)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && major) {
-               *major = _pvr->major;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(major);
 
-       return ret;
+       *major = _pvr->major;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_minor(media_pvr_h pvr, int *minor)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && minor) {
-               *minor = _pvr->minor;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(minor);
 
-       return ret;
+       *minor = _pvr->minor;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_channel_type(media_pvr_h pvr, int *channel_type)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && channel_type) {
-               *channel_type = _pvr->channel_type;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(channel_type);
 
-       return ret;
+       *channel_type = _pvr->channel_type;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_program_num(media_pvr_h pvr, int *program_num)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && program_num) {
-               *program_num = _pvr->program_num;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(program_num);
 
-       return ret;
+       *program_num = _pvr->program_num;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_service_profile(media_pvr_h pvr, unsigned int *service_profile)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && service_profile) {
-               *service_profile = _pvr->service_profile;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(service_profile);
 
-       return ret;
+       *service_profile = _pvr->service_profile;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_duration(media_pvr_h pvr, int *duration)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && duration) {
-               *duration = _pvr->duration;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(duration);
 
-       return ret;
+       *duration = _pvr->duration;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_embargo_time(media_pvr_h pvr, int *embargo_time)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && embargo_time) {
-               *embargo_time = _pvr->embargo_time;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(embargo_time);
 
-       return ret;
+       *embargo_time = _pvr->embargo_time;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_expiry_time(media_pvr_h pvr, int *expiry_time)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && expiry_time) {
-               *expiry_time = _pvr->expiry_time;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(expiry_time);
 
-       return ret;
+       *expiry_time = _pvr->expiry_time;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_parental_rating(media_pvr_h pvr, int *parental_rating)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && parental_rating) {
-               *parental_rating = _pvr->parental_rating;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(parental_rating);
 
-       return ret;
+       *parental_rating = _pvr->parental_rating;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_start_time(media_pvr_h pvr, int *start_time)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && start_time) {
-               *start_time = _pvr->start_time;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(start_time);
 
-       return ret;
+       *start_time = _pvr->start_time;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_program_start_time(media_pvr_h pvr, int *program_start_time)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && program_start_time) {
-               *program_start_time = _pvr->program_start_time;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(program_start_time);
 
-       return ret;
+       *program_start_time = _pvr->program_start_time;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_program_end_time(media_pvr_h pvr, int *end_time)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && end_time) {
-               *end_time = _pvr->program_end_time;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(end_time);
 
-       return ret;
+       *end_time = _pvr->program_end_time;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_program_date(media_pvr_h pvr, int *program_date)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && program_date) {
-               *program_date = _pvr->program_date;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(program_date);
 
-       return ret;
+       *program_date = _pvr->program_date;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_timer_record(media_pvr_h pvr, bool* timer_record)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && timer_record) {
-               *timer_record = _pvr->timer_record;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(timer_record);
 
-       return ret;
+       *timer_record = _pvr->timer_record;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_series_record(media_pvr_h pvr, bool* series_record)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && series_record) {
-               *series_record = _pvr->series_record;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(series_record);
 
-       return ret;
+       *series_record = _pvr->series_record;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_hd(media_pvr_h pvr, int* hd)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && hd) {
-               *hd = _pvr->hd;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(hd);
 
-       return ret;
+       *hd = _pvr->hd;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_subtitle(media_pvr_h pvr, bool* subtitle)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && subtitle) {
-               *subtitle = _pvr->subtitle;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(subtitle);
 
-       return ret;
+       *subtitle = _pvr->subtitle;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_ttx(media_pvr_h pvr, bool* ttx)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && ttx) {
-               *ttx = _pvr->ttx;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(ttx);
 
-       return ret;
+       *ttx = _pvr->ttx;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_ad(media_pvr_h pvr, bool* ad)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && ad) {
-               *ad = _pvr->ad;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(ad);
 
-       return ret;
+       *ad = _pvr->ad;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_hard_of_hearing_radio(media_pvr_h pvr, bool* hard_of_hearing_radio)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && hard_of_hearing_radio) {
-               *hard_of_hearing_radio = _pvr->hard_of_hearing_radio;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(hard_of_hearing_radio);
 
-       return ret;
+       *hard_of_hearing_radio = _pvr->hard_of_hearing_radio;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_data_service(media_pvr_h pvr, bool* data_service)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && data_service) {
-               *data_service = _pvr->data_service;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(data_service);
 
-       return ret;
+       *data_service = _pvr->data_service;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_content_lock(media_pvr_h pvr, bool* content_lock)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && content_lock) {
-               *content_lock = _pvr->content_lock;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(content_lock);
 
-       return ret;
+       *content_lock = _pvr->content_lock;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_content_watch(media_pvr_h pvr, bool* content_watch)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && content_watch) {
-               *content_watch = _pvr->content_watch;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(content_watch);
 
-       return ret;
+       *content_watch = _pvr->content_watch;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_has_audio_only(media_pvr_h pvr, bool* has_audio_only)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && has_audio_only) {
-               *has_audio_only = _pvr->has_audio_only;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(has_audio_only);
 
-       return ret;
+       *has_audio_only = _pvr->has_audio_only;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_is_local_record(media_pvr_h pvr, bool* is_local_record)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && is_local_record) {
-               *is_local_record = _pvr->is_local_record;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(is_local_record);
 
-       return ret;
+       *is_local_record = _pvr->is_local_record;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_resolution(media_pvr_h pvr, media_pvr_resolution_e* resolution)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && resolution) {
-               *resolution = _pvr->resolution;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(resolution);
 
-       return ret;
+       *resolution = _pvr->resolution;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_aspectratio(media_pvr_h pvr, media_pvr_aspectratio_e* aspectratio)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && aspectratio) {
-               *aspectratio = _pvr->aspectratio;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(aspectratio);
 
-       return ret;
+       *aspectratio = _pvr->aspectratio;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_modified_month(media_pvr_h pvr, char **modified_month)
@@ -800,66 +660,50 @@ int media_pvr_get_modified_month(media_pvr_h pvr, char **modified_month)
 
 int media_pvr_get_sports_type(media_pvr_h pvr, int* sports_type)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && sports_type) {
-               *sports_type = _pvr->sports_type;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(sports_type);
 
-       return ret;
+       *sports_type = _pvr->sports_type;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_guidance_length(media_pvr_h pvr, int* guidance_length)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && guidance_length) {
-               *guidance_length = _pvr->guidance_length;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(guidance_length);
 
-       return ret;
+       *guidance_length = _pvr->guidance_length;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_tvmode(media_pvr_h pvr, int* tvmode)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && tvmode) {
-               *tvmode = _pvr->tvmode;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(tvmode);
 
-       return ret;
+       *tvmode = _pvr->tvmode;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_play_count(media_pvr_h pvr, int* play_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && play_count) {
-               *play_count = _pvr->play_count;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(play_count);
 
-       return ret;
+       *play_count = _pvr->play_count;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_get_private_data(media_pvr_h pvr, char **private_data)
@@ -877,33 +721,25 @@ int media_pvr_get_private_data(media_pvr_h pvr, char **private_data)
 
 int media_pvr_get_highlight(media_pvr_h pvr, bool *highlight)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr && highlight) {
-               *highlight = _pvr->highlight;
-               ret = MEDIA_CONTENT_ERROR_NONE;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(highlight);
 
-       return ret;
+       *highlight = _pvr->highlight;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_set_play_count(media_pvr_h pvr, int play_count)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr != NULL) {
-               _pvr->play_count = play_count;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
 
-       return ret;
+       _pvr->play_count = play_count;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_set_program_title(media_pvr_h pvr, const char *program_title)
@@ -921,47 +757,35 @@ int media_pvr_set_program_title(media_pvr_h pvr, const char *program_title)
 
 int media_pvr_set_content_lock(media_pvr_h pvr, bool content_lock)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr != NULL) {
-               _pvr->content_lock = content_lock;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
 
-       return ret;
+       _pvr->content_lock = content_lock;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_set_content_watch(media_pvr_h pvr, bool content_watch)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr != NULL) {
-               _pvr->content_watch = content_watch;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
 
-       return ret;
+       _pvr->content_watch = content_watch;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_set_highlight(media_pvr_h pvr, bool highlight)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
 
-       if (_pvr != NULL) {
-               _pvr->highlight = highlight;
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
+       content_retip_if_fail(pvr);
 
-       return ret;
+       _pvr->highlight = highlight;
+
+       return MEDIA_CONTENT_ERROR_NONE;
 }
 
 int media_pvr_update_to_db(media_pvr_h pvr)
@@ -970,34 +794,28 @@ int media_pvr_update_to_db(media_pvr_h pvr)
        media_pvr_s *_pvr = (media_pvr_s*)pvr;
        char *update_query = NULL;
 
-       if (_pvr != NULL && STRING_VALID(_pvr->media_id) && STRING_VALID(_pvr->path)) {
-               update_query = sqlite3_mprintf(UPDATE_PVR_META_FROM_PVR, _pvr->program_title, _pvr->content_lock, _pvr->content_watch, _pvr->play_count, _pvr->highlight, _pvr->media_id);
+       content_retip_if_fail(pvr);
+       content_retip_if_fail(STRING_VALID(_pvr->media_id));
+       content_retip_if_fail(STRING_VALID(_pvr->path));
 
-               ret = _content_query_sql(update_query);
-               sqlite3_free(update_query);
+       update_query = sqlite3_mprintf(UPDATE_PVR_META_FROM_PVR, _pvr->program_title, _pvr->content_lock, _pvr->content_watch, _pvr->play_count, _pvr->highlight, _pvr->media_id);
+
+       ret = _content_query_sql(update_query);
+       sqlite3_free(update_query);
+
+       if (ret == MEDIA_CONTENT_ERROR_NONE)
+               media_svc_update_pvr_info(_content_get_db_handle(), _pvr->path, _pvr->program_title, _pvr->content_lock);
 
-               if (ret == MEDIA_CONTENT_ERROR_NONE)
-                       media_svc_update_pvr_info(_content_get_db_handle(), _pvr->path, _pvr->program_title, _pvr->content_lock);
-       } else {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       }
 
        return ret;
 }
 
 int media_pvr_group_foreach_media_from_db(const char *group_name, media_group_e group, filter_h filter, media_pvr_cb callback, void *user_data)
 {
-       int ret = MEDIA_CONTENT_ERROR_NONE;
+       content_retip_if_fail(callback);
+       content_retvm_if((group < MEDIA_PVR_GROUP_DURATION) || (group >= MEDIA_GROUP_MAX), MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "Invalid group");
 
-       if ((callback == NULL) || (group < MEDIA_PVR_GROUP_DURATION) || (group >= MEDIA_GROUP_MAX)) {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       } else {
-               ret = _media_db_get_pvr_group_item(group_name, filter, group, callback, user_data);
-       }
-
-       return ret;
+       return _media_db_get_pvr_group_item(group_name, filter, group, callback, user_data);
 }
 
 int media_pvr_set_is_local_record(const char *pvr_path, bool is_local_record)
@@ -1006,25 +824,17 @@ int media_pvr_set_is_local_record(const char *pvr_path, bool is_local_record)
        char *update_query = NULL;
        char storage_id[MEDIA_CONTENT_UUID_SIZE+1] = {0,};
 
-       if (pvr_path == NULL) {
-               content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-               ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-       } else {
+       content_retip_if_fail(pvr_path);
 
-               /*get the storage uuid*/
-               ret = media_svc_get_storage_id(_content_get_db_handle(), pvr_path, storage_id, tzplatform_getuid(TZ_USER_NAME));
-               if (ret == MS_MEDIA_ERR_NONE) {
-                       content_error("storage uuid [%s]", storage_id);
-                       update_query = sqlite3_mprintf(UPDATE_PVR_LOCAL_RECORD_PVR, is_local_record, pvr_path, storage_id);
-
-                       ret = _content_query_sql(update_query);
-                       sqlite3_free(update_query);
-               } else {
-                       content_error("INVALID_PARAMETER(0x%08x)", MEDIA_CONTENT_ERROR_INVALID_PARAMETER);
-                       content_error("pvr path[%s] error[%d]", pvr_path, ret);
-                       ret = MEDIA_CONTENT_ERROR_INVALID_PARAMETER;
-               }
-       }
+       /*get the storage uuid*/
+       ret = media_svc_get_storage_id(_content_get_db_handle(), pvr_path, storage_id, tzplatform_getuid(TZ_USER_NAME));
+       content_retvm_if(ret != MS_MEDIA_ERR_NONE, MEDIA_CONTENT_ERROR_INVALID_PARAMETER, "Invalid pvr path[%s] error[%d]", pvr_path, ret);
+
+       content_error("storage uuid [%s]", storage_id);
+       update_query = sqlite3_mprintf(UPDATE_PVR_LOCAL_RECORD_PVR, is_local_record, pvr_path, storage_id);
+
+       ret = _content_query_sql(update_query);
+       sqlite3_free(update_query);
 
        return ret;
 }