Fixed: Coding rule issues.

[apps/native/ug-wifi-direct.git] / ug-wifidirect / src / wfd_ug_multiconnect_view.c

diff --git a/ug-wifidirect/src/wfd_ug_multiconnect_view.c b/ug-wifidirect/src/wfd_ug_multiconnect_view.c
index a849c2f..989a6f7 100644 (file)
--- a/ug-wifidirect/src/wfd_ug_multiconnect_view.c
+++ b/ug-wifidirect/src/wfd_ug_multiconnect_view.c
@@ -143,9 +143,8 @@ void wfd_free_multi_selected_peers(void *data)
        wifi_direct_destroy_group();
 
        /* release the selected peers */
-       for (i = 0; i < ugd->raw_multi_selected_peer_cnt; i++) {
+       for (i = 0; i < ugd->raw_multi_selected_peer_cnt; i++)
                memset(&ugd->raw_multi_selected_peers[i], 0x00, sizeof(device_type_s));
-       }
 
        ugd->raw_multi_selected_peer_cnt = 0;
        ugd->multi_connect_mode = WFD_MULTI_CONNECT_MODE_NONE;
@@ -174,28 +173,26 @@ int wfd_stop_multi_connect(void *data)
        wfd_ug_view_refresh_glitem(ugd->multi_connect_wfd_item);
        wfd_ug_update_toolbar(ugd);
 
-       if (ugd->gl_connected_peer_cnt > 0) {
+       if (ugd->gl_connected_peer_cnt > 0)
                wfd_client_set_p2p_group_owner_intent(7);
-       } else {
+       else
                wifi_direct_destroy_group();
-       }
 
-       if(ugd->timer_multi_reset > 0) {
+       if (ugd->timer_multi_reset > 0)
                g_source_remove(ugd->timer_multi_reset);
-       }
+
        /* after 30s, remove the failed peers */
        ugd->timer_multi_reset = g_timeout_add(30000 /*ms*/,
                __wfd_multi_connect_reset_cb, ugd);
 
-       if (ugd->g_source_multi_connect_next > 0) {
+       if (ugd->g_source_multi_connect_next > 0)
                g_source_remove(ugd->g_source_multi_connect_next);
-       }
+
        ugd->g_source_multi_connect_next = 0;
 
        /*when multi-connect end and auto_exit is true, exit UG*/
-       if (ugd->is_auto_exit) {
+       if (ugd->is_auto_exit)
                _wfd_ug_auto_exit(ugd);
-       }
 
        __FUNC_EXIT__;
        return 0;
@@ -239,8 +236,8 @@ void wfd_sort_multi_selected_peers(struct ug_data *ugd)
        int j = 0;
        device_type_s peer;
 
-       for(i = 0; i < ugd->raw_multi_selected_peer_cnt; i++) {
-               for(j = 0; j < ugd->raw_multi_selected_peer_cnt-i-1; j++) {
+       for (i = 0; i < ugd->raw_multi_selected_peer_cnt; i++) {
+               for (j = 0; j < ugd->raw_multi_selected_peer_cnt-i-1; j++) {
                        if (strcasecmp(ugd->raw_multi_selected_peers[j].ssid, ugd->raw_multi_selected_peers[j + 1].ssid) > 0) {
                                peer = ugd->raw_multi_selected_peers[j];
                                ugd->raw_multi_selected_peers[j] = ugd->raw_multi_selected_peers[j + 1];
@@ -264,9 +261,9 @@ gboolean wfd_multi_connect_next_cb(void *data)
        int res;
 
        // Reset g_source handler..
-       if (ugd->g_source_multi_connect_next > 0) {
+       if (ugd->g_source_multi_connect_next > 0)
                g_source_remove(ugd->g_source_multi_connect_next);
-       }
+
        ugd->g_source_multi_connect_next = 0;
 
        if (ugd->raw_multi_selected_peer_cnt > 0) {
@@ -315,9 +312,9 @@ void _connect_btn_cb(void *data, Evas_Object *obj, void *event_info)
 
        peer = ugd->multi_conn_dev_list_start;
        while (peer != NULL) {
-               if(peer->dev_sel_state) {
+               if (peer->dev_sel_state)
                        count++;
-               }
+
                peer = peer->next;
        }
        DBG(LOG_INFO, "MultiSelected Peer Count: %d", count);
@@ -334,7 +331,7 @@ void _connect_btn_cb(void *data, Evas_Object *obj, void *event_info)
        peer = ugd->multi_conn_dev_list_start;
        count = 0;
        while (peer != NULL) {
-               if(peer->dev_sel_state) {
+               if (peer->dev_sel_state) {
                        DBG(LOG_INFO, "peer_name[%s] select state[%d]", peer->ssid, peer->dev_sel_state);
                        memcpy(&ugd->raw_multi_selected_peers[count], peer, sizeof(device_type_s));
                        ugd->raw_multi_selected_peers[count].conn_status = PEER_CONN_STATUS_WAIT_FOR_CONNECT;
@@ -396,26 +393,25 @@ static void _wfd_gl_multi_sel_cb(void *data, Evas_Object *obj, void *event_info)
 
        ugd->is_multi_check_all_selected = TRUE;
 
-       if(event_info != NULL) {
+       if (event_info != NULL) {
                Evas_Object *content = elm_object_item_part_content_get(event_info, "elm.icon.2");
                chk_box = elm_object_part_content_get(content, "elm.swallow.content");
                state = elm_check_state_get(chk_box);
                DBG(LOG_INFO, "state = %d \n", state);
-               if (chk_box ==  NULL) {
+               if (chk_box ==  NULL)
                        DBG(LOG_INFO, "Check box is null\n");
-               }
        }
 
        object_type = evas_object_type_get(obj);
         if (g_strcmp0(object_type, "elm_genlist") == 0) {
                Elm_Object_Item *item = (Elm_Object_Item *)event_info;
-               if (item) {
+               if (item)
                        elm_genlist_item_selected_set(item, EINA_FALSE);
-               }
+
                DBG(LOG_INFO, "State elm_genlist %d\n", state);
                elm_check_state_set(chk_box, !state);
                peer->dev_sel_state = !state;
-       } else if (g_strcmp0(object_type, "elm_check") == 0){
+       } else if (g_strcmp0(object_type, "elm_check") == 0) {
                DBG(LOG_INFO, "elm_check state; %d\n", peer->dev_sel_state);
                peer->dev_sel_state = !peer->dev_sel_state;
        }
@@ -426,7 +422,7 @@ static void _wfd_gl_multi_sel_cb(void *data, Evas_Object *obj, void *event_info)
 
        peer = ugd->multi_conn_dev_list_start;
        while (peer != NULL) {
-               if(peer->dev_sel_state) {
+               if (peer->dev_sel_state) {
                        is_sel = TRUE;
                        sel_count++;
                } else {
@@ -474,9 +470,8 @@ static char *_wfd_gl_device_label_get(void *data, Evas_Object *obj, const char *
        device_type_s *peer = (device_type_s *)data;
        char *ssid;
 
-       if (NULL == peer) {
+       if (NULL == peer)
                return NULL;
-       }
 
        if (!strcmp("elm.text", part)) {
                if (strlen(peer->ssid) != 0) {
@@ -581,9 +576,9 @@ static Evas_Object *_wfd_gl_device_icon_get(void *data, Evas_Object *obj, const
                icon = elm_check_add(icon_layout);
                elm_object_style_set(icon, "default/genlist");
                evas_object_propagate_events_set(icon, EINA_FALSE);
-               if (peer->dev_sel_state == EINA_TRUE) {
+               if (peer->dev_sel_state == EINA_TRUE)
                        elm_check_state_set(icon, EINA_TRUE);
-               }
+
                evas_object_smart_callback_add(icon,
                        "changed", _wfd_gl_multi_sel_cb, (void *)data);
                elm_layout_content_set(icon_layout, "elm.swallow.content", icon);
@@ -615,7 +610,7 @@ static void _gl_unrealized(void *data, Evas_Object *obj, void *event_info)
        if (ugd->gl_available_dev_cnt_at_multiconn_view > 0) {
                peer = ugd->multi_conn_dev_list_start;
                while (peer != NULL) {
-                       if(peer->dev_sel_state) {
+                       if (peer->dev_sel_state) {
                                is_sel = TRUE;
                                sel_count++;
                        }
@@ -649,9 +644,8 @@ int wfd_free_multiconnect_device(struct ug_data *ugd)
 {
        __FUNC_ENTER__;
 
-       if (ugd->multiconn_view_genlist == NULL) {
+       if (ugd->multiconn_view_genlist == NULL)
                return 0;
-       }
 
        ugd->gl_available_dev_cnt_at_multiconn_view = 0;
        if (ugd->multi_conn_dev_list_start != NULL) {
@@ -677,7 +671,7 @@ Evas_Object *_create_no_device_multiconnect_genlist(struct ug_data *ugd)
 
        if (!ugd->mcview_nodevice_item) {
                ugd->mcview_nodevice_item = elm_genlist_item_append(ugd->multiconn_view_genlist, &noitem_itc, (void *)ugd, NULL, ELM_GENLIST_ITEM_NONE, NULL, NULL);
-               if(ugd->mcview_nodevice_item != NULL)
+               if (ugd->mcview_nodevice_item != NULL)
                        elm_genlist_item_select_mode_set(ugd->mcview_nodevice_item , ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
        }
 
@@ -701,27 +695,24 @@ int wfd_update_multiconnect_device(struct ug_data *ugd, bool is_free_all_peers)
        int res = 0;
 
        genlist = ugd->multiconn_view_genlist;
-       if (ugd->multiconn_view_genlist == NULL) {
+       if (ugd->multiconn_view_genlist == NULL)
                return 0;
-       }
 
-       if (is_free_all_peers) {
+       if (is_free_all_peers)
                wfd_free_multiconnect_device(ugd);
-       }
 
        GList *iterator = NULL;
        for (iterator = ugd->raw_discovered_peer_list; iterator; iterator = iterator->next) {
                device = (device_type_s *)iterator->data;
-               if (device->is_connected == FALSE && device->is_group_owner == FALSE) {
+               if (device->is_connected == FALSE && device->is_group_owner == FALSE)
                        count++;
-               }
        }
 
        if (count == 0) {
                DBG(LOG_INFO, "There are No peers\n");
                if (!ugd->mcview_title_item) {
                        ugd->mcview_title_item = elm_genlist_item_append(genlist, &multi_view_title_itc, ugd, NULL, ELM_GENLIST_ITEM_NONE, NULL, NULL);
-                       if(ugd->mcview_title_item != NULL)
+                       if (ugd->mcview_title_item != NULL)
                                elm_genlist_item_select_mode_set(ugd->mcview_title_item, ELM_OBJECT_SELECT_MODE_DISPLAY_ONLY);
                } else {
                        wfd_ug_view_refresh_glitem(ugd->mcview_title_item);
@@ -741,9 +732,9 @@ int wfd_update_multiconnect_device(struct ug_data *ugd, bool is_free_all_peers)
                                        item = get_insert_postion(device, ugd->mcview_title_item, ugd->gl_available_dev_cnt_at_multiconn_view);
                                        res = insert_gl_item(genlist, item, &device_itc, &ugd->multi_conn_dev_list_start,
                                                                device, _wfd_gl_multi_sel_cb);
-                                       if (res != 0) {
+                                       if (res != 0)
                                                break;
-                                       }
+
                                        elm_check_state_set(ugd->select_all_icon, EINA_FALSE);
                                        ugd->is_select_all_checked = FALSE;
 
@@ -796,7 +787,7 @@ void wfd_genlist_select_all_check_changed_cb(void *data, Evas_Object * obj, void
        }
 
        state = elm_check_state_get(ugd->select_all_icon);
-       DBG(LOG_INFO, "state = %d",state);
+       DBG(LOG_INFO, "state = %d", state);
        if (state == EINA_TRUE) {
                if (ugd->is_multi_check_all_selected == FALSE) {
                        int sel_count = 0;
@@ -864,15 +855,14 @@ void _multi_scan_btn_cb(void *data, Evas_Object *obj, void *event_info)
        if (ugd->multiconn_conn_btn) {
                peer = ugd->multi_conn_dev_list_start;
                while (peer) {
-                       if(peer->dev_sel_state) {
+                       if (peer->dev_sel_state) {
                                wfd_ug_view_refresh_button(ugd->multiconn_conn_btn, "IDS_WIFI_SK_CONNECT", TRUE);
                                break;
                        }
                        peer = peer->next;
                }
-               if (!peer) {
+               if (!peer)
                        wfd_ug_view_refresh_button(ugd->multiconn_conn_btn, "IDS_WIFI_SK_CONNECT", FALSE);
-               }
        }
 
        btn_text = elm_object_text_get(ugd->multiconn_scan_stop_btn);
@@ -973,7 +963,7 @@ void wfd_create_multiconnect_view(struct ug_data *ugd)
        elm_naviframe_item_pop_cb_set(navi_item, _multiconnect_view_pop_cb, (void *)ugd);
        ugd->select_all_view_genlist = elm_genlist_item_append(genlist,
                        &select_all_multi_connect_itc, ugd, NULL, ELM_GENLIST_ITEM_NONE,
-                       wfd_genlist_select_all_check_changed_cb, (void *)ugd );
+                       wfd_genlist_select_all_check_changed_cb, (void *)ugd);
 
 
        btn1 = elm_button_add(ugd->multiconn_layout);
@@ -987,7 +977,7 @@ void wfd_create_multiconnect_view(struct ug_data *ugd)
                                "IDS_WIFI_SK_STOP");
        }
        elm_object_part_content_set(ugd->multiconn_layout, "button.prev", btn1);
-       evas_object_smart_callback_add(btn1, "clicked",_multi_scan_btn_cb, (void *)ugd);
+       evas_object_smart_callback_add(btn1, "clicked", _multi_scan_btn_cb, (void *)ugd);
        evas_object_show(btn1);
        ugd->multiconn_scan_stop_btn = btn1;
 
@@ -996,7 +986,7 @@ void wfd_create_multiconnect_view(struct ug_data *ugd)
        elm_object_domain_translatable_text_set(btn2, PACKAGE,
                        "IDS_WIFI_SK_CONNECT");
        elm_object_part_content_set(ugd->multiconn_layout, "button.next", btn2);
-       evas_object_smart_callback_add(btn2, "clicked",_connect_btn_cb, (void *)ugd);
+       evas_object_smart_callback_add(btn2, "clicked", _connect_btn_cb, (void *)ugd);
        evas_object_show(btn2);
        ugd->multiconn_conn_btn = btn2;
        elm_object_disabled_set(ugd->multiconn_conn_btn, EINA_TRUE);