I really don't know where it is better to fix that, I mean inside the caller or inside the callee code.
On 01/14/2014 12:02 PM, ryuan Choi wrote: > 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 <mailto:daniel.za...@samsung.com> > <daniel.za...@samsung.com <mailto: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 > <mailto:daniel.za...@samsung.com> > <daniel.za...@samsung.com <mailto: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 > <mailto:daniel.za...@samsung.com> > <daniel.za...@samsung.com > <mailto: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 > <mailto: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 > <mailto: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 > <mailto: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