2013/2/17 Daniel Juyung Seo <seojuyu...@gmail.com>

> Hello, thanks for the patch but this looks wrong.
>
> Genlist item has two types of elements:
> 1. Text
>   "Text" is a text data that can be set to TEXT or TEXTBLOCK part of
> genlist item.
>   The text is retrieved by text_get callback and set in
> _item_text_realize() function.
> 2. Content
>   "Content" is an object that ca be swallowed into SWALLOW part of genlist
> item.
>   The object is retrieved by content_get callback and swallowed in
> _item_content_realize() function.
>
> So, _item_content_get_hook() is used to get the swallowed object to the
> SWALLOW part.
> So edje_object_part_swallow_get() is correct.
>
>
Oh yes you're right, I missundestood the code, and it's not the case in
trunk, i did the modification myself :D



>
> Daniel Juyung Seo (SeoZ)
>
> On Sun, Feb 17, 2013 at 7:02 AM, Nicolas Aguirre
> <aguirre.nico...@gmail.com>wrote:
>
> > Hi,
> >
> > I found a strange behaviour in the item_content_get_hook function of
> > genlist in the 1.7.x branch.
> > When elm_object_part_content_get is called, only swallow parts are
> > returned. Attached a fix wich return any parts of the item object. It
> seems
> > that this is already corrected in the trunk version of elementary so i
> > think that we can see this patch as a backport ?
> >
> > So should i commit this patch or not ?
> >
> > --
> > Nicolas Aguirre
> > Mail: aguirre.nico...@gmail.com
> > Web: http://enna.geexbox.org
> > Blog: http://dev.enlightenment.fr/~captainigloo/
> >
> >
> >
> ------------------------------------------------------------------------------
> > The Go Parallel Website, sponsored by Intel - in partnership with
> Geeknet,
> > is your hub for all things parallel software development, from weekly
> > thought
> > leadership blogs to news, videos, case studies, tutorials, tech docs,
> > whitepapers, evaluation guides, and opinion stories. Check out the most
> > recent posts - join the conversation now.
> > http://goparallel.sourceforge.net/
> > _______________________________________________
> > enlightenment-devel mailing list
> > enlightenment-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> >
> >
>
> ------------------------------------------------------------------------------
> The Go Parallel Website, sponsored by Intel - in partnership with Geeknet,
> is your hub for all things parallel software development, from weekly
> thought
> leadership blogs to news, videos, case studies, tutorials, tech docs,
> whitepapers, evaluation guides, and opinion stories. Check out the most
> recent posts - join the conversation now.
> http://goparallel.sourceforge.net/
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>



-- 
Nicolas Aguirre
Mail: aguirre.nico...@gmail.com
Web: http://enna.geexbox.org
Blog: http://dev.enlightenment.fr/~captainigloo/
------------------------------------------------------------------------------
The Go Parallel Website, sponsored by Intel - in partnership with Geeknet, 
is your hub for all things parallel software development, from weekly thought 
leadership blogs to news, videos, case studies, tutorials, tech docs, 
whitepapers, evaluation guides, and opinion stories. Check out the most 
recent posts - join the conversation now. http://goparallel.sourceforge.net/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to