[FFmpeg-devel] [PATCH] smacker: remove unnecessary call to avctx->release_buffer in decode_frame()

Michael Niedermayer michaelni at gmx.at
Thu Apr 28 16:49:10 CEST 2011


On Wed, Apr 27, 2011 at 12:14:51PM +0200, Stefano Sabatini wrote:
> On date Tuesday 2011-04-26 11:37:36 +0200, Stefano Sabatini encoded:
> > The release_buffer was cleaning the provided frame, thus causing the
> > successive call to avctx->reget_buffer() to allocate a new frame.  In
> > case the returned frame was not the same one previously returned but a
> > new one with different data, it resulted in artifacts.
> > 
> > Fix trac issue #87.
> 
> Adding libav-devel to recipients, so Kostya can comment on this (this
> regression was discussed some time ago on irc).

>  smacker.c |    2 --
>  1 file changed, 2 deletions(-)
> 25c965db02dbcebf7711d3175189a1c921b32202  0001-smacker-remove-unnecessary-call-to-avctx-release_buf.patch
> From 843caf61b77925152d928a8ba53f7a559b1a98fc Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> Date: Tue, 26 Apr 2011 11:31:21 +0200
> Subject: [PATCH] smacker: remove unnecessary call to avctx->release_buffer in decode_frame()
> 
> The release_buffer was cleaning the provided frame, thus causing the
> successive call to avctx->reget_buffer() to allocate a new frame.  In
> case the returned frame was not the same one previously returned but a
> new one with different data, it resulted in artifacts.
> 
> Fix trac issue #87.

has been applied & pulled

[...]

-- 
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: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110428/7da5c255/attachment.asc>


More information about the ffmpeg-devel mailing list