On Mar 10, 2016 8:10 AM, "Tom Hacohen" <t...@osg.samsung.com> wrote:
>
> Huh? It is valid and the code is correct. What seems to be the problem?
> Works here with no warnings (clang), but anyhow, the code is correct.

>From gcc point of view gl was undefined when set to sd->genlist and that
did come with a big fat warning :-)

> --
> Tom.
>
> On 10/03/16 00:59, Cedric BAIL wrote:
> > cedric pushed a commit to branch master.
> >
> >
http://git.enlightenment.org/core/elementary.git/commit/?id=da8287fa370bb579a4a592dd280ac5c8a6a25830
> >
> > commit da8287fa370bb579a4a592dd280ac5c8a6a25830
> > Author: Cedric BAIL <ced...@osg.samsung.com>
> > Date:   Wed Mar 9 16:57:02 2016 -0800
> >
> >      combobox: fix borkage after eo_add change.
> >
> >      toto = titi = eo_add is no longer valid.
> > ---
> >   src/lib/elc_combobox.c | 6 ++++--
> >   1 file changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/lib/elc_combobox.c b/src/lib/elc_combobox.c
> > index 4227cea..8f2b141 100644
> > --- a/src/lib/elc_combobox.c
> > +++ b/src/lib/elc_combobox.c
> > @@ -359,7 +359,8 @@ _elm_combobox_eo_base_constructor(Eo *obj,
Elm_Combobox_Data *sd)
> >      elm_table_pack(sd->tbl, sd->spacer, 0, 0, 1, 1);
> >
> >      // This is the genlist object that will take over the genlist call
> > -   sd->genlist = gl = eo_add(&gl, ELM_GENLIST_CLASS, obj);
> > +   eo_add(&gl, ELM_GENLIST_CLASS, obj);
> > +   sd->genlist = gl;
> >      elm_genlist_filter_set(gl, NULL);
> >      elm_widget_mirrored_automatic_set(gl, EINA_FALSE);
> >      elm_widget_mirrored_set(gl, elm_widget_mirrored_get(obj));
> > @@ -373,7 +374,8 @@ _elm_combobox_eo_base_constructor(Eo *obj,
Elm_Combobox_Data *sd)
> >      elm_table_pack(sd->tbl, gl, 0, 0, 1, 1);
> >
> >      // This is the entry object that will take over the entry call
> > -   sd->entry = entry = eo_add(&entry, ELM_ENTRY_CLASS, obj);
> > +   eo_add(&entry, ELM_ENTRY_CLASS, obj);
> > +   sd->entry = entry;
> >      elm_widget_mirrored_automatic_set(entry, EINA_FALSE);
> >      elm_widget_mirrored_set(entry, elm_widget_mirrored_get(obj));
> >      elm_scroller_policy_set(entry, ELM_SCROLLER_POLICY_OFF,
> >
>
>
>
------------------------------------------------------------------------------
> Transform Data into Opportunity.
> Accelerate data analysis in your applications with
> Intel Data Analytics Acceleration Library.
> Click to learn more.
> http://pubads.g.doubleclick.net/gampad/clk?id=278785111&iu=/4140
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>
------------------------------------------------------------------------------
Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://pubads.g.doubleclick.net/gampad/clk?id=278785111&iu=/4140
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to