From: Jihoon Kim <jihoon48.kim@samsung.com>
Date: Thu, 23 Feb 2017 02:33:17 +0000 (+0900)
Subject: Reduce the number of string compare in processing back key
X-Git-Tag: accepted/tizen/3.0/common/20170224.130029~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F96%2F116096%2F1;p=platform%2Fcore%2Fuifw%2Fise-default.git

Reduce the number of string compare in processing back key

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

diff --git a/src/ise.cpp b/src/ise.cpp
index 5123ece..1434428 100644
--- a/src/ise.cpp
+++ b/src/ise.cpp
@@ -1522,9 +1522,19 @@ void ise_update_table(const vector<string> &vec_str)
 
 void ise_process_key_event(scim::KeyEvent& key, sclu32 &ret)
 {
-    if (key.get_key_string().compare("KeyRelease+XF86Back") == 0 || key.get_key_string().compare("XF86Back") == 0) {
+    Eina_Bool back_key_pressed = EINA_FALSE;
+    Eina_Bool back_key_released = EINA_FALSE;
+
+    if (key.get_key_string().compare("XF86Back") == 0) {
+        back_key_pressed = EINA_TRUE;
+    }
+    else if (key.get_key_string().compare("KeyRelease+XF86Back") == 0) {
+        back_key_released = EINA_TRUE;
+    }
+
+    if (back_key_pressed || back_key_released) {
         if (g_popup_opened == TRUE) {
-            if (key.get_key_string().compare("KeyRelease+XF86Back") == 0) {
+            if (back_key_released) {
                 g_ui->close_all_popups();
                 g_popup_opened = FALSE;
             }