Use empty() function instead of size() for readability

Change-Id: If554556881981e0f66df7fba11736c80ea6e4423
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

diff --git a/src/ise-emoticon-mode.cpp b/src/ise-emoticon-mode.cpp
index 58849e4..6a1af35 100644 (file)
--- a/src/ise-emoticon-mode.cpp
+++ b/src/ise-emoticon-mode.cpp
@@ -242,7 +242,7 @@ void ise_emoticon_show_layout(emoticon_group_t emoticon_group, const int screen_
 
     if (emoticon_group == EMOTICON_GROUP_RECENTLY_USED) {
         //ise_read_recent_emoticon_list_from_scim();
-        if (emoticon_list_recent.size() == 0) {
+        if (emoticon_list_recent.empty()) {
             //PRINTFUNC(DLOG_ERROR,"Cannot display recently used emoticons group. No recently used emoticons available");
             return;
         }
@@ -383,7 +383,7 @@ void ise_emoticon_change_mode(emoticon_group_t emoticon_group)
 {
     if (emoticon_group == EMOTICON_GROUP_RECENTLY_USED) {
         //ise_read_recent_emoticon_list_from_scim();
-        if (emoticon_list_recent.size() == 0) {
+        if (emoticon_list_recent.empty()) {
             //PRINTFUNC(DLOG_ERROR,"Cannot display recently used emoticons group. No recently used emoticons available");
             return;
         }

diff --git a/src/ise-sticker-mode.cpp b/src/ise-sticker-mode.cpp
index 3e45799..3d15c1a 100644 (file)
--- a/src/ise-sticker-mode.cpp
+++ b/src/ise-sticker-mode.cpp
@@ -409,17 +409,10 @@ void ise_sticker_show_layout(sticker_group_e sticker_group, int screen_degree, b
     if (sticker_consumer)
         ise_sticker_destroy_layout();
 
-    if (sticker_group_list.size() != 0)
-        sticker_group_list.clear();
-
-    if (sticker_group_image_list.size() != 0)
-        sticker_group_image_list.clear();
-
-    if (sticker_list.size() != 0)
-        sticker_list.clear();
-
-    if (recent_sticker_list.size() != 0)
-        recent_sticker_list.clear();
+    sticker_group_list.clear();
+    sticker_group_image_list.clear();
+    sticker_list.clear();
+    recent_sticker_list.clear();
 
     ise_sticker_create_consumer_handle();
     ise_sticker_get_all_sticker();

diff --git a/src/languages.cpp b/src/languages.cpp
index 3231d6e..5959fe5 100644 (file)
--- a/src/languages.cpp
+++ b/src/languages.cpp
@@ -388,7 +388,7 @@ ISELanguageManager::enable_languages(const vector<string> &vec_language_id)
 /* FIXME A temporary way for enable default language */
 sclboolean ISELanguageManager::enable_default_language()
 {
-    if (_language_vector.size()) {
+    if (!_language_vector.empty()) {
         LANGUAGE_INFO &default_language = _language_vector.at(0);
         default_language.enabled = TRUE;
         default_language.enabled_temporarily = FALSE;
@@ -402,7 +402,7 @@ sclboolean ISELanguageManager::set_enabled_languages(const vector<string> &vec_l
 {
     sclboolean ret = FALSE;
 
-    if (vec_language_id.size() == 0 || FALSE == enable_languages(vec_language_id)) {
+    if (vec_language_id.empty() || FALSE == enable_languages(vec_language_id)) {
         ret = enable_default_language();
     }
 

diff --git a/src/option.cpp b/src/option.cpp
index 6ac8194..b1712ec 100644 (file)
--- a/src/option.cpp
+++ b/src/option.cpp
@@ -1197,7 +1197,7 @@ Evas_Object* create_option_main_view(Evas_Object *parent, Evas_Object *naviframe
             if (info && info->enabled) {
                 strncpy(language_itemdata[loop].main_text, info->display_name.c_str(), ITEM_DATA_STRING_LEN - 1);
 #ifdef _WEARABLE
-                if (info->input_modes.size() > 0) {
+                if (!info->input_modes.empty()) {
                     INPUT_MODE_INFO input_mode = info->input_modes.at(0);
                     strncpy(language_itemdata[loop].sub_text, input_mode.display_name.c_str(), ITEM_DATA_STRING_LEN - 1);
                 }
@@ -1324,7 +1324,7 @@ static Evas_Object* create_option_language_view(Evas_Object *naviframe)
             if (info) {
                 strncpy(language_itemdata[loop].main_text, info->display_name.c_str(), ITEM_DATA_STRING_LEN - 1);
 #ifdef _WEARABLE
-                if (info->input_modes.size() > 0) {
+                if (!info->input_modes.empty()) {
                     INPUT_MODE_INFO input_mode = info->input_modes.at(0);
                     strncpy(language_itemdata[loop].sub_text, input_mode.display_name.c_str(), ITEM_DATA_STRING_LEN - 1);
                 }
@@ -1455,7 +1455,7 @@ static void read_options(Evas_Object *naviframe)
             if (info && info->enabled) {
                 strncpy(language_itemdata[loop].main_text, info->display_name.c_str(), ITEM_DATA_STRING_LEN - 1);
 #ifdef _WEARABLE
-                if (info->input_modes.size() > 0) {
+                if (!info->input_modes.empty()) {
                     INPUT_MODE_INFO input_mode = info->input_modes.at(0);
                     strncpy(language_itemdata[loop].sub_text, input_mode.display_name.c_str(), ITEM_DATA_STRING_LEN - 1);
                 }
@@ -1578,7 +1578,7 @@ static void language_selected(void *data, Evas_Object *obj, void *event_info)
                         }
                     }
                 }
-                if (enabled_languages.size() > 0) {
+                if (!enabled_languages.empty()) {
                     config_values->enabled_languages = enabled_languages;
                     LOGD("Enabled languages size:%zu\n", config_values->enabled_languages.size());
                     if (!selected_language_found) {