On Fri, Jul 19, 2013 at 11:31 AM, Cedric BAIL <cedric.b...@free.fr> wrote:

> On Fri, Jul 19, 2013 at 3:12 PM, José Roberto de Souza - Enlightenment
> Git <no-re...@enlightenment.org> wrote:
> > zehortigoza pushed a commit to branch master.
> >
> > commit 4732679935de40babf02bfe9473e21a23662f276
> > Author: José Roberto de Souza <jose.so...@intel.com>
> > Date:   Fri Jul 19 10:09:09 2013 -0300
> >
> >     edje: Add missing ChangeLog and NEWS of signal callbacks fixes
>
> Not necessary, the bug was introduced during the 1.8 release cycle and
> btw thanks for the fix ! Really good catch !
>


SPANK CEDRIC... it took us days to figure it out... just because mister
cedric-the-crazy-french decided it was worth to save 4bits BITS -- yes 4
BITS, given that for these "wasted" 4 bits we use real memory for 3
pointers (3 * sizeof(void*) == 12(32bits) or 24(64bits))... with a really
bad code readability :-(

I've asked José to rewrite that code so flags is actually struct { unsigned
char delete_me:1, propagate:1, just_added:1}... while we "loose 4 bits"
(OMG, world collapses!) we gain readability again :-D

-- 
Gustavo Sverzut Barbieri
http://profusion.mobi embedded systems
--------------------------------------
MSN: barbi...@gmail.com
Skype: gsbarbieri
Mobile: +55 (19) 9225-2202
------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&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