On Mon, Aug 8, 2011 at 4:19 AM, Carsten Haitzler <ras...@rasterman.com> wrote:
> On Wed, 03 Aug 2011 06:15:13 +0000 (GMT) 김재환 <jae.hwan....@samsung.com> said:
>
> your patch seems good except documentation. docs are too short. you dont
> even say @see another func (eg elm_scroller_current_page_get() should see
> elm_scroller_last_page_get() and elm_scroller_page_show() and
> elm_scroller_page_bring_in().  you don't give examples of how to use these in
> "real life usage" in the docs. need more docs (examples etc.)

also not sure about the last_page_get(), it's similar to _count() in
list, so why not provide a _count() that is shorter and more used?

-- 
Gustavo Sverzut Barbieri
http://profusion.mobi embedded systems
--------------------------------------
MSN: barbi...@gmail.com
Skype: gsbarbieri
Mobile: +55 (19) 9225-2202

------------------------------------------------------------------------------
uberSVN's rich system and user administration capabilities and model 
configuration take the hassle out of deploying and managing Subversion and 
the tools developers use with it. Learn more about uberSVN and get a free 
download at:  http://p.sf.net/sfu/wandisco-dev2dev
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to