[FFmpeg-devel] [PATCH 1/6] lavu/fifo: add av_fifo_freep function

Michael Niedermayer michaelni at gmx.at
Thu May 8 04:12:27 CEST 2014


On Wed, May 07, 2014 at 11:46:32PM +0200, Lukasz Marek wrote:
> On 06.05.2014 22:04, Nicolas George wrote:
> >Le septidi 17 floréal, an CCXXII, Lukasz Marek a écrit :
> >>+void av_fifo_freep(AVFifoBuffer **f)
> >>+{
> >>+    if (f && *f) {
> >>+        av_free((*f)->buffer);
> >>+        av_freep(f);
> >>+    }
> >>+}
> >
> >I suggest:
> >
> >     if (f)
> >	av_fifo_free(*f);
> 
> Changed. Patch attached.
> 
> I also uploaded whole patchset into my repo, so may be merged when
> found fine.

merged

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140508/c124278d/attachment.asc>


More information about the ffmpeg-devel mailing list