From: Hyoyoung Chang <hyoyoung@gmail.com>
authorHyoyoung Chang <hyoyoung@gmail.com>
Thu, 22 Dec 2011 07:56:01 +0000 (07:56 +0000)
committerCarsten Haitzler <raster@rasterman.com>
Thu, 22 Dec 2011 07:56:01 +0000 (07:56 +0000)
Subject: [E-devel] [patch] elc_multibuttonentry - delayed reset string

I was sent email about elc_multibuttonentry's bug. It reviewed by seoz.
And he found some path can make segfault.
So I changed reset string position in the widget.

I patched resetting internal entry's position, before that some label
are broken by too early resetting.

SVN revision: 66461

src/lib/elc_multibuttonentry.c

index f1ee8ba..410801c 100644 (file)
@@ -772,13 +772,13 @@ _add_button_item(Evas_Object *obj, const char *str, Multibuttonentry_Pos pos, co
         if (!(item_filter->callback_func(obj, str, data, item_filter->data)))
           return NULL;
      }
-   //entry is cleared when text is made to button
-   elm_entry_entry_set(wd->entry, "");
-
    // add button
    btn = edje_object_add(evas_object_evas_get(obj));
    str_utf8 = elm_entry_markup_to_utf8(str);
 
+   //entry is cleared when text is made to button
+   elm_entry_entry_set(wd->entry, "");
+
    _elm_theme_object_set(obj, btn, "multibuttonentry", "btn", elm_widget_style_get(obj));
    edje_object_part_text_set(btn, "elm.btn.text", str_utf8);
    edje_object_part_geometry_get(btn, "elm.btn.text", NULL, NULL, &width, &height);