[FFmpeg-devel] [PATCH] Make avfilter_formats_unref() handle the case where *ref is NULL

Stefano Sabatini stefano.sabatini-lala
Thu Jan 7 00:37:08 CET 2010


On date Thursday 2010-01-07 00:14:54 +0100, Michael Niedermayer encoded:
> On Wed, Jan 06, 2010 at 11:33:38PM +0100, Stefano Sabatini wrote:
[...]
> > +fail:
> > +    if (formats)
> > +        avfilter_formats_unref(&formats);
> 
> do we need the if() ?
> in general functions not needing explicit NULL checks are nicer IMHO

Do you mean like this?
-- 
FFmpeg = Friendly Faboulous Mean Ponderous Extreme Guru
-------------- next part --------------
A non-text attachment was scrubbed...
Name: make-avff-unref-manage-null.patch
Type: text/x-diff
Size: 1330 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100107/006be81a/attachment.patch>



More information about the ffmpeg-devel mailing list