[EGIT] [core/elementary] master 01/01: elm popup - simplify if logic to be more compact and readable

[Carsten Haitzler]

raster pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=762c2a95b99a126e5263c9668a9c17784f143a3c

commit 762c2a95b99a126e5263c9668a9c17784f143a3c
Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com>
Date:   Tue Aug 18 14:34:30 2015 +0900

    elm popup - simplify if logic to be more compact and readable
---
 src/lib/elc_popup.c | 24 ++++++++++--------------
 1 file changed, 10 insertions(+), 14 deletions(-)

diff --git a/src/lib/elc_popup.c b/src/lib/elc_popup.c
index 12558f0..ed9cc85 100644
--- a/src/lib/elc_popup.c
+++ b/src/lib/elc_popup.c
@@ -297,21 +297,18 @@ _access_obj_process(Eo *obj, Eina_Bool is_access)
 static void
 _populate_theme_scroll(Elm_Popup_Data *sd)
 {
-   const char *content_area_width;
-
    if (sd->content_area)
      {
-        content_area_width = edje_object_data_get(
-           elm_layout_edje_get(sd->content_area), "scroller_enable");
-        if (!content_area_width)
-          sd->theme_scroll = EINA_FALSE;
-        else if (!strcmp(content_area_width, "on"))
-          sd->theme_scroll = EINA_TRUE;
-        else
-          sd->theme_scroll = EINA_FALSE;
+        const char *content_area_width =
+          edje_object_data_get(elm_layout_edje_get(sd->content_area),
+                               "scroller_enable");
+        if ((content_area_width) && (!strcmp(content_area_width, "on")))
+          {
+             sd->theme_scroll = EINA_TRUE;
+             return;
+          }
      }
-   else
-     sd->theme_scroll = EINA_FALSE;
+   sd->theme_scroll = EINA_FALSE;
 }
 
 EOLIAN static Eina_Bool
@@ -450,8 +447,7 @@ _elm_popup_elm_layout_sizing_eval(Eo *obj, Elm_Popup_Data 
*sd)
 
         if (!sd->theme_scroll && sd->scroll)
           {
-             if (minw < sd->max_sc_w)
-               minw = sd->max_sc_w;
+             if (minw < sd->max_sc_w) minw = sd->max_sc_w;
           }
         evas_object_size_hint_min_set(sd->content_area, minw, minh);
 

--