edje: Remove bool return from scale_set
authorJean-Philippe Andre <jp.andre@samsung.com>
Fri, 9 Jun 2017 07:51:43 +0000 (16:51 +0900)
committerJean-Philippe Andre <jp.andre@samsung.com>
Mon, 12 Jun 2017 02:50:30 +0000 (11:50 +0900)
It was always returning true. There is little point in returning
a bool here, an invalid scale value (eg. <= 0) wouuld lead to a
state where scale_get() != scale_set() and that's about it.

src/lib/edje/edje_legacy.c
src/lib/edje/edje_util.c
src/lib/efl/interfaces/efl_ui_base.eo
src/lib/elementary/elm_widget.c

index 3fcedce..7658608 100644 (file)
@@ -120,7 +120,8 @@ EAPI const char *edje_object_language_get(const Edje_Object *obj)
 
 EAPI Eina_Bool edje_object_scale_set(Edje_Object *obj, double scale)
 {
-   return efl_ui_scale_set(obj, scale);
+   efl_ui_scale_set(obj, scale);
+   return EINA_TRUE;
 }
 
 EAPI double edje_object_scale_get(const Edje_Object *obj)
index db1d0d0..e76077e 100644 (file)
@@ -363,7 +363,7 @@ edje_password_show_last_timeout_set(double password_show_last_timeout)
    _edje_password_show_last_timeout = password_show_last_timeout;
 }
 
-EOLIAN Eina_Bool
+EOLIAN void
 _edje_object_efl_ui_base_scale_set(Eo *obj EINA_UNUSED, Edje *ed, double scale)
 {
    Edje *ged;
@@ -371,7 +371,7 @@ _edje_object_efl_ui_base_scale_set(Eo *obj EINA_UNUSED, Edje *ed, double scale)
    Eina_List *l;
    unsigned short i;
 
-   if (EQ(ed->scale, FROM_DOUBLE(scale))) return EINA_TRUE;
+   if (EQ(ed->scale, FROM_DOUBLE(scale))) return;
    ed->scale = FROM_DOUBLE(scale);
    EINA_LIST_FOREACH(ed->groups, l, ged)
      edje_object_scale_set(ged->obj, scale);
@@ -387,8 +387,6 @@ _edje_object_efl_ui_base_scale_set(Eo *obj EINA_UNUSED, Edje *ed, double scale)
           }
      }
    _edje_recalc(ed);
-
-   return EINA_TRUE;
 }
 
 EOLIAN double
index 8e49edb..b4bd11d 100644 (file)
@@ -48,7 +48,6 @@ interface Efl.Ui.Base
          ]]
          set {
             [[Sets the scaling factor of an object.]]
-            return: bool; [[$true on success, $false otherwise]]
          }
          get {
             [[Gets an object's scaling factor.]]
index 5c50d51..c3abb76 100644 (file)
@@ -3664,7 +3664,7 @@ _elm_widget_scroll_freeze_get(Eo *obj EINA_UNUSED, Elm_Widget_Smart_Data *sd)
    return sd->scroll_freeze;
 }
 
-EOLIAN static Eina_Bool
+EOLIAN static void
 _elm_widget_efl_ui_base_scale_set(Eo *obj, Elm_Widget_Smart_Data *sd, double scale)
 {
    if (scale < 0.0) scale = 0.0;
@@ -3673,7 +3673,6 @@ _elm_widget_efl_ui_base_scale_set(Eo *obj, Elm_Widget_Smart_Data *sd, double sca
         sd->scale = scale;
         elm_widget_theme(obj);
      }
-   return EINA_TRUE;
 }
 
 EOLIAN static double