Wow, amazing. I will try to fix them if it is correct approach.(I think so)
Thank you so much about good feedback. Best Regards, Ryuan Choi 2014/1/14 daniel.za...@samsung.com <daniel.za...@samsung.com> > We finally found one! > > > On 01/14/2014 11:14 AM, Yakov Goldberg wrote: > >> Hi Ryuan, >> we (Jack 'boss' Danielz) and I found the usecase when it print's >> errors. >> >> Try to add item to toolbar with NULL icon. >> I believe it will also fail, if you add any element with NULL icon (or >> maybe NULL content) >> >> it = elm_toolbar_item_append(toolbar, NULL, "Toolbar item", NULL, NULL); >> elm_object_item_disabled_set(it, EINA_TRUE); >> >> Yakov >> >> On 01/13/2014 08:39 PM, ryuan Choi wrote: >> >>> Sure, I checked several items of elementary_test (not all). They don't >>> complain to me at least in my laptop. >>> It looks fixed. >>> >>> >>> >>> 2014/1/13 daniel.za...@samsung.com <daniel.za...@samsung.com> >>> >>> I don't have an example just here now. Did you check tests of >>>> elementary_test? >>>> I just remember that there were a lot of prints. Maybe it has been >>>> fixed. >>>> >>>> >>>> On 01/13/2014 03:32 PM, ryuan Choi wrote: >>>> >>>> Weird, >>>> >>>> I tested little bit and ELM_WIDGET_CHECK with null object just printed >>>> only one line complaint. >>>> I thought that null object is wrong case which we should fix (or blame) >>>> >>>> Could you let me know more information or reproduce step to get a >>>> lot of >>>> errors to ensure that my reading is wrong. >>>> >>>> Best Regards, >>>> Ryuan Choi >>>> >>>> >>>> >>>> 2014/1/13 daniel.za...@samsung.com <daniel.za...@samsung.com> >>>> >>>> Hi Ryuan, >>>>> >>>>> The problem here is that now you will have a lot of errors from eo_isa >>>>> and his brother. At least, this check was helping on this. >>>>> >>>>> JackDanielZ, alias Daniel the 3rd >>>>> >>>>> >>>>> On 01/13/2014 02:18 PM, Ryuan Choi wrote: >>>>> >>>>> ryuan pushed a commit to branch master. >>>>>> >>>>>> >>>>>> http://git.enlightenment.org/core/elementary.git/commit/?id= >>>>>> 4c3e23de9e46fd89672ab657f5da153a3c26a029 >>>>>> >>>>>> commit 4c3e23de9e46fd89672ab657f5da153a3c26a029 >>>>>> Author: Ryuan Choi <ryuan.c...@gmail.com> >>>>>> Date: Mon Jan 13 21:11:04 2014 +0900 >>>>>> >>>>>> Widget: Removed !obj checks from ELM_WIDGET_DATA_GET_OR_RETURN >>>>>> and >>>>>> ELM_WIDGET_CHECK >>>>>> eo_data_scope_get and eo_isa returns NULL with complaint >>>>>> when >>>>>> NULL is passed. >>>>>> So, Removed unnecessary !obj check routine. >>>>>> In addition, adopt EINA_UNLIKELY for these macros and fix >>>>>> indentation. >>>>>> --- >>>>>> src/lib/elm_widget.h | 16 +++++++--------- >>>>>> 1 file changed, 7 insertions(+), 9 deletions(-) >>>>>> >>>>>> diff --git a/src/lib/elm_widget.h b/src/lib/elm_widget.h >>>>>> index f7170f6..ded37bc 100644 >>>>>> --- a/src/lib/elm_widget.h >>>>>> +++ b/src/lib/elm_widget.h >>>>>> @@ -801,19 +801,17 @@ EAPI void elm_widget_tree_dump(const >>>>>> Evas_Object *top); >>>>>> EAPI void elm_widget_tree_dot_dump(const Evas_Object >>>>>> *top, >>>>>> FILE *output); >>>>>> #define ELM_WIDGET_DATA_GET_OR_RETURN(o, ptr, ...) \ >>>>>> - Elm_Widget_Smart_Data *ptr; \ >>>>>> - ptr = (o ? \ >>>>>> - eo_data_scope_get(o, ELM_OBJ_WIDGET_CLASS) :\ >>>>>> - NULL); \ >>>>>> - if (!ptr) \ >>>>>> + Elm_Widget_Smart_Data *ptr; \ >>>>>> + ptr = eo_data_scope_get(o, ELM_OBJ_WIDGET_CLASS); \ >>>>>> + if (EINA_UNLIKELY(!ptr)) \ >>>>>> { \ >>>>>> - CRI("no widget data for object %p (%s)", \ >>>>>> - o, evas_object_type_get(o)); \ >>>>>> + CRI("no widget data for object %p (%s)", \ >>>>>> + o, evas_object_type_get(o)); \ >>>>>> return __VA_ARGS__; \ >>>>>> } >>>>>> -#define ELM_WIDGET_CHECK(obj) \ >>>>>> - if (!obj || !eo_isa((obj), ELM_OBJ_WIDGET_CLASS)) \ >>>>>> +#define ELM_WIDGET_CHECK(obj) \ >>>>>> + if (EINA_UNLIKELY(!eo_isa((obj), ELM_OBJ_WIDGET_CLASS))) \ >>>>>> return >>>>>> /** >>>>>> >>>>>> >>>>>> ------------------------------------------------------------ >>> ------------------ >>> CenturyLink Cloud: The Leader in Enterprise Cloud Services. >>> Learn Why More Businesses Are Choosing CenturyLink Cloud For >>> Critical Workloads, Development Environments & Everything In Between. >>> Get a Quote or Start a Free Trial Today. >>> http://pubads.g.doubleclick.net/gampad/clk?id=119420431& >>> iu=/4140/ostg.clktrk >>> _______________________________________________ >>> enlightenment-devel mailing list >>> enlightenment-devel@lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel >>> >>> >> ------------------------------------------------------------ >> ------------------ >> CenturyLink Cloud: The Leader in Enterprise Cloud Services. >> Learn Why More Businesses Are Choosing CenturyLink Cloud For >> Critical Workloads, Development Environments & Everything In Between. >> Get a Quote or Start a Free Trial Today. >> http://pubads.g.doubleclick.net/gampad/clk?id=119420431& >> iu=/4140/ostg.clktrk >> _______________________________________________ >> enlightenment-devel mailing list >> enlightenment-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel >> >> > ------------------------------------------------------------------------------ CenturyLink Cloud: The Leader in Enterprise Cloud Services. Learn Why More Businesses Are Choosing CenturyLink Cloud For Critical Workloads, Development Environments & Everything In Between. Get a Quote or Start a Free Trial Today. http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel