[Elm] Ok, since we don't have time to address all functions, just

return val. on the most critical one: file_set (on bg).

SVN revision: 68856

diff --git a/src/lib/elm_bg.c b/src/lib/elm_bg.c
index 4d507e0..edc23f1 100644 (file)
--- a/src/lib/elm_bg.c
+++ b/src/lib/elm_bg.c
@@ -243,28 +243,28 @@ elm_bg_file_set(Evas_Object *obj, const char *file, const char *group)
    return EINA_TRUE;
 }
 
-EAPI Eina_Bool
+EAPI void
 elm_bg_file_get(const Evas_Object *obj, const char **file, const char **group)
 {
-   ELM_CHECK_WIDTYPE(obj, widtype) EINA_FALSE;
+   ELM_CHECK_WIDTYPE(obj, widtype);
    Widget_Data *wd = elm_widget_data_get(obj);
    if (file) *file = wd->file;
    if (group) *group = wd->group;
 
-   return EINA_TRUE;
+   return;
 }
 
-EAPI Eina_Bool
+EAPI void
 elm_bg_option_set(Evas_Object *obj, Elm_Bg_Option option)
 {
-   ELM_CHECK_WIDTYPE(obj, widtype) EINA_FALSE;
+   ELM_CHECK_WIDTYPE(obj, widtype);
    Widget_Data *wd;
 
    wd = elm_widget_data_get(obj);
    wd->option = option;
    _custom_resize(wd, NULL, NULL, NULL);
 
-   return EINA_TRUE;
+   return;
 }
 
 EAPI Elm_Bg_Option
@@ -277,10 +277,10 @@ elm_bg_option_get(const Evas_Object *obj)
    return wd->option;
 }
 
-EAPI Eina_Bool
+EAPI void
 elm_bg_color_set(Evas_Object *obj, int r, int g, int b)
 {
-   ELM_CHECK_WIDTYPE(obj, widtype) EINA_FALSE;
+   ELM_CHECK_WIDTYPE(obj, widtype);
    Widget_Data *wd;
 
    wd = elm_widget_data_get(obj);
@@ -293,34 +293,34 @@ elm_bg_color_set(Evas_Object *obj, int r, int g, int b)
      }
    evas_object_color_set(wd->rect, r, g, b, 255);
 
-   return EINA_TRUE;
+   return;
 }
 
-EAPI Eina_Bool
+EAPI void
 elm_bg_color_get(const Evas_Object *obj, int *r, int *g, int *b)
 {
-   ELM_CHECK_WIDTYPE(obj, widtype) EINA_FALSE;
+   ELM_CHECK_WIDTYPE(obj, widtype);
    Widget_Data *wd;
 
    wd = elm_widget_data_get(obj);
    evas_object_color_get(wd->rect, r, g, b, NULL);
 
-   return EINA_TRUE;
+   return;
 }
 
-EAPI Eina_Bool
+EAPI void
 elm_bg_load_size_set(Evas_Object *obj, Evas_Coord w, Evas_Coord h)
 {
-   ELM_CHECK_WIDTYPE(obj, widtype) EINA_FALSE;
+   ELM_CHECK_WIDTYPE(obj, widtype);
    Widget_Data *wd = elm_widget_data_get(obj);
    const char *p;
-   if (!wd) return EINA_FALSE;
+   if (!wd) return;
    wd->load_opts.w = w;
    wd->load_opts.h = h;
-   if (!wd->img) return EINA_FALSE;
+   if (!wd->img) return;
    if (!(((p = strrchr(wd->file, '.'))) && (!strcasecmp(p, ".edj"))))
      evas_object_image_load_size_set(wd->img, w, h);
 
-   return EINA_TRUE;
+   return;
 }
 

diff --git a/src/lib/elm_bg.h b/src/lib/elm_bg.h
index 1af2aa3..74d9962 100644 (file)
--- a/src/lib/elm_bg.h
+++ b/src/lib/elm_bg.h
@@ -90,21 +90,19 @@ EAPI Eina_Bool                    elm_bg_file_set(Evas_Object *obj, const char *
  * @param file Where to store the requested file's path
  * @param group Where to store the optional key within @a file, @b if
  * it's an Edje file
- * @return @c EINA_TRUE on success, @c EINA_FALSE otherwise
  *
  * @note Use @c NULL pointers on the file components you're not
  * interested in: they'll be ignored by the function.
  *
  * @ingroup Bg
  */
-EAPI Eina_Bool                    elm_bg_file_get(const Evas_Object *obj, const char **file, const char **group);
+EAPI void                         elm_bg_file_get(const Evas_Object *obj, const char **file, const char **group);
 
 /**
  * Set the mode of display for a given background widget's image
  *
  * @param obj The background object handle
  * @param option The desired background option (see #Elm_Bg_Option)
- * @return @c EINA_TRUE on success, @c EINA_FALSE otherwise
  *
  * This sets how the background widget will display its image. This
  * will only work if the elm_bg_file_set() was previously called with
@@ -115,7 +113,7 @@ EAPI Eina_Bool                    elm_bg_file_get(const Evas_Object *obj, const
  *
  * @ingroup Bg
  */
-EAPI Eina_Bool                    elm_bg_option_set(Evas_Object *obj, Elm_Bg_Option option);
+EAPI void                         elm_bg_option_set(Evas_Object *obj, Elm_Bg_Option option);
 
 /**
  * Get the mode of display for a given background widget's image
@@ -137,7 +135,6 @@ EAPI Elm_Bg_Option                elm_bg_option_get(const Evas_Object *obj);
  * @param r The red color component's value
  * @param g The green color component's value
  * @param b The blue color component's value
- * @return @c EINA_TRUE on success, @c EINA_FALSE otherwise
  *
  * This sets the color used for the background rectangle, in RGB
  * format. Each color component's range is from 0 to 255.
@@ -150,7 +147,7 @@ EAPI Elm_Bg_Option                elm_bg_option_get(const Evas_Object *obj);
  *
  * @ingroup Bg
  */
-EAPI Eina_Bool                    elm_bg_color_set(Evas_Object *obj, int r, int g, int b);
+EAPI void                         elm_bg_color_set(Evas_Object *obj, int r, int g, int b);
 
 /**
  * Get the color set on a given background widget
@@ -159,7 +156,6 @@ EAPI Eina_Bool                    elm_bg_color_set(Evas_Object *obj, int r, int
  * @param r Where to store the red color component's value
  * @param g Where to store the green color component's value
  * @param b Where to store the blue color component's value
- * @return @c EINA_TRUE on success, @c EINA_FALSE otherwise
  *
  * @note Use @c NULL pointers on the file components you're not
  * interested in: they'll be ignored by the function.
@@ -168,7 +164,7 @@ EAPI Eina_Bool                    elm_bg_color_set(Evas_Object *obj, int r, int
  *
  * @ingroup Bg
  */
-EAPI Eina_Bool                    elm_bg_color_get(const Evas_Object *obj, int *r, int *g, int *b);
+EAPI void                         elm_bg_color_get(const Evas_Object *obj, int *r, int *g, int *b);
 
 /**
  * Set the size of the pixmap representation of the image set on a
@@ -193,7 +189,7 @@ EAPI Eina_Bool                    elm_bg_color_get(const Evas_Object *obj, int *
  *
  * @ingroup Bg
  */
-EAPI Eina_Bool                    elm_bg_load_size_set(Evas_Object *obj, Evas_Coord w, Evas_Coord h);
+EAPI void                         elm_bg_load_size_set(Evas_Object *obj, Evas_Coord w, Evas_Coord h);
 
 /**
  * @}