Fixed: Coding rule issues.

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

diff --git a/ug-wifidirect/src/wfd_ug_genlist.c b/ug-wifidirect/src/wfd_ug_genlist.c
index a1a4f26..18fabf1 100755 (executable)
--- a/ug-wifidirect/src/wfd_ug_genlist.c
+++ b/ug-wifidirect/src/wfd_ug_genlist.c
@@ -72,9 +72,8 @@ static char *_gl_wfd_onoff_text_get(void *data, Evas_Object *obj,
                DBG(LOG_INFO, "%s", ugd->dev_name);
                if (ugd->dev_name) {
                        dev_name = elm_entry_utf8_to_markup(ugd->dev_name);
-                       if (NULL == dev_name) {
+                       if (NULL == dev_name)
                                DBG(LOG_ERROR, "elm_entry_utf8_to_markup failed.\n");
-                       }
                }
        }
 
@@ -82,7 +81,7 @@ static char *_gl_wfd_onoff_text_get(void *data, Evas_Object *obj,
        return dev_name;
 }
 
-static Evas_Object *_gl_wfd_onoff_content_get(void *data,Evas_Object *obj,
+static Evas_Object *_gl_wfd_onoff_content_get(void *data, Evas_Object *obj,
                const char *part)
 {
        __FUNC_ENTER__;
@@ -97,11 +96,11 @@ static Evas_Object *_gl_wfd_onoff_content_get(void *data,Evas_Object *obj,
                elm_layout_theme_set(icon, "layout", "list/C/type.3", "default");
 
                /* Wi-Fi on indication button */
-               btn= elm_check_add(icon);
+               btn = elm_check_add(icon);
                elm_object_style_set(btn, "on&off");
                elm_check_state_set(btn, ugd->wfd_onoff);
                evas_object_propagate_events_set(btn, EINA_FALSE);
-               evas_object_smart_callback_add(btn, "changed",_onoff_changed_cb, ugd);
+               evas_object_smart_callback_add(btn, "changed", _onoff_changed_cb, ugd);
                ugd->on_off_check = btn;
                evas_object_size_hint_align_set(btn, EVAS_HINT_FILL, EVAS_HINT_FILL);
                evas_object_size_hint_weight_set(btn, EVAS_HINT_EXPAND,
@@ -173,9 +172,9 @@ static char *_gl_title_label_get(void *data, Evas_Object *obj, const char *part)
 
        WFD_RETV_IF(data == NULL, NULL, "Incorrect parameter(NULL)\n");
        DBG(LOG_INFO, "%s", part);
-       if (!strcmp("elm.text", part)) {
+       if (!strcmp("elm.text", part))
                return g_strdup(D_("IDS_WIFI_HEADER_AVAILABLE_DEVICES_ABB"));
-       }
+
        __FUNC_EXIT__;
        return NULL;
 }
@@ -188,9 +187,8 @@ static char *_gl_title_no_device_label_get(void *data, Evas_Object *obj,
 
        WFD_RETV_IF(data == NULL, NULL, "Incorrect parameter(NULL)\n");
        DBG(LOG_INFO, "%s", part);
-       if (!strcmp("elm.text", part)) {
+       if (!strcmp("elm.text", part))
                return g_strdup(D_("IDS_WIFI_HEADER_AVAILABLE_DEVICES_ABB"));
-       }
 
        __FUNC_EXIT__;
        return NULL;
@@ -201,9 +199,8 @@ static char *_gl_multi_title_label_get(void *data, Evas_Object *obj, const char
        __FUNC_ENTER__;
 
        WFD_RETV_IF(data == NULL, NULL, "Incorrect parameter(NULL)\n");
-       if (!strcmp("elm.text", part)) {
+       if (!strcmp("elm.text", part))
                return g_strdup(D_("IDS_WIFI_HEADER_AVAILABLE_DEVICES_ABB"));
-       }
 
        __FUNC_EXIT__;
        return NULL;
@@ -238,9 +235,8 @@ static char *_gl_available_title_label_get(void *data, Evas_Object *obj, const c
                return NULL;
        }
        DBG(LOG_INFO, "available- %s", part);
-       if (!strcmp("elm.text", part)) {
+       if (!strcmp("elm.text", part))
                return g_strdup(D_("IDS_WIFI_HEADER_AVAILABLE_DEVICES_ABB"));
-       }
 
        __FUNC_EXIT__;
        return NULL;
@@ -323,11 +319,10 @@ static char *_gl_peer_label_get(void *data, Evas_Object *obj, const char *part)
                        g_strlcpy(buf, D_("IDS_WIFI_BODY_CONNECTING_ING"), WFD_GLOBALIZATION_STR_LENGTH);
                        break;
                case PEER_CONN_STATUS_CONNECTED:
-                       if (peer->is_group_owner == FALSE) {
+                       if (peer->is_group_owner == FALSE)
                                g_strlcpy(buf, D_("IDS_COM_BODY_CONNECTED_M_STATUS"), WFD_GLOBALIZATION_STR_LENGTH);
-                       } else {
+                       else
                                g_strlcpy(buf, D_("IDS_CHATON_BODY_AVAILABLE"), WFD_GLOBALIZATION_STR_LENGTH);
-                       }
                        break;
                case PEER_CONN_STATUS_FAILED_TO_CONNECT:
                        g_strlcpy(buf, D_("IDS_CHATON_HEADER_CONNECTION_FAILED_ABB2"), WFD_GLOBALIZATION_STR_LENGTH);
@@ -403,15 +398,14 @@ static Evas_Object *_gl_peer_icon_get(void *data, Evas_Object *obj, const char *
                        img_name = WFD_ICON_DEVICE_DISPLAY;
                        break;
                case WFD_DEVICE_TYPE_MM_DEVICES:
-                       if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_STB) {
+                       if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_STB)
                                img_name = WFD_ICON_DEVICE_STB;
-                       } else if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_MS_MA_ME) {
+                       else if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_MS_MA_ME)
                                img_name = WFD_ICON_DEVICE_DONGLE;
-                       } else if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_PVP) {
+                       else if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_PVP)
                                img_name = WFD_ICON_DEVICE_BD;
-                       } else {
+                       else
                                img_name = WFD_ICON_DEVICE_MULTIMEDIA;
-                       }
                        break;
                case WFD_DEVICE_TYPE_GAME_DEVICES:
                        img_name = WFD_ICON_DEVICE_GAMING;
@@ -420,11 +414,10 @@ static Evas_Object *_gl_peer_icon_get(void *data, Evas_Object *obj, const char *
                        img_name = WFD_ICON_DEVICE_TELEPHONE;
                        break;
                case WFD_DEVICE_TYPE_AUDIO:
-                       if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_AUDIO_TUNER) {
+                       if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_AUDIO_TUNER)
                                img_name = WFD_ICON_DEVICE_HOME_THEATER;
-                       } else {
+                       else
                                img_name = WFD_ICON_DEVICE_HEADSET;
-                       }
                        break;
                default:
                        img_name = WFD_ICON_DEVICE_UNKNOWN;
@@ -515,15 +508,14 @@ static Evas_Object *_gl_conn_peer_icon_get(void *data, Evas_Object *obj, const c
                        img_name = WFD_ICON_DEVICE_DISPLAY;
                        break;
                case WFD_DEVICE_TYPE_MM_DEVICES:
-                       if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_STB) {
+                       if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_STB)
                                img_name = WFD_ICON_DEVICE_STB;
-                       } else if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_MS_MA_ME) {
+                       else if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_MS_MA_ME)
                                img_name = WFD_ICON_DEVICE_DONGLE;
-                       } else if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_PVP) {
+                       else if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_MULTIMEDIA_PVP)
                                img_name = WFD_ICON_DEVICE_BD;
-                       } else {
+                       else
                                img_name = WFD_ICON_DEVICE_MULTIMEDIA;
-                       }
                        break;
                case WFD_DEVICE_TYPE_GAME_DEVICES:
                        img_name = WFD_ICON_DEVICE_GAMING;
@@ -532,11 +524,10 @@ static Evas_Object *_gl_conn_peer_icon_get(void *data, Evas_Object *obj, const c
                        img_name = WFD_ICON_DEVICE_TELEPHONE;
                        break;
                case WFD_DEVICE_TYPE_AUDIO:
-                       if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_AUDIO_TUNER) {
+                       if (peer->sub_category == WIFI_DIRECT_SECONDARY_DEVICE_TYPE_AUDIO_TUNER)
                                img_name = WFD_ICON_DEVICE_HOME_THEATER;
-                       } else {
+                       else
                                img_name = WFD_ICON_DEVICE_HEADSET;
-                       }
                        break;
                default:
                        img_name = WFD_ICON_DEVICE_UNKNOWN;
@@ -576,10 +567,10 @@ static char *_gl_noitem_text_get(void *data, Evas_Object *obj, const char *part)
                return NULL;
        }
 
-       DBG(LOG_INFO,"part = %s",part);
-       if (!strcmp("elm.text", part)) {
+       DBG(LOG_INFO, "part = %s", part);
+       if (!strcmp("elm.text", part))
                return g_strdup(D_("IDS_BT_BODY_NO_DEVICES_FOUND_ABB"));
-       }
+
        __FUNC_EXIT__;
        return NULL;
 }
@@ -599,10 +590,9 @@ static char *_gl_conn_dev_title_label_get(void *data, Evas_Object *obj, const ch
                DBG(LOG_ERROR, "Incorrect parameter(NULL)\n");
                return NULL;
        }
-       DBG(LOG_INFO,"part = %s",part);
-       if (!strcmp("elm.text", part)) {
+       DBG(LOG_INFO, "part = %s", part);
+       if (!strcmp("elm.text", part))
                return g_strdup(D_("IDS_ST_HEADER_CONNECTED_DEVICES"));
-       }
 
        __FUNC_EXIT__;
        return NULL;
@@ -613,11 +603,11 @@ char* ConvertRGBAtoHex(int r, int g, int b, int a)
        int hexcolor = 0;
        char *string = NULL;
        string = g_try_malloc0(MAX_HEX_COLOR_LENGTH);
-       if (string == NULL) {
+       if (string == NULL)
                return string;
-       }
+
        hexcolor = (r << 24) + (g << 16) + (b << 8) + a;
-       snprintf(string, MAX_HEX_COLOR_LENGTH, "%08x", hexcolor );
+       snprintf(string, MAX_HEX_COLOR_LENGTH, "%08x", hexcolor);
        return string;
 }
 
@@ -688,9 +678,8 @@ static char *_gl_conn_failed_dev_title_label_get(void *data, Evas_Object *obj,
                return NULL;
        }
 
-       if (!strcmp("elm.text", part)) {
+       if (!strcmp("elm.text", part))
                return g_strdup(D_("IDS_WIFI_SBODY_NOT_CONNECTED_M_STATUS_ABB"));
-       }
 
        __FUNC_EXIT__;
        return NULL;
@@ -758,11 +747,10 @@ static char *_gl_multi_connect_dev_title_label_get(void *data, Evas_Object *obj,
        DBG(LOG_INFO, "%s", part);
 
        if (!strcmp("elm.text", part) && ugd) {
-               if (ugd->multi_connect_mode == WFD_MULTI_CONNECT_MODE_IN_PROGRESS) {
+               if (ugd->multi_connect_mode == WFD_MULTI_CONNECT_MODE_IN_PROGRESS)
                        return g_strdup(D_("IDS_WIFI_HEADER_AVAILABLE_DEVICES_ABB"));
-               } else if (ugd->multi_connect_mode == WFD_MULTI_CONNECT_MODE_COMPLETED) {
+               else if (ugd->multi_connect_mode == WFD_MULTI_CONNECT_MODE_COMPLETED)
                        return g_strdup(D_("IDS_WIFI_SBODY_NOT_CONNECTED_M_STATUS_ABB"));
-               }
        }
 
        __FUNC_EXIT__;
@@ -817,9 +805,9 @@ static Evas_Object *_wfd_gl_select_all_icon_get(void *data, Evas_Object *obj, co
                DBG(LOG_INFO, "Part %s", part);
                check = elm_check_add(icon_layout);
                ugd->select_all_icon = check;
-               if (ugd->is_select_all_checked == EINA_TRUE) {
+               if (ugd->is_select_all_checked == EINA_TRUE)
                        elm_check_state_set(ugd->select_all_icon, TRUE);
-               }
+
                elm_object_style_set(check, "default/genlist");
                evas_object_propagate_events_set(check, EINA_FALSE);
                evas_object_smart_callback_add(check, "changed",
@@ -848,9 +836,8 @@ static char *_gl_busy_dev_title_label_get(void *data, Evas_Object *obj,
        }
        DBG(LOG_INFO, "%s", part);
 
-       if (!strcmp("elm.text", part)) {
+       if (!strcmp("elm.text", part))
                return g_strdup(D_("IDS_ST_HEADER_BUSY_DEVICES"));
-       }
 
        __FUNC_EXIT__;
        return NULL;