[vlc-devel] Re: [PATCH] - fix frees w/out NULLs
Laurent Aimar
fenrir at via.ecp.fr
Thu Jun 7 23:32:34 CEST 2007
On Thu, Jun 07, 2007, Pierre d'Herbemont wrote:
> Seems that you are mixing space/tabs here (more occurence in the rest of
> the patch though). Spaces should be used.
> Also, I am not sure we really want to do this. For instance in the above
> this is absolutely unecessary as we won't reuse the ptr. So I guess your
> patch add a lot of junk code.
IMHO it offuscates the code way too much, valgrind will detect those case
perfectly fine anyway...
Regards,
--
fenrir
--
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html
More information about the vlc-devel
mailing list