add excluding coverage comments for tdm_vblank.c 68/161668/2
authorKonstantin Drabeniuk <k.drabeniuk@samsung.com>
Fri, 24 Nov 2017 13:19:45 +0000 (15:19 +0200)
committerRoman Marchenko <r.marchenko@samsung.com>
Sun, 26 Nov 2017 11:25:55 +0000 (13:25 +0200)
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 <k.drabeniuk@samsung.com>
src/tdm_vblank.c

index 462eaf5..39eeca6 100644 (file)
@@ -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 */