From: Konstantin Drabeniuk Date: Fri, 24 Nov 2017 13:19:45 +0000 (+0200) Subject: add excluding coverage comments for tdm_vblank.c X-Git-Tag: accepted/tizen/unified/20171205.060640~18 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;ds=sidebyside;h=c82ff146433678e371da26fcf96ab8b613f1a7a1;p=platform%2Fcore%2Fuifw%2Flibtdm.git add excluding coverage comments for tdm_vblank.c add excluding coverage comments for tdm_vblank.c for the folowing code: - _tdm_vblank_cb_output_change; - pthread's fail; - calloc's fail; - tdm_event_loop_source_timer_update() func's fail; - _tdm_vblank_sw_timer_update() func's fail; - funcs for tdm_monitor. Change-Id: I67a22c38fd1ed19fde6ebf12a08681c60554f16c Signed-off-by: Konstantin Drabeniuk --- diff --git a/src/tdm_vblank.c b/src/tdm_vblank.c index 462eaf5..39eeca6 100644 --- a/src/tdm_vblank.c +++ b/src/tdm_vblank.c @@ -352,6 +352,7 @@ _tdm_vblank_update_output_info(tdm_private_vblank *private_vblank) private_vblank->vrefresh, private_vblank->fps, private_vblank->fps_changeable); } +/* LCOV_EXCL_START */ static void _tdm_vblank_cb_output_change(tdm_output *output, tdm_output_change_type type, tdm_value value, void *user_data) @@ -388,6 +389,7 @@ _tdm_vblank_cb_output_change(tdm_output *output, tdm_output_change_type type, break; } } +/* LCOV_EXCL_STOP */ EXTERN tdm_error tdm_vblank_set_client_vblank_fps(unsigned int pid, const char *name, unsigned int fps) @@ -470,10 +472,12 @@ tdm_vblank_create(tdm_display *dpy, tdm_output *output, tdm_error *error) if (!vblank_list_inited) { if (pthread_mutex_init(&valid_list_lock, NULL)) { + /* LCOV_EXCL_START */ TDM_ERR("mutex init failed: %m"); if (error) *error = TDM_ERROR_OPERATION_FAILED; return NULL; + /* LCOV_EXCL_STOP */ } LIST_INITHEAD(&valid_vblank_list); LIST_INITHEAD(&valid_wait_list); @@ -482,10 +486,12 @@ tdm_vblank_create(tdm_display *dpy, tdm_output *output, tdm_error *error) private_vblank = calloc(1, sizeof * private_vblank); if (!private_vblank) { + /* LCOV_EXCL_START */ if (error) *error = TDM_ERROR_OUT_OF_MEMORY; VER("alloc failed"); return NULL; + /* LCOV_EXCL_STOP */ } tdm_output_add_change_handler(output, _tdm_vblank_cb_output_change, private_vblank); @@ -740,9 +746,11 @@ _tdm_vblank_sw_timer_update(tdm_private_vblank *private_vblank) ret = tdm_event_loop_source_timer_update(private_vblank->SW_timer, ms_delay); if (ret != TDM_ERROR_NONE) { + /* LCOV_EXCL_START */ tdm_display_unlock(private_vblank->dpy); VER("couldn't update timer"); return ret; + /* LCOV_EXCL_STOP */ } tdm_display_unlock(private_vblank->dpy); @@ -1038,9 +1046,11 @@ _tdm_vblank_wait_SW(tdm_vblank_wait_info *wait_info) ret = _tdm_vblank_sw_timer_update(private_vblank); if (ret != TDM_ERROR_NONE) { + /* LCOV_EXCL_START */ LIST_DEL(&wait_info->link); VER("couldn't update sw timer"); return ret; + /* LCOV_EXCL_STOP */ } return TDM_ERROR_NONE; @@ -1079,8 +1089,10 @@ tdm_vblank_wait(tdm_vblank *vblank, unsigned int req_sec, unsigned int req_usec, wait_info = calloc(1, sizeof * wait_info); if (!wait_info) { + /* LCOV_EXCL_START */ VER("alloc failed"); return TDM_ERROR_OUT_OF_MEMORY; + /* LCOV_EXCL_STOP */ } /* set request time to current time if 0. This function seems to be called in server side. */ @@ -1139,6 +1151,7 @@ tdm_vblank_wait(tdm_vblank *vblank, unsigned int req_sec, unsigned int req_usec, } if (ret != TDM_ERROR_NONE) { + /* LCOV_EXCL_START */ LIST_DEL(&wait_info->link); _tdm_vblank_valid_list_del(&wait_info->valid_link); @@ -1148,6 +1161,7 @@ tdm_vblank_wait(tdm_vblank *vblank, unsigned int req_sec, unsigned int req_usec, free(wait_info); return ret; + /* LCOV_EXCL_STOP */ } return TDM_ERROR_NONE; @@ -1204,6 +1218,7 @@ tdm_vblank_wait_seq(tdm_vblank *vblank, unsigned int req_sec, unsigned int req_u return tdm_vblank_wait(vblank, req_sec, req_usec, interval, func, user_data); } +/* LCOV_EXCL_START */ INTERN tdm_error tdm_vblank_set_add_front(tdm_vblank *vblank, unsigned int add_front) { @@ -1220,6 +1235,7 @@ tdm_vblank_set_add_front(tdm_vblank *vblank, unsigned int add_front) return TDM_ERROR_NONE; } +/* LCOV_EXCL_STOP */ INTERN tdm_error tdm_vblank_set_resource(tdm_vblank *vblank, struct wl_resource *resource) @@ -1244,6 +1260,7 @@ tdm_vblank_get_stamp(tdm_vblank *vblank) return private_vblank->stamp; } +/* LCOV_EXCL_START */ INTERN void tdm_vblank_get_vblank_list_information(tdm_display *dpy, char *reply, int *len) { @@ -1276,4 +1293,5 @@ tdm_vblank_get_vblank_list_information(tdm_display *dpy, char *reply, int *len) TDM_SNPRINTF(reply, len, "\n"); } +/* LCOV_EXCL_STOP */