A huge gain in code readability and reduced number of lines.

S.

Michael Blumenkrantz <michael.blumenkra...@gmail.com>:

>On Sat, 07 Dec 2013 12:12:43 -0800
>Sebastian Dransfeld <s...@tango.flipp.net> wrote:
>
>> englebass pushed a commit to branch master.
>> 
>> http://git.enlightenment.org/core/efl.git/commit/?id=50feb1f8daf5c3b7be1fd7c5cd344f9f8619d899
>> 
>> commit 50feb1f8daf5c3b7be1fd7c5cd344f9f8619d899
>> Author: Sebastian Dransfeld <s...@tango.flipp.net>
>> Date:   Sat Dec 7 16:31:55 2013 +0100
>> 
>>     ecore_x: Use eina_inlist
>>     
>>     ecore_x_selection implemented it's own single linked list. Use
>>     eina_inlist instead.
>> ---
>>  src/lib/ecore_x/xlib/ecore_x_private.h   |   4 +-
>>  src/lib/ecore_x/xlib/ecore_x_selection.c | 133 
>> +++++++++----------------------
>>  2 files changed, 39 insertions(+), 98 deletions(-)
>> 
>
>while I agree with the premise of using our own apis internally, this one 
>triples the memory usage for the list (8bytes -> 24bytes). not a huge 
>difference I guess, but it feels wrong since there's no gain here.
>
>------------------------------------------------------------------------------
>Rapidly troubleshoot problems before they affect your business. Most IT 
>organizations don't have a clear picture of how application performance 
>affects their revenue. With AppDynamics, you get 100% visibility into your 
>Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro!
>http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk
>_______________________________________________
>enlightenment-devel mailing list
>enlightenment-devel@lists.sourceforge.net
>https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
------------------------------------------------------------------------------
Rapidly troubleshoot problems before they affect your business. Most IT 
organizations don't have a clear picture of how application performance 
affects their revenue. With AppDynamics, you get 100% visibility into your 
Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro!
http://pubads.g.doubleclick.net/gampad/clk?id=84349831&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