From ac775dc7dc73cd3dcf48496e6aec683496d31ef6 Mon Sep 17 00:00:00 2001 From: Minje Ahn Date: Thu, 26 Mar 2020 13:15:56 +0900 Subject: [PATCH] Add macro for invalid parameter case Change-Id: I28a8bac83c2603a404e4c5fb47261856241f2563 Signed-off-by: Minje Ahn --- include/media_info_private.h | 14 ++++++++++++++ include_product/media_info_private.h | 14 ++++++++++++++ 2 files changed, 28 insertions(+) diff --git a/include/media_info_private.h b/include/media_info_private.h index f999e0e..07b5a6f 100755 --- a/include/media_info_private.h +++ b/include/media_info_private.h @@ -566,6 +566,20 @@ int _media_filter_build_option(filter_h filter, char **result); } \ } while (0) +#define content_retip_if_fail(expr) do { \ + if (!(expr)) { \ + LOGE(FONT_COLOR_RED"[%ld] Invalid [%s]"FONT_COLOR_RESET, media_content_gettid(), #expr); \ + return MEDIA_CONTENT_ERROR_INVALID_PARAMETER; \ + } \ + } while (0) + +#define content_retipm_if_fail(expr, fmt, arg...) do { \ + if (!(expr)) { \ + LOGE(FONT_COLOR_RED"[%ld] Cond[%s] value["fmt"]"FONT_COLOR_RESET, media_content_gettid(), #expr, ##arg); \ + return MEDIA_CONTENT_ERROR_INVALID_PARAMETER; \ + } \ + } while (0) + #define content_warn(fmt, arg...) do { \ LOGW(FONT_COLOR_GREEN"[%ld]"fmt""FONT_COLOR_RESET, media_content_gettid(), ##arg); \ } while (0) diff --git a/include_product/media_info_private.h b/include_product/media_info_private.h index 22b91af..88bedc3 100755 --- a/include_product/media_info_private.h +++ b/include_product/media_info_private.h @@ -791,6 +791,20 @@ GMutex* _content_get_db_mutex(void); } \ } while (0) +#define content_retip_if_fail(expr) do { \ + if (!(expr)) { \ + LOGE(FONT_COLOR_RED"[%ld] Invalid [%s]"FONT_COLOR_RESET, media_content_gettid(), #expr); \ + return MEDIA_CONTENT_ERROR_INVALID_PARAMETER; \ + } \ + } while (0) + +#define content_retipm_if_fail(expr, fmt, arg...) do { \ + if (!(expr)) { \ + LOGE(FONT_COLOR_RED"[%ld] Cond[%s] value["fmt"]"FONT_COLOR_RESET, media_content_gettid(), #expr, ##arg); \ + return MEDIA_CONTENT_ERROR_INVALID_PARAMETER; \ + } \ + } while (0) + #define content_warn(fmt, arg...) do { \ LOGW(FONT_COLOR_GREEN"[%ld]"fmt""FONT_COLOR_RESET, media_content_gettid(), ##arg); \ } while (0) -- 2.7.4