[FFmpeg-devel] [PATCH 3/6] avcodec/apng: Dispose previous frame properly

Michael Niedermayer michaelni at gmx.at
Mon Jun 8 15:38:38 CEST 2015


On Mon, Jun 08, 2015 at 02:28:05PM +0200, Paul B Mahol wrote:
> On 6/7/15, Donny Yang <work at kota.moe> wrote:
> > The spec specifies the dispose operation as how the current (i.e.,
> > currently
> > being rendered) frame should be disposed when the next frame is blended onto
> > it
> >
> > This is contrary to ffmpeg's current behaviour of interpreting the dispose
> > operation as how the previous (i.e., already rendered) frame should be
> > disposed
> >
> > This patch fixes ffmpeg's behaviour to match those of the spec, which
> > involved
> > a rewrite of the blending function
> >
> > Signed-off-by: Donny Yang <work at kota.moe>
> > ---
> >  libavcodec/pngdec.c | 179
> > +++++++++++++++++++++++++++-------------------------
> >  1 file changed, 92 insertions(+), 87 deletions(-)
> >
> 
> lgtm

applied

thanks

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

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150608/7581801b/attachment.asc>


More information about the ffmpeg-devel mailing list