On Fri, 2011-01-21 at 13:23 +0900, Mike McCormack wrote:
> -          free(g);
> -          free(g->keyname);
> 
> I think the essence of the patch is that the above two lines are in the wrong 
> order.
> 
> It would be clearer if whitespace changes were not mixed with fixes.


Actually, I didn't even look about the code itself, the whitespace
changes (especially since they are wrong) threw me off :)

--
Tom.


------------------------------------------------------------------------------
Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)!
Finally, a world-class log management solution at an even better price-free!
Download using promo code Free_Logger_4_Dev2Dev. Offer expires 
February 28th, so secure your free ArcSight Logger TODAY! 
http://p.sf.net/sfu/arcsight-sfd2d
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to