Fixed: Coding rule issues.
[apps/native/ug-wifi-direct.git] / ug-wifidirect / src / wfd_ug_main_view.c
index a53be85..7b3dba0 100755 (executable)
@@ -49,7 +49,7 @@ void scan_button_create(struct ug_data *ugd)
                wfd_ug_view_refresh_button(btn, "IDS_WIFI_SK4_SCAN",
                        FALSE);
        }
-       evas_object_smart_callback_add(btn, "clicked",_scan_btn_cb, (void *)ugd);
+       evas_object_smart_callback_add(btn, "clicked", _scan_btn_cb, (void *)ugd);
        elm_layout_content_set(ugd->button_layout, "button.big", btn);
        ugd->scan_toolbar = btn;
 
@@ -98,15 +98,13 @@ Eina_Bool _back_btn_cb(void *data, Elm_Object_Item *it)
                if (ugd->is_conn_incoming) {
                        DBG(LOG_INFO, "Reject the incoming connection before client deregister \n");
                        ret = wifi_direct_reject_connection(ugd->mac_addr_connecting);
-                       if (ret != WIFI_DIRECT_ERROR_NONE) {
+                       if (ret != WIFI_DIRECT_ERROR_NONE)
                                DBG(LOG_ERROR, "Failed to send reject request [%d]\n", ret);
-                       }
                } else {
                        DBG(LOG_INFO, "Cancel the outgoing connection before client deregister \n");
                        ret = wifi_direct_cancel_connection(ugd->mac_addr_connecting);
-                       if (ret != WIFI_DIRECT_ERROR_NONE) {
+                       if (ret != WIFI_DIRECT_ERROR_NONE)
                                DBG(LOG_ERROR, "Failed to send cancel request [%d]\n", ret);
-                       }
                }
                ugd->mac_addr_connecting = NULL;
        }
@@ -114,9 +112,8 @@ Eina_Bool _back_btn_cb(void *data, Elm_Object_Item *it)
        if (ugd->raw_connected_peer_cnt == 0) {
                ret = wifi_direct_is_group_owner(&owner);
                if (ret == WIFI_DIRECT_ERROR_NONE) {
-                       if (owner) {
+                       if (owner)
                                wifi_direct_destroy_group();
-                       }
                }
        }
 
@@ -126,11 +123,10 @@ cleanup:
        if (ret) {
                DBG(LOG_ERROR, "Failed to create control");
        } else {
-               if (ugd->wfd_status > WIFI_DIRECT_STATE_CONNECTING) {
+               if (ugd->wfd_status > WIFI_DIRECT_STATE_CONNECTING)
                        app_control_add_extra_data(control, "Connection", "TRUE");
-               } else {
+               else
                        app_control_add_extra_data(control, "Connection", "FALSE");
-               }
 
                ug_send_result(ugd->ug, control);
                app_control_destroy(control);
@@ -156,7 +152,7 @@ void _smart_back_btn_cb(void *data, Evas_Object *obj, void *event_info)
        bool owner = FALSE;
        app_control_h control = NULL;
 
-       if(ugd == NULL) {
+       if (ugd == NULL) {
                DBG(LOG_ERROR, "The param is NULL");
                return;
        }
@@ -184,15 +180,13 @@ void _smart_back_btn_cb(void *data, Evas_Object *obj, void *event_info)
                if (ugd->is_conn_incoming) {
                        DBG(LOG_INFO, "Reject the incoming connection before client deregister \n");
                        ret = wifi_direct_reject_connection(ugd->mac_addr_connecting);
-                       if (ret != WIFI_DIRECT_ERROR_NONE) {
+                       if (ret != WIFI_DIRECT_ERROR_NONE)
                                DBG(LOG_ERROR, "Failed to send reject request [%d]\n", ret);
-                       }
                } else {
                        DBG(LOG_INFO, "Cancel the outgoing connection before client deregister \n");
                        ret = wifi_direct_cancel_connection(ugd->mac_addr_connecting);
-                       if (ret != WIFI_DIRECT_ERROR_NONE) {
+                       if (ret != WIFI_DIRECT_ERROR_NONE)
                                DBG(LOG_ERROR, "Failed to send cancel request [%d]\n", ret);
-                       }
                }
                ugd->mac_addr_connecting = NULL;
        }
@@ -200,9 +194,8 @@ void _smart_back_btn_cb(void *data, Evas_Object *obj, void *event_info)
        if (ugd->raw_connected_peer_cnt == 0) {
                ret = wifi_direct_is_group_owner(&owner);
                if (ret == WIFI_DIRECT_ERROR_NONE) {
-                       if (owner) {
+                       if (owner)
                                wifi_direct_destroy_group();
-                       }
                }
        }
 
@@ -212,11 +205,10 @@ cleanup:
        if (ret) {
                DBG(LOG_ERROR, "Failed to create control");
        } else {
-               if (ugd->wfd_status > WIFI_DIRECT_STATE_CONNECTING) {
+               if (ugd->wfd_status > WIFI_DIRECT_STATE_CONNECTING)
                        app_control_add_extra_data(control, "Connection", "TRUE");
-               } else {
+               else
                        app_control_add_extra_data(control, "Connection", "FALSE");
-               }
 
                ug_send_result(ugd->ug, control);
                app_control_destroy(control);
@@ -231,9 +223,9 @@ void wfd_cancel_progressbar_stop_timer(struct ug_data *ugd)
 {
        __FUNC_ENTER__;
 
-       if(ugd->timer_stop_progress_bar > 0) {
+       if (ugd->timer_stop_progress_bar > 0)
                g_source_remove(ugd->timer_stop_progress_bar);
-       }
+
        ugd->timer_stop_progress_bar = 0;
 
        __FUNC_EXIT__;
@@ -243,9 +235,9 @@ void wfd_cancel_not_alive_delete_timer(struct ug_data *ugd)
 {
        __FUNC_ENTER__;
 
-       if(ugd->timer_delete_not_alive_peer > 0) {
+       if (ugd->timer_delete_not_alive_peer > 0)
                g_source_remove(ugd->timer_delete_not_alive_peer);
-       }
+
        ugd->timer_delete_not_alive_peer = 0;
 
        __FUNC_EXIT__;
@@ -314,7 +306,7 @@ void _scan_btn_cb(void *data, Evas_Object *obj, void *event_info)
        int ret = -1;
        const char *btn_text = NULL;
        btn_text = elm_object_part_text_get(ugd->scan_toolbar, "default");
-       DBG(LOG_INFO, "Button text=%s",btn_text);
+       DBG(LOG_INFO, "Button text=%s", btn_text);
 
        if (!g_strcmp0(elm_object_text_get(obj), D_("IDS_WIFI_SK4_SCAN"))) {
                wfd_refresh_wifi_direct_state(ugd);
@@ -354,18 +346,18 @@ void _scan_btn_cb(void *data, Evas_Object *obj, void *event_info)
 void wfd_check_gl_busy_peers(struct ug_data *ugd)
 {
        __FUNC_ENTER__;
-       if (ugd->gl_busy_peer_cnt == 0) {
+       if (ugd->gl_busy_peer_cnt == 0)
                WFD_IF_DEL_ITEM(ugd->busy_wfd_item);
-       }
+
        __FUNC_EXIT__;
 }
 
 void wfd_check_gl_available_peers(struct ug_data *ugd)
 {
        __FUNC_ENTER__;
-       if (ugd->gl_available_peer_cnt == 0 && ugd->avlbl_wfd_item != NULL) {
+       if (ugd->gl_available_peer_cnt == 0 && ugd->avlbl_wfd_item != NULL)
                WFD_IF_DEL_ITEM(ugd->avlbl_wfd_item);
-       }
+
        __FUNC_EXIT__;
 }
 
@@ -388,11 +380,11 @@ void free_gl_peer(device_type_s **start_pos, const char *mac_addr, int *cnt)
        }
 
        while (peer) {
-               if(strcmp(peer->mac_addr, mac_addr)) {
+               if (strcmp(peer->mac_addr, mac_addr)) {
                        peer_tmp = peer;
                        peer = peer->next;
                } else {
-                       if(peer->next != NULL) {
+                       if (peer->next != NULL) {
                                peer_tmp->next = peer->next;
                                break;
                        } else {
@@ -442,15 +434,15 @@ static void _gl_peer_sel(void *data, Evas_Object *obj, void *event_info)
        }
 
        wfd_ug_get_connected_peers(ugd);
-       DBG(LOG_INFO, "No of connected peers= %d",ugd->raw_connected_peer_cnt);
+       DBG(LOG_INFO, "No of connected peers= %d", ugd->raw_connected_peer_cnt);
 
        if (ugd->raw_connected_peer_cnt >= MAX_CONNECTED_PEER_NUM) {
                format_str = D_("IDS_ST_POP_YOU_CAN_CONNECT_UP_TO_PD_DEVICES_AT_THE_SAME_TIME");
                snprintf(popup_text, MAX_POPUP_TEXT_SIZE, format_str, MAX_CONNECTED_PEER_NUM);
                wfd_ug_warn_popup(ugd, popup_text, POP_TYPE_MULTI_CONNECT_POPUP);
-               if (item) {
+               if (item)
                        elm_genlist_item_selected_set(item, EINA_FALSE);
-               }
+
                return;
        }
 
@@ -462,9 +454,8 @@ static void _gl_peer_sel(void *data, Evas_Object *obj, void *event_info)
                elm_layout_content_set(ugd->button_layout, "button.big", ugd->scan_toolbar);
        }
 
-       if (item) {
+       if (item)
                elm_genlist_item_selected_set(item, EINA_FALSE);
-       }
 
        GList *iterator = NULL;
 
@@ -480,7 +471,7 @@ static void _gl_peer_sel(void *data, Evas_Object *obj, void *event_info)
                /* peer exists only in genlist, waiting to be deleted */
                device_type_s *peer_start = ugd->gl_avlb_peers_start;
                for (iterator = ugd->raw_discovered_peer_list; iterator; iterator = iterator->next) {
-                       if(!strncmp(peer_start->mac_addr, peer->mac_addr, MAC_LENGTH)) {
+                       if (!strncmp(peer_start->mac_addr, peer->mac_addr, MAC_LENGTH)) {
                                DBG(LOG_INFO, "Device [%s] found in genlist, but it is already lost", ((device_type_s *)iterator->data)->ssid);
                                snprintf(txt, sizeof(txt), "Cannot find device %s", ((device_type_s *)iterator->data)->ssid);
                                free_gl_peer(&ugd->gl_avlb_peers_start, ((device_type_s *)iterator->data)->mac_addr, &ugd->gl_available_peer_cnt);
@@ -531,9 +522,8 @@ static void _gl_busy_peer_sel(void *data, Evas_Object *obj, void *event_info)
        struct ug_data *ugd = (struct ug_data *)wfd_get_ug_data();
        Elm_Object_Item *item = (Elm_Object_Item *)event_info;
 
-       if (item) {
+       if (item)
                elm_genlist_item_selected_set(item, EINA_FALSE);
-       }
 
        if (NULL == ugd) {
                DBG(LOG_ERROR, "Data is NULL\n");
@@ -583,26 +573,26 @@ void _ctxpopup_move()
        changed_ang = elm_win_rotation_get(ugd->win);
 
        switch (changed_ang) {
-               case 0:
-               case 180:
-                       move_x = win_w/2;
-                       move_y = win_h;
-                       break;
+       case 0:
+       case 180:
+               move_x = win_w/2;
+               move_y = win_h;
+               break;
 
-               case 90:
-                       move_x = win_h/2;
-                       move_y = win_w;
-                       break;
+       case 90:
+               move_x = win_h/2;
+               move_y = win_w;
+               break;
 
-               case 270:
-                       move_x = win_h/2;
-                       move_y = win_w;
-                       break;
+       case 270:
+               move_x = win_h/2;
+               move_y = win_w;
+               break;
 
-               default:
-                       move_x = 0;
-                       move_y = 0;
-                       break;
+       default:
+               move_x = 0;
+               move_y = 0;
+               break;
        }
 
        evas_object_move(ugd->ctxpopup, move_x, move_y);
@@ -648,9 +638,8 @@ void _more_button_cb(void *data, Evas_Object *obj, void *event_info)
 
        ugd->more_btn_multiconnect_item = NULL;
 
-       if (ugd->ctxpopup) {
+       if (ugd->ctxpopup)
                evas_object_del(ugd->ctxpopup);
-       }
 
        ugd->ctxpopup = elm_ctxpopup_add(naviframe);
        elm_object_style_set(ugd->ctxpopup, "more/default");
@@ -692,14 +681,12 @@ void _wfd_realize_item(Elm_Object_Item *pre_item, int count)
 {
        __FUNC_ENTER__;
        int i = 0;
-       if (count < 1 || pre_item == NULL) {
+       if (count < 1 || pre_item == NULL)
                return;
-       }
 
        Elm_Object_Item *item = elm_genlist_item_next_get(pre_item);
-       if (item == NULL) {
+       if (item == NULL)
                return;
-       }
 
        if (count == 1) {
                elm_object_item_signal_emit(item, "elm,state,normal", "");
@@ -707,13 +694,12 @@ void _wfd_realize_item(Elm_Object_Item *pre_item, int count)
        }
 
        for (i = 0; i < count; i++) {
-               if (i == 0) {
+               if (i == 0)
                        elm_object_item_signal_emit(item, "elm,state,top", "");
-               } else if (i == count - 1) {
+               else if (i == count - 1)
                        elm_object_item_signal_emit(item, "elm,state,bottom", "");
-               } else {
+               else
                        elm_object_item_signal_emit(item, "elm,state,center", "");
-               }
 
                item = elm_genlist_item_next_get(item);
        }
@@ -771,11 +757,10 @@ static void _gl_realized(void *data, Evas_Object *obj, void *event_info)
                Evas_Object *check = elm_object_item_part_content_get(item, "elm.icon");
                if (check) {
                        Eina_Bool state = elm_check_state_get(check);
-                       if (state) {
+                       if (state)
                                sr_msg = strdup(SR_CHECKBOX_ON_MSG);
-                       } else {
+                       else
                                sr_msg = strdup(SR_CHECKBOX_OFF_MSG);
-                       }
 
                        if (sr_msg) {
                                Evas_Object *ao = NULL;
@@ -839,9 +824,9 @@ void _wfd_ug_cancel_connection_button_cb(void *data, Evas_Object * obj, void *ev
 void wfd_ug_view_refresh_glitem(Elm_Object_Item *gl_item)
 {
        __FUNC_ENTER__;
-       if (gl_item != NULL) {
+       if (gl_item != NULL)
                elm_genlist_item_update(gl_item);
-       }
+
        __FUNC_EXIT__;
 }
 
@@ -903,23 +888,19 @@ static bool __wfd_is_device_busy(struct ug_data *ugd, device_type_s *dev)
        __FUNC_ENTER__;
 
        if (ugd->I_am_group_owner == TRUE) {
-               if (dev->is_connected || dev->is_group_owner) {
+               if (dev->is_connected || dev->is_group_owner)
                        return TRUE;
-               } else {
+               else
                        return FALSE;
-               }
        } else {
-               if (dev->is_connected == TRUE && dev->is_group_owner == TRUE) {
+               if (dev->is_connected == TRUE && dev->is_group_owner == TRUE)
                        return FALSE;
-               }
 
-               if (dev->is_connected == TRUE && dev->is_group_owner == FALSE) {
+               if (dev->is_connected == TRUE && dev->is_group_owner == FALSE)
                        return TRUE;
-               }
 
-               if (dev->is_connected == FALSE) {
+               if (dev->is_connected == FALSE)
                        return FALSE;
-               }
        }
 
        __FUNC_EXIT__;
@@ -939,13 +920,12 @@ static bool __wfd_is_any_device_available(struct ug_data *ugd, int* no_of_availa
 
        for (iterator = ugd->raw_discovered_peer_list; iterator; iterator = iterator->next) {
                /* Not include the device which is connected with me */
-               if (__wfd_is_device_connected_with_me(ugd, (device_type_s *)iterator->data)) {
+               if (__wfd_is_device_connected_with_me(ugd, (device_type_s *)iterator->data))
                        continue;
-               }
+
                if (!__wfd_is_device_busy(ugd, (device_type_s *)iterator->data) &&
-                       ((device_type_s *)iterator->data)->conn_status != PEER_CONN_STATUS_FAILED_TO_CONNECT) {
+                       ((device_type_s *)iterator->data)->conn_status != PEER_CONN_STATUS_FAILED_TO_CONNECT)
                        (*no_of_available_dev)++;
-               }
        }
 
        __FUNC_EXIT__;
@@ -965,12 +945,11 @@ static bool __wfd_is_any_device_busy(struct ug_data *ugd, int* no_of_busy_dev)
 
        for (iterator = ugd->raw_discovered_peer_list; iterator; iterator = iterator->next) {
                /* Not include the device which is connected with me */
-               if (__wfd_is_device_connected_with_me(ugd, (device_type_s *)iterator->data)) {
+               if (__wfd_is_device_connected_with_me(ugd, (device_type_s *)iterator->data))
                        continue;
-               }
-               if (__wfd_is_device_busy(ugd, (device_type_s *)iterator->data)) {
+
+               if (__wfd_is_device_busy(ugd, (device_type_s *)iterator->data))
                        (*no_of_busy_dev)++;
-               }
        }
 
        __FUNC_EXIT__;
@@ -990,9 +969,9 @@ static bool __wfd_is_any_device_connect_failed(struct ug_data *ugd, int* no_of_c
 
        for (iterator = ugd->raw_discovered_peer_list; iterator; iterator = iterator->next) {
                /* Not include the device which is connected with me */
-               if (__wfd_is_device_connected_with_me(ugd, (device_type_s *)iterator->data)) {
+               if (__wfd_is_device_connected_with_me(ugd, (device_type_s *)iterator->data))
                        continue;
-               }
+
                if (!__wfd_is_device_busy(ugd, (device_type_s *)iterator->data) &&
                        ((device_type_s *)iterator->data)->conn_status == PEER_CONN_STATUS_FAILED_TO_CONNECT) {
                        (*no_of_connect_failed_dev)++;
@@ -1024,7 +1003,7 @@ static Evas_Object *_create_basic_genlist(void *data)
        /* Wifi ON/OFF toggle button */
 #ifdef WFD_ON_OFF_GENLIST
        ugd->item_wifi_onoff = elm_genlist_item_append(genlist, &wfd_onoff_itc, ugd,
-                       NULL, ELM_GENLIST_ITEM_NONE,_onoff_changed_cb, ugd);
+                       NULL, ELM_GENLIST_ITEM_NONE, _onoff_changed_cb, ugd);
        elm_genlist_item_selected_set(ugd->item_wifi_onoff,
                                        EINA_FALSE);
 #endif
@@ -1032,7 +1011,7 @@ static Evas_Object *_create_basic_genlist(void *data)
        evas_object_smart_callback_add(genlist, "unrealized", _gl_unrealized, ugd);
        ugd->device_name_item = elm_genlist_item_append(genlist, &device_name_itc, ugd, NULL,
                ELM_GENLIST_ITEM_NONE, NULL, NULL);
-       if(ugd->device_name_item != NULL)
+       if (ugd->device_name_item != NULL)
                elm_genlist_item_select_mode_set(ugd->device_name_item, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
        __FUNC_EXIT__;
@@ -1055,9 +1034,9 @@ Evas_Object *_create_no_device_genlist(void *data)
 
        if (ugd->conn_wfd_item != NULL) {
                last_item = ugd->conn_wfd_item;
-               for (i = 0; i < ugd->gl_connected_peer_cnt; i++) {
+               for (i = 0; i < ugd->gl_connected_peer_cnt; i++)
                        last_item = elm_genlist_item_next_get(last_item);
-               }
+
        } else {
                last_item = ugd->device_name_item;
        }
@@ -1065,12 +1044,12 @@ Evas_Object *_create_no_device_genlist(void *data)
        ugd->nodevice_title_item = elm_genlist_item_insert_after(ugd->genlist,
                &title_no_device_itc, (void *)ugd, NULL, last_item,
                ELM_GENLIST_ITEM_NONE, NULL, NULL);
-       if(ugd->nodevice_title_item != NULL)
+       if (ugd->nodevice_title_item != NULL)
                elm_genlist_item_select_mode_set(ugd->nodevice_title_item, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
        ugd->nodevice_item = elm_genlist_item_insert_after(ugd->genlist, &noitem_itc, (void *)ugd, NULL,
                ugd->nodevice_title_item, ELM_GENLIST_ITEM_NONE, NULL, NULL);
-       if(ugd->nodevice_item != NULL)
+       if (ugd->nodevice_item != NULL)
                elm_genlist_item_select_mode_set(ugd->nodevice_item, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
 
        __FUNC_EXIT__;
@@ -1089,7 +1068,7 @@ int _create_busy_dev_list(void *data)
 
        ugd->busy_wfd_item = elm_genlist_item_append(ugd->genlist, &title_busy_itc, (void *)ugd,
                NULL, ELM_GENLIST_ITEM_NONE, NULL, NULL);
-       if(ugd->busy_wfd_item != NULL)
+       if (ugd->busy_wfd_item != NULL)
                elm_genlist_item_select_mode_set(ugd->busy_wfd_item, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
        __FUNC_EXIT__;
        return 0;
@@ -1119,16 +1098,16 @@ int _create_available_dev_genlist(void *data)
 
        if (ugd->conn_wfd_item != NULL) {
                last_item = ugd->conn_wfd_item;
-               for (i=0; i<ugd->gl_connected_peer_cnt; i++) {
+               for (i = 0; i < ugd->gl_connected_peer_cnt; i++)
                        last_item = elm_genlist_item_next_get(last_item);
-               }
+
        } else {
                last_item = ugd->device_name_item;
        }
 
        ugd->avlbl_wfd_item = elm_genlist_item_insert_after(ugd->genlist, &title_available_itc, (void *)ugd, NULL,
                last_item, ELM_GENLIST_ITEM_NONE, NULL, NULL);
-       if(ugd->avlbl_wfd_item != NULL) {
+       if (ugd->avlbl_wfd_item != NULL) {
                elm_genlist_item_select_mode_set(ugd->avlbl_wfd_item, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
                elm_genlist_item_update(ugd->avlbl_wfd_item);
        }
@@ -1148,7 +1127,7 @@ static int _create_multi_connect_dev_genlist(void *data)
 
        ugd->multi_connect_wfd_item = elm_genlist_item_insert_after(ugd->genlist, &title_multi_connect_itc, (void *)ugd,
                                                                NULL, ugd->device_name_item, ELM_GENLIST_ITEM_NONE, NULL, NULL);
-       if(ugd->multi_connect_wfd_item != NULL)
+       if (ugd->multi_connect_wfd_item != NULL)
                elm_genlist_item_select_mode_set(ugd->multi_connect_wfd_item, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
        __FUNC_EXIT__;
        return 0;
@@ -1166,7 +1145,7 @@ int _create_connected_dev_genlist(void *data)
 
        ugd->conn_wfd_item = elm_genlist_item_insert_after(ugd->genlist, &title_conn_itc, (void *)ugd, NULL,
                ugd->device_name_item, ELM_GENLIST_ITEM_NONE, NULL, NULL);
-       if(ugd->conn_wfd_item != NULL) {
+       if (ugd->conn_wfd_item != NULL) {
                elm_genlist_item_select_mode_set(ugd->conn_wfd_item, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
                elm_genlist_item_update(ugd->conn_wfd_item);
        }
@@ -1189,21 +1168,21 @@ int _create_connected_failed_dev_genlist(void *data)
 
        if (ugd->avlbl_wfd_item != NULL) {
                last_item = ugd->avlbl_wfd_item;
-               for (i=0; i<ugd->gl_available_peer_cnt; i++) {
+               for (i = 0; i < ugd->gl_available_peer_cnt; i++)
                        last_item = elm_genlist_item_next_get(last_item);
-               }
+
        } else if (ugd->conn_wfd_item != NULL) {
                last_item = ugd->conn_wfd_item;
-               for (i=0; i<ugd->gl_connected_peer_cnt; i++) {
+               for (i = 0; i < ugd->gl_connected_peer_cnt; i++)
                        last_item = elm_genlist_item_next_get(last_item);
-               }
+
        } else {
                last_item = ugd->device_name_item;
        }
 
        ugd->conn_failed_wfd_item = elm_genlist_item_insert_after(ugd->genlist, &title_conn_failed_itc, (void *)ugd,
                                                                NULL, last_item, ELM_GENLIST_ITEM_NONE, NULL, NULL);
-       if(ugd->conn_failed_wfd_item != NULL)
+       if (ugd->conn_failed_wfd_item != NULL)
                elm_genlist_item_select_mode_set(ugd->conn_failed_wfd_item, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
        __FUNC_EXIT__;
        return 0;
@@ -1226,9 +1205,8 @@ static Eina_Bool _connect_failed_peers_display_cb(void *user_data)
                return ECORE_CALLBACK_CANCEL;
        }
 
-       if (ugd->avlbl_wfd_item == NULL) {
+       if (ugd->avlbl_wfd_item == NULL)
                _create_available_dev_genlist(ugd);
-       }
 
        if (ugd->wfd_discovery_status == WIFI_DIRECT_DISCOVERY_BACKGROUND) {
                DBG(LOG_INFO, "Background mode\n");
@@ -1238,9 +1216,8 @@ static Eina_Bool _connect_failed_peers_display_cb(void *user_data)
 
        /* check the timeout, if not timeout, keep the cb */
        interval = time(NULL) - ugd->last_display_time;
-       if (interval < MAX_DISPLAY_TIME_OUT) {
+       if (interval < MAX_DISPLAY_TIME_OUT)
                return ECORE_CALLBACK_RENEW;
-       }
 
        if (ugd->is_paused == false) {
                DBG(LOG_INFO, "Not Paused");
@@ -1279,9 +1256,9 @@ void wfd_ug_view_free_peer(device_type_s *gl_peers_start)
 void wfd_check_gl_conn_peers(struct ug_data *ugd)
 {
        __FUNC_ENTER__;
-       if (ugd->gl_connected_peer_cnt == 0) {
+       if (ugd->gl_connected_peer_cnt == 0)
                WFD_IF_DEL_ITEM(ugd->conn_wfd_item);
-       }
+
        __FUNC_EXIT__;
 }
 
@@ -1324,9 +1301,8 @@ void wfd_ug_view_free_peers(void *data)
                ugd->gl_mul_conn_peers_start = NULL;
        }
 
-       if (ugd->gl_connected_peer_cnt == 0) {
+       if (ugd->gl_connected_peer_cnt == 0)
                WFD_IF_DEL_ITEM(ugd->conn_wfd_item);
-       }
 
        if (ugd->display_timer != NULL) {
                ecore_timer_del(ugd->display_timer);
@@ -1350,7 +1326,7 @@ void wfd_ug_update_toolbar(struct ug_data *ugd)
        wfd_refresh_wifi_direct_state(ugd);
        if (ugd->wfd_status == WIFI_DIRECT_STATE_CONNECTING) {
                DBG(LOG_INFO, "WIFI_DIRECT_STATE_CONNECTING\n");
-               ifugd->multi_connect_wfd_item != NULL) {
+               if (ugd->multi_connect_wfd_item != NULL) {
                        DBG(LOG_INFO, "multi_connect_toolbar_item\n");
                        btn = elm_button_add(ugd->button_layout);
                        /* Use "bottom" style button */
@@ -1425,9 +1401,8 @@ void wfd_ug_view_init_genlist(void *data, bool is_free_all_peers)
        int no_of_available_dev = 0;
        int no_of_conn_failed_dev = 0;
 
-       if (is_free_all_peers) {
+       if (is_free_all_peers)
                wfd_ug_view_free_peers(ugd);
-       }
 
        if (ugd->gl_failed_peers_start != NULL) {
                DBG(LOG_INFO, "These are failed peers, must clear them");
@@ -1490,16 +1465,16 @@ void delete_not_alive_peers(struct ug_data *ugd, device_type_s **start_pos, int
        device_type_s *peer_tmp = NULL;
        while (peer != NULL) {
                peer_tmp = peer->next;
-               if(peer->is_alive == false) {
+               if (peer->is_alive == false)
                         free_gl_peer(start_pos, peer->mac_addr, cnt);
-               }
+
                peer = peer_tmp;
        }
 
 //     wfd_check_gl_available_peers(ugd);
-       if (ugd->gl_busy_peer_cnt == 0) {
+       if (ugd->gl_busy_peer_cnt == 0)
                WFD_IF_DEL_ITEM(ugd->busy_wfd_item);
-       }
+
        __FUNC_EXIT__;
        return;
 }
@@ -1533,14 +1508,14 @@ Elm_Object_Item *get_insert_postion(device_type_s *peer, Elm_Object_Item *pre_it
        Elm_Object_Item *head = elm_genlist_item_next_get(pre_item);
        Elm_Object_Item *item = NULL;
 
-       if(peer_cnt == 0) {
+       if (peer_cnt == 0) {
                DBG(LOG_INFO, "first peer [%s] would be added", peer->ssid);
                return pre_item;
        }
 
        peer_ite = (device_type_s *)elm_object_item_data_get(head);
-       if(peer_ite->gl_item != NULL) {
-               for(i=0; i < peer_cnt; i++) {
+       if (peer_ite->gl_item != NULL) {
+               for (i = 0; i < peer_cnt; i++) {
                        if (strcasecmp(peer_ite->ssid, peer->ssid) > 0) {
                                /* if peer_ite is greater than peer, return previous item */
                                __FUNC_EXIT__;
@@ -1578,11 +1553,11 @@ int insert_gl_item(Evas_Object *genlist, Elm_Object_Item *item, Elm_Gen_Item_Cla
        memcpy(peer, peer_for_insert, sizeof(device_type_s));
        peer->next = NULL;
 
-       if(*start_pos == NULL) {
+       if (*start_pos == NULL) {
                *start_pos = peer;
        } else {
                peer_ite = *start_pos;
-               while(peer_ite->next != NULL) {
+               while (peer_ite->next != NULL) {
                        /* move pointer to the last peer */
                        peer_ite = peer_ite->next;
                }
@@ -1592,9 +1567,9 @@ int insert_gl_item(Evas_Object *genlist, Elm_Object_Item *item, Elm_Gen_Item_Cla
        peer->is_alive = true;
        peer->gl_item = elm_genlist_item_insert_after(genlist, itc, (void *)peer, NULL, item,
                ELM_GENLIST_ITEM_NONE, callback, (void *)peer);
-       if (callback == NULL && peer->gl_item != NULL) {
+       if (callback == NULL && peer->gl_item != NULL)
                elm_genlist_item_select_mode_set(peer->gl_item, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
-       }
+
        __FUNC_EXIT__;
        return 0;
 }
@@ -1615,13 +1590,11 @@ void wfd_ug_update_connected_peers(void *data)
        Elm_Object_Item *item = NULL;
 
        res = wifi_direct_is_group_owner(&is_group_owner);
-       if (res == WIFI_DIRECT_ERROR_NONE) {
+       if (res == WIFI_DIRECT_ERROR_NONE)
                DBG(LOG_INFO, "is_group_owner=[%d]", is_group_owner);
-       }
 
-       if (!ugd->conn_wfd_item) {
+       if (!ugd->conn_wfd_item)
                _create_connected_dev_genlist(ugd);
-       }
 
        for (i = 0; i < ugd->raw_connected_peer_cnt; i++) {
                if (find_peer_in_glist(ugd->gl_conn_peers_start, ugd->raw_connected_peers[i].mac_addr) == NULL) {
@@ -1644,9 +1617,9 @@ void wfd_ug_update_connected_peers(void *data)
                                ugd->gl_connected_peer_cnt);
                        res = insert_gl_item(ugd->genlist, item, &peer_conn_itc, &ugd->gl_conn_peers_start,
                                        &ugd->raw_connected_peers[i], NULL);
-                       if (res != 0) {
+                       if (res != 0)
                                break;
-                       }
+
                        ugd->gl_connected_peer_cnt++;
                }
        }
@@ -1701,9 +1674,9 @@ void wfd_ug_view_update_multiconn_peers(void *data)
                                        ugd->multi_connect_wfd_item, ugd->gl_multi_connect_peer_cnt);
                                res = insert_gl_item(ugd->genlist, item, &peer_itc, &ugd->gl_mul_conn_peers_start,
                                                &ugd->raw_multi_selected_peers[i], NULL);
-                               if (res != 0) {
+                               if (res != 0)
                                        break;
-                               }
+
                                ugd->gl_multi_connect_peer_cnt++;
                        }
                }
@@ -1748,15 +1721,15 @@ void wfd_ug_update_available_peers(void *data)
        }
 
        if (no_of_conn_dev == 0 || is_group_owner == TRUE) {
-               if (ugd->avlbl_wfd_item == NULL) {
+               if (ugd->avlbl_wfd_item == NULL)
                        _create_available_dev_genlist(ugd);
-               }
+
 
                for (iterator = ugd->raw_discovered_peer_list; iterator; iterator = iterator->next) {
                        /* Not include the device which is connected with me */
-                       if (__wfd_is_device_connected_with_me(ugd, (device_type_s *)iterator->data)) {
+                       if (__wfd_is_device_connected_with_me(ugd, (device_type_s *)iterator->data))
                                continue;
-                       }
+
                        if (!__wfd_is_device_busy(ugd, (device_type_s *)iterator->data) &&
                                ((device_type_s *)iterator->data)->conn_status != PEER_CONN_STATUS_FAILED_TO_CONNECT &&
                                ((device_type_s *)iterator->data)->conn_status != PEER_CONN_STATUS_CONNECTED) {
@@ -1769,14 +1742,12 @@ void wfd_ug_update_available_peers(void *data)
                                /* free busy gl peer, which is available now */
                                if (find_peer_in_glist(ugd->gl_busy_peers_start, ((device_type_s *)iterator->data)->mac_addr) != NULL) {
                                        free_gl_peer(&ugd->gl_busy_peers_start, ((device_type_s *)iterator->data)->mac_addr, &ugd->gl_busy_peer_cnt);
-                                       if (ugd->gl_busy_peer_cnt == 0) {
+                                       if (ugd->gl_busy_peer_cnt == 0)
                                                WFD_IF_DEL_ITEM(ugd->busy_wfd_item);
-                                       }
                                }
 
-                               if (find_peer_in_glist(ugd->gl_failed_peers_start, (const char *)((device_type_s *)iterator->data)->mac_addr) != NULL) {
+                               if (find_peer_in_glist(ugd->gl_failed_peers_start, (const char *)((device_type_s *)iterator->data)->mac_addr) != NULL)
                                        continue;
-                               }
 
                                peer = find_peer_in_glist(ugd->gl_avlb_peers_start, (const char *)((device_type_s *)iterator->data)->mac_addr);
                                if (peer == NULL) {
@@ -1784,9 +1755,9 @@ void wfd_ug_update_available_peers(void *data)
                                                ugd->avlbl_wfd_item, ugd->gl_available_peer_cnt);
                                        res = insert_gl_item(ugd->genlist, item, &peer_itc, &ugd->gl_avlb_peers_start,
                                                        (device_type_s *)iterator->data, _gl_peer_sel);
-                                       if (res != 0) {
+                                       if (res != 0)
                                                break;
-                                       }
+
                                        ugd->gl_available_peer_cnt++;
                                } else if (no_of_conn_dev > 0 && ((device_type_s *)iterator->data)->is_group_owner == TRUE) {
                                        /* if peer is GO, free it */
@@ -1801,15 +1772,14 @@ void wfd_ug_update_available_peers(void *data)
        wfd_check_gl_conn_peers(ugd);
 
        if (no_of_conn_dev == 0 && no_of_busy_dev > 0) {
-               if (ugd->busy_wfd_item == NULL) {
+               if (ugd->busy_wfd_item == NULL)
                        _create_busy_dev_list(ugd);
-               }
 
                for (iterator = ugd->raw_discovered_peer_list; iterator; iterator = iterator->next) {
                        /* Not include the device which is connected with me */
-                       if (__wfd_is_device_connected_with_me(ugd, (device_type_s *)iterator->data)) {
+                       if (__wfd_is_device_connected_with_me(ugd, (device_type_s *)iterator->data))
                                continue;
-                       }
+
                        if (__wfd_is_device_busy(ugd, (device_type_s *)iterator->data) == TRUE) {
                                if (find_peer_in_glist(ugd->gl_busy_peers_start, ((device_type_s *)iterator->data)->mac_addr) == NULL) {
                                        if (find_peer_in_glist(ugd->gl_avlb_peers_start,
@@ -1822,9 +1792,9 @@ void wfd_ug_update_available_peers(void *data)
                                                ugd->gl_busy_peer_cnt);
                                        res = insert_gl_item(ugd->genlist, item, &peer_busy_itc, &ugd->gl_busy_peers_start,
                                                        (device_type_s *)iterator->data, _gl_busy_peer_sel);
-                                       if (res != 0) {
+                                       if (res != 0)
                                                break;
-                                       }
+
                                        ugd->gl_busy_peer_cnt++;
                                }
                        }
@@ -1851,9 +1821,8 @@ void wfd_ug_update_failed_peers(void *data)
        __wfd_is_any_device_connect_failed(ugd, &no_of_conn_failed_dev);
        DBG(LOG_INFO, "conn_failed_dev=[%d]", no_of_conn_failed_dev);
 
-       if (no_of_conn_failed_dev == 0) {
+       if (no_of_conn_failed_dev == 0)
                return;
-       }
 
        /* add timer for disappearing failed peers after N secs */
        if (NULL == ugd->display_timer) {
@@ -1892,11 +1861,11 @@ void _onoff_changed_cb(void *data, Evas_Object *obj, void *event_info)
        WFD_RET_IF(ugd == NULL, "Incorrect parameter(NULL)\n");
        WFD_RET_IF(ugd->on_off_check == NULL, "on_off_check(NULL)\n");
        wfd_refresh_wifi_direct_state(ugd);
-       if(ugd->device_name_item != NULL)
+       if (ugd->device_name_item != NULL)
                elm_genlist_item_update(ugd->device_name_item);
 
        elm_object_disabled_set(ugd->on_off_check, TRUE);
-       if(ugd->disconnect_btn) {
+       if (ugd->disconnect_btn) {
                Evas_Object *content;
                content = elm_object_part_content_unset(ugd->button_layout, "button.next");
                WFD_IF_DEL_OBJ(content);
@@ -1932,16 +1901,15 @@ void wfd_ug_refresh_on_off_check(struct ug_data *ugd)
 
        wfd_refresh_wifi_direct_state(ugd);
        if (ugd->wfd_status == WIFI_DIRECT_STATE_DEACTIVATING ||
-               ugd->wfd_status == WIFI_DIRECT_STATE_ACTIVATING ) {
+               ugd->wfd_status == WIFI_DIRECT_STATE_ACTIVATING)
                elm_object_disabled_set(ugd->on_off_check, TRUE);
-       } else {
+       else
                elm_object_disabled_set(ugd->on_off_check, FALSE);
-       }
-       if (ugd->wfd_status > WIFI_DIRECT_STATE_ACTIVATING) {
+
+       if (ugd->wfd_status > WIFI_DIRECT_STATE_ACTIVATING)
                elm_check_state_set(ugd->on_off_check, TRUE);
-       } else {
+       else
                elm_check_state_set(ugd->on_off_check, FALSE);
-       }
 
        __FUNC_EXIT__;
 }
@@ -2023,9 +1991,8 @@ void create_wfd_ug_view(void *data)
 
        wifi_direct_initialize();
        wifi_direct_get_state(&ugd->wfd_status);
-       if (ugd->wfd_status > WIFI_DIRECT_STATE_DEACTIVATING) {
+       if (ugd->wfd_status > WIFI_DIRECT_STATE_DEACTIVATING)
                scan_button_create(ugd);
-       }
 
        if (ugd->view_type && g_strcmp0(D_(ugd->view_type), D_("IDS_WIFI_BUTTON_MULTI_CONNECT")) == 0) {
                int ret = 0;