On Thu, Feb 26, 2009 at 12:22 AM, The Rasterman Carsten Haitzler
<ras...@rasterman.com> wrote:
> On Wed, 25 Feb 2009 14:41:41 +0100 Cedric BAIL <cedric.b...@free.fr> said:
>
>> On Wed, Feb 25, 2009 at 2:33 PM, Gustavo Sverzut Barbieri
>> <barbi...@profusion.mobi> wrote:
>> > On Wed, Feb 25, 2009 at 9:47 AM, Cedric BAIL <cedric.b...@free.fr> wrote:
>> >> On Tue, Feb 24, 2009 at 10:33 PM, Gustavo Sverzut Barbieri
>> >> <barbi...@profusion.mobi> wrote:
>> >>> On Wed, Jan 28, 2009 at 9:04 PM, Toma <tomha...@gmail.com> wrote:
>> >>>> Thought Id ping the mailing list about this:
>> >>>>
>> >>>> http://trac.enlightenment.org/e/wiki/Release
>> >>>>
>> >>>> As the top of the page says, we need to finalize the list of things
>> >>>> TODO. Mekius mentioned removing a couple of the EFM todos and Id
>> >>>> welcome that. Im in the process of finishing off the icons, but think
>> >>>> some of the dialogs could either get the chop, or get renamed. One of
>> >>>> these is 'Interactions', as it has a pile of thumbscroll options.
>> >>>>
>> >>>> Shall we set a date for closing off the TODO feature list? Any
>> >>>> additions after that date will be bugs related to the features.
>> >>>>
>> >>>> I propose February 14th. Its close, realistic, and might dull the
>> >>>> loneliness of Valentines Day. :)
>> >>>
>> >>> So time has come and we're working on that release list. In the last
>> >>> days I've being working on some show stopper stuff like file
>> >>> management and I plan to get ride of most items by March.
>> >>
>> >> Did the same on the eina front :)
>> >>
>> >>> Some items in the list are very demanding, like finish converting
>> >>> ecore data types to eina, but Cedric has some pending patches in
>> >>> queue. If people can help with these "easy" items, then we can work on
>> >>> more complex stuff and get E17 released soon, making everybody happy,
>> >>> with ready to use packages in most distros :-)
>> >>
>> >> All my pending patch are now in. It should not introduce any
>> >> regression. Now e_dbus, efreet and ecore should only return Eina_List.
>> >>
>> >> Still on the TODO, but this could be done one after another with very
>> >> small commit :
>> >> - remove ecore_dlist (replace by eina_list)
>> >> - remove Ecore_List2 (could be replaced by eina_inlist)
>> >> - review every little loop in the code around list and check, if it
>> >> would not be better to use EINA_LIST_FREE or EINA_LIST_FOREACH (at the
>> >> same time, look if we can avoid useless strdup).
>> > ...
>> >
>> > I'd say we should inspect places where lists and specially list
>> > *copies* are returned and use eina_iterator or eina_accessor instead.
>> > In many places we dup the list (1 walk), then use it (+1 walk), then
>> > free it (+1 walk), so 3 walk for a simple task. If we used the
>> > iterator stuff we could alloc a small amount of memory and do the same
>> > thing, just more clean, safer and faster.
>> >
>> > Comes to mind evas_render_method_list() and similar.
>>
>> Yes, returning Eina_Iterator would be a better solution in many place
>> in the EFL API. Could be faster and more memory efficient for many
>> case, and it will explicitely say that a the content of a list should
>> not be destroyed/modified by the caller.
>
> beware of being too gung-ho. some of htese (like evas_render_method_list())
> are called so rarely you likely just make the api harder to use for no real
> gain. :)

not harder, easier! Just get used to iterators and accessors, you'll love them.

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

------------------------------------------------------------------------------
Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA
-OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise
-Strategies to boost innovation and cut costs with open source participation
-Receive a $600 discount off the registration fee with the source code: SFAD
http://p.sf.net/sfu/XcvMzF8H
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to