in On Wed, Dec 19, 2012 at 2:07 PM, Maxime Villard <rusty...@gmx.fr> wrote:
> Hi, > here is a patch. > > 1. free() already null-checks the passed argument, so it is not > necessary to do 'if(x) free(x)'. > > 2. Moved some '{free(x); x = NULL}' to E_FREE. > > 3. In e_start_main.c, 'buf' is allocated with the size of the > two arguments + '=', so we don't need to check for lenght; > we should use sprintf(). > > thanks > ------------------------------------------------------------------------------ LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial Remotely access PCs and mobile devices and provide instant support Improve your efficiency, and focus on delivering more value-add services Discover what IT Professionals Know. Rescue delivers http://p.sf.net/sfu/logmein_12329d2d _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel