[FFmpeg-devel] [PATCH] overlay: clear cur_buf on main input link.

Michael Niedermayer michaelni at gmx.at
Sun Jul 22 18:32:19 CEST 2012


On Sun, Jul 22, 2012 at 06:20:06PM +0200, Nicolas George wrote:
> A reference is kept in the queue and freed after use.
> 
> Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> ---
>  libavfilter/vf_overlay.c |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)

LGTM, maybe a assert inpicref == cur_buf could be added, it would
detect if the case where cur_buf by bug or API change is something
else as well as document implicitly why cur_buf is set to null

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120722/dc014f56/attachment.asc>


More information about the ffmpeg-devel mailing list