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
