elm: restore old elm_web behaviour
authorMarcel Hollerbach <mail@marcel-hollerbach.de>
Mon, 14 Jan 2019 19:12:32 +0000 (20:12 +0100)
committerJunsuChoi <jsuya.choi@samsung.com>
Thu, 24 Jan 2019 05:20:19 +0000 (14:20 +0900)
Everything that is owned must have a free function. In commit
1afd3c215fe5e8933a5ccf0a49aa2fee41043286 the string was
converted to const(string) it was not const before.
So this is converting back to the old behaviour.

Reviewed-by: Cedric BAIL <cedric.bail@free.fr>
Differential Revision: https://phab.enlightenment.org/D7626

src/lib/elementary/elm_web.eo
src/modules/elementary/web/none/elm_web_none.c

index 4293088..00cdc72 100644 (file)
@@ -278,8 +278,7 @@ abstract Elm.Web extends Efl.Ui.Widget implements Efl.Ui.Legacy, Efl.Ui.Zoom
               The string returned must be freed by the user when it's done
               with it.
             ]]
-            /* FIXME: owned const string? */
-            return: string @owned @warn_unused; [[A newly allocated string, or
+            return: mstring @owned @warn_unused; [[A newly allocated string, or
                                                       $null if nothing is selected
                                                       or an error occurred.]]
          }
index 4a8b161..7ad4da3 100644 (file)
@@ -146,7 +146,7 @@ _elm_web_none_elm_web_bg_color_get(const Eo *obj EINA_UNUSED, Elm_Web_None_Data
    if (a) *a = 0;
 }
 
-EOLIAN static const char*
+EOLIAN static char*
 _elm_web_none_elm_web_selection_get(const Eo *obj EINA_UNUSED, Elm_Web_None_Data *_pd EINA_UNUSED)
 {
    return NULL;