From 2b9275884822c075062caaea48b0a893cab91a76 Mon Sep 17 00:00:00 2001 From: Junkyeong Kim Date: Fri, 26 Jul 2019 11:55:47 +0900 Subject: [PATCH] erase unnecessary comment Change-Id: I81f5f55fd33ef068628d885aa6b8e7b8d58eea53 Signed-off-by: Junkyeong Kim --- src/eom-wayland.c | 74 ++++++++++++++++++------------------------------------- src/eom.c | 30 ++++++++-------------- 2 files changed, 34 insertions(+), 70 deletions(-) diff --git a/src/eom-wayland.c b/src/eom-wayland.c index 126942a..1f18d56 100644 --- a/src/eom-wayland.c +++ b/src/eom-wayland.c @@ -159,8 +159,7 @@ _convert_to_eom_output_type(enum wl_eom_type eom_type) return output_type; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static eom_output_mode_e _convert_to_eom_output_mode(enum wl_eom_mode eom_mode) { @@ -183,8 +182,7 @@ _convert_to_eom_output_mode(enum wl_eom_mode eom_mode) return output_mode; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static eom_output_attribute_e _convert_to_eom_output_attribute(enum wl_eom_attribute eom_attribute) { @@ -210,8 +208,7 @@ _convert_to_eom_output_attribute(enum wl_eom_attribute eom_attribute) return output_attribute; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static eom_output_attribute_state_e _convert_to_eom_output_attribute_state( enum wl_eom_attribute_state eom_attribute_state) @@ -239,8 +236,7 @@ _convert_to_eom_output_attribute_state( return output_attribute_state; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static enum wl_eom_attribute _convert_to_wl_eom_attribute(eom_output_attribute_e attr) { @@ -266,8 +262,7 @@ _convert_to_wl_eom_attribute(eom_output_attribute_e attr) return eom_attribute; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_wayland_client_call_notify(EomWaylandOutput *eom_wl_output, eom_output_notify_type_e type) @@ -355,8 +350,7 @@ _eom_wayland_client_call_notify(EomWaylandOutput *eom_wl_output, if (array) g_array_free(array, FALSE); } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static EomWaylandOutput * _eom_wayland_client_find_output_from_wl_output( struct wl_list *eom_wl_output_list, int output_id) @@ -366,8 +360,7 @@ _eom_wayland_client_find_output_from_wl_output( EomWaylandOutput *ret = NULL; if (!wl_list_empty(eom_wl_output_list)) { - wl_list_for_each_safe(eom_wl_output, - tmp, eom_wl_output_list, link) { + wl_list_for_each_safe(eom_wl_output, tmp, eom_wl_output_list, link) { if (eom_wl_output->id == output_id) { ret = eom_wl_output; break; @@ -377,8 +370,7 @@ _eom_wayland_client_find_output_from_wl_output( return ret; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static EomWaylandOutput * _eom_wayland_client_find_output_from_eom_output( struct wl_list *eom_wl_output_list, eom_output_id id) @@ -388,8 +380,7 @@ _eom_wayland_client_find_output_from_eom_output( EomWaylandOutput *ret = NULL; if (!wl_list_empty(eom_wl_output_list)) { - wl_list_for_each_safe(eom_wl_output, - tmp, eom_wl_output_list, link) { + wl_list_for_each_safe(eom_wl_output, tmp, eom_wl_output_list, link) { if (eom_wl_output->id == id) { ret = eom_wl_output; break; @@ -505,8 +496,7 @@ _eom_wl_eom_output_count(void *data, eom_client_info->num_outputs = count; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_wl_eom_output_info(void *data, struct wl_eom *wl_eom, @@ -548,11 +538,9 @@ _eom_wl_eom_output_info(void *data, INFO("INFO send connection: %d", connection); if (connection == WL_EOM_STATUS_CONNECTION) - _eom_wayland_client_call_notify(eom_wl_output, - EOM_OUTPUT_NOTIFY_ADD); + _eom_wayland_client_call_notify(eom_wl_output, EOM_OUTPUT_NOTIFY_ADD); else if (connection == WL_EOM_STATUS_DISCONNECTION) - _eom_wayland_client_call_notify(eom_wl_output, - EOM_OUTPUT_NOTIFY_REMOVE); + _eom_wayland_client_call_notify(eom_wl_output, EOM_OUTPUT_NOTIFY_REMOVE); } /* check the eom mode and call the notify */ @@ -561,8 +549,7 @@ _eom_wl_eom_output_info(void *data, INFO("INFO send mode:%d", mode); - _eom_wayland_client_call_notify(eom_wl_output, - EOM_OUTPUT_NOTIFY_MODE_CHANGED); + _eom_wayland_client_call_notify(eom_wl_output, EOM_OUTPUT_NOTIFY_MODE_CHANGED); } if (skip) @@ -577,8 +564,7 @@ _eom_wl_eom_output_info(void *data, INFO("INFO NEW send attribute:%d", attribute); INFO("INFO NEW send attribute state:%d", attribute_state); - _eom_wayland_client_call_notify(eom_wl_output, - EOM_OUTPUT_NOTIFY_ATTRIBUTE_CHANGED); + _eom_wayland_client_call_notify(eom_wl_output, EOM_OUTPUT_NOTIFY_ATTRIBUTE_CHANGED); } eom_wl_output->error = error; @@ -604,8 +590,7 @@ _eom_wl_eom_output_info(void *data, wl_list_insert(&eom_client_info->eom_wl_output_list, &eom_wl_output->link); } } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_wl_eom_output_type(void *data, struct wl_eom *wl_eom, @@ -631,15 +616,12 @@ _eom_wl_eom_output_type(void *data, eom_wl_output->eom_status = status; if (status == WL_EOM_STATUS_CONNECTION) - _eom_wayland_client_call_notify(eom_wl_output, - EOM_OUTPUT_NOTIFY_ADD); + _eom_wayland_client_call_notify(eom_wl_output, EOM_OUTPUT_NOTIFY_ADD); else if (status == WL_EOM_STATUS_DISCONNECTION) - _eom_wayland_client_call_notify(eom_wl_output, - EOM_OUTPUT_NOTIFY_REMOVE); + _eom_wayland_client_call_notify(eom_wl_output, EOM_OUTPUT_NOTIFY_REMOVE); } } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_wl_eom_output_mode(void *data, struct wl_eom *wl_eom, @@ -659,12 +641,10 @@ _eom_wl_eom_output_mode(void *data, if (eom_wl_output->eom_mode != mode) { eom_wl_output->eom_mode = mode; - _eom_wayland_client_call_notify(eom_wl_output, - EOM_OUTPUT_NOTIFY_MODE_CHANGED); + _eom_wayland_client_call_notify(eom_wl_output, EOM_OUTPUT_NOTIFY_MODE_CHANGED); } } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_wl_eom_output_attribute(void *data, struct wl_eom *wl_eom, @@ -688,8 +668,7 @@ _eom_wl_eom_output_attribute(void *data, eom_wl_output->eom_attribute = attribute; eom_wl_output->eom_attribute_state = attribute_state; - _eom_wayland_client_call_notify(eom_wl_output, - EOM_OUTPUT_NOTIFY_ATTRIBUTE_CHANGED); + _eom_wayland_client_call_notify(eom_wl_output, EOM_OUTPUT_NOTIFY_ATTRIBUTE_CHANGED); } eom_wl_output->error = error; @@ -723,7 +702,6 @@ static const struct wl_eom_listener eom_wl_eom_listener = { _eom_wl_eom_output_set_window, }; - static void _eom_wl_registry_handle_global(void *data, struct wl_registry *registry, uint32_t name, const char *interface, uint32_t version) @@ -821,8 +799,7 @@ _eom_wayland_client_initialize() EomWaylandOutput *eom_wl_output = NULL; EomWaylandOutput *tmp = NULL; - wl_list_for_each_safe(eom_wl_output, tmp, - &wl_client_info.eom_wl_output_list, link) { + wl_list_for_each_safe(eom_wl_output, tmp, &wl_client_info.eom_wl_output_list, link) { if (eom_wl_output->eom_type == WL_EOM_TYPE_NONE) { WARN("[EOM_CLIENT] eom_type is NONE. remove output:%d\n", eom_wl_output->id); @@ -867,8 +844,7 @@ _eom_wayland_client_deinitialize() /* remove all eom_wl_outputs */ if (!wl_list_empty(&wl_client_info.eom_wl_output_list)) { - wl_list_for_each_safe(eom_wl_output, tmp, - &wl_client_info.eom_wl_output_list, link) { + wl_list_for_each_safe(eom_wl_output, tmp, &wl_client_info.eom_wl_output_list, link) { if (eom_wl_output->output) wl_output_destroy(eom_wl_output->output); free(eom_wl_output); @@ -937,9 +913,7 @@ eom_wayland_client_get_output_ids(void) /*LCOV_EXCL_START*/ array = g_array_new(FALSE, FALSE, sizeof(GValue)); - wl_list_for_each_safe(eom_wl_output, tmp, - &wl_client_info.eom_wl_output_list, link) { - + wl_list_for_each_safe(eom_wl_output, tmp, &wl_client_info.eom_wl_output_list, link) { if (eom_wl_output) { g_value_init(&v, G_TYPE_INT); g_value_set_int(&v, eom_wl_output->id); diff --git a/src/eom.c b/src/eom.c index 8dc35bc..300e249 100644 --- a/src/eom.c +++ b/src/eom.c @@ -228,8 +228,7 @@ _eom_set_output_info_mode(eom_output_info *output_info, output_info->output_mode = output_mode; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_set_output_attribute(eom_output_info *output_info, eom_output_attribute_e attribute) @@ -238,8 +237,7 @@ _eom_set_output_attribute(eom_output_info *output_info, output_info->attribute = attribute; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_set_output_attribute_state(eom_output_info *output_info, eom_output_attribute_state_e state) @@ -248,8 +246,7 @@ _eom_set_output_attribute_state(eom_output_info *output_info, output_info->state = state; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_set_output_info_size(eom_output_info *output_info, int w, int h) @@ -259,8 +256,7 @@ _eom_set_output_info_size(eom_output_info *output_info, output_info->width = w; output_info->height = h; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_set_output_info_phy_size(eom_output_info *output_info, int w_mm, int h_mm) @@ -270,8 +266,7 @@ _eom_set_output_info_phy_size(eom_output_info *output_info, output_info->mm_width = w_mm; output_info->mm_height = h_mm; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_reset_output_info(eom_output_info *output_info) { @@ -282,8 +277,7 @@ _eom_reset_output_info(eom_output_info *output_info) output_info->mm_width = 0; output_info->mm_height = 0; } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_output_call_notify_cb(eom_output_notify_s *notify) { @@ -341,8 +335,7 @@ _eom_output_call_notify_cb(eom_output_notify_s *notify) } } } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static void _eom_output_process_notify_cb(void *data, GArray *array) { @@ -477,24 +470,22 @@ _eom_output_process_notify_cb(void *data, GArray *array) exit: _eom_mutex_unlock(); } -/*LCOV_EXCL_STOP*/ + static eom_output_info* _eom_find_output_info(eom_output_id output_id) { GList *l; for (l = output_info_list; l; l = g_list_next(l)) { -/*LCOV_EXCL_START*/ eom_output_info *output_info = (eom_output_info *)l->data; if (output_info && (output_id == output_info->id)) return output_info; -/*LCOV_EXCL_STOP*/ } return NULL; } -/*LCOV_EXCL_START*/ + static void _eom_free_output_info(eom_output_info **output_info) { @@ -503,8 +494,7 @@ _eom_free_output_info(eom_output_info **output_info) *output_info = NULL; } } -/*LCOV_EXCL_STOP*/ -/*LCOV_EXCL_START*/ + static eom_output_info* _eom_alloc_output_info(int output_id, int output_type) { -- 2.7.4