On Wed, Sep 11, 2013 at 4:11 AM, Cedric Bail - Enlightenment Git
<no-re...@enlightenment.org> wrote:
> cedric pushed a commit to branch master.
>
> commit cd77853222d1c11d71da4ca9066abe35390dcd5e
> Author: Cedric Bail <cedric.b...@samsung.com>
> Date:   Wed Sep 11 16:08:06 2013 +0900
>
>     eo: add eo_childrens_iterator_new().
>
>     The goal would be to replace the smart children list and friends. The
>     problem is that they differ in content. Smart children and Eo children are
>     the same, but Elm children and them differ. If I put this function as a
>     virtual, it would be possible to override the list of children and if we
>     start using it in Evas render loop, that could result in "weird" behavior.

1st, as glima pointed out children is already plural form, no need for
trailing "s"

2nd, I think this may cause a confusion. If one want all the Eo whose
eo_parent_get() is X, they may want to call
eo_children_iterator_new(X), but this virtual may not return that
exact list, but a subset. I don't remember exactly about the Elm case,
but I recall we had cases where one object is bound to another without
being member_add(), in that case eo_children_iterator_new() would
return objects that eo_parent_get(child) != X.

I see what you're trying to do, but I don't know if the naming is correct.


-- 
Gustavo Sverzut Barbieri
--------------------------------------
Mobile: +55 (19) 9225-2202
Contact: http://www.gustavobarbieri.com.br/contact

------------------------------------------------------------------------------
How ServiceNow helps IT people transform IT departments:
1. Consolidate legacy IT systems to a single system of record for IT
2. Standardize and globalize service processes across IT
3. Implement zero-touch automation to replace manual, redundant tasks
http://pubads.g.doubleclick.net/gampad/clk?id=51271111&iu=/4140/ostg.clktrk
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to