[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