On Thu, Aug 25, 2011 at 6:11 PM, Mike Blumenkrantz <m...@zentific.com> wrote: > On Thu, 25 Aug 2011 14:09:43 -0700 > "Enlightenment SVN" <no-re...@enlightenment.org> wrote: > >> Log: >> emotion: g_free already checks if pointer is NULL >> >> semantic patch: >> >> @r1@ >> expression E1; >> @@ >> >> - if (E1) >> - g_free(E1); >> + g_free(E1); >> >> >> >> >> Author: lucas >> Date: 2011-08-25 14:09:42 -0700 (Thu, 25 Aug 2011) >> New Revision: 62834 >> Trac: http://trac.enlightenment.org/e/changeset/62834 >> >> Modified: >> trunk/emotion/src/modules/gstreamer/emotion_gstreamer.c > uhhh free(NULL) is totally valid as well...
sure... Because of this I put the semantic patch in the log. You can apply it if you want. Lucas De Marchi ------------------------------------------------------------------------------ EMC VNX: the world's simplest storage, starting under $10K The only unified storage solution that offers unified management Up to 160% more powerful than alternatives and 25% more efficient. Guaranteed. http://p.sf.net/sfu/emc-vnx-dev2dev _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel