[libdvdnav-devel] [PATCH] Set all freed pointers to NULL; for ifofiles, set pointers to NULL instead of zero

Jean-Baptiste Kempf jb at videolan.org
Sat Jan 3 21:34:32 CET 2015


On 03 Jan, Steve Dibb wrote :
>  src/ifo_read.c | 197
> +++++++++++++++++++++++++++++++++++++++++++-------------- 1 file
> changed, 148 insertions(+), 49 deletions(-)

If you want to do that, you should at least to a FREENULL macro, and
replace.

However, for local scope variables that cannot be used outside, it's a
bit useless, and does not help to cleanup the code.

With my kindest regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device


More information about the libdvdnav-devel mailing list