[FFmpeg-devel] [PATCH]lavc/amrwb: Output silence for frames marked as broken

Carl Eugen Hoyos ceffmpeg at gmail.com
Sat Apr 11 15:19:47 EEST 2020


Am Di., 7. Apr. 2020 um 16:05 Uhr schrieb Anton Khirnov <anton at khirnov.net>:
>
> Quoting Carl Eugen Hoyos (2020-04-06 15:32:43)
> > Am Mo., 6. Apr. 2020 um 10:42 Uhr schrieb Anton Khirnov <anton at khirnov.net>:
> > >
> > > Quoting Carl Eugen Hoyos (2020-04-05 17:37:15)
> > > > Am So., 5. Apr. 2020 um 02:01 Uhr schrieb Carl Eugen Hoyos <ceffmpeg at gmail.com>:
> > > >
> > > > > Attached patch makes the output of the file in ticket #7113 very
> > > > > similar to the reference decoder.
> > > >
> > > > New patch attached.
> > >
> > > You could say how is this patch different and why.
> >
> > The source file has changed, the original patch did not
> > apply anymore.
> >
> > > It seems that handling ctx->fr_cur_mode > MODE_SID
> > > is now gone for no obvious reason.
> >
> > The condition is dead code in current FFmpeg, if the
> > condition is moved, two error messages are shown for
> > every invalid (but not corrupt) frame isntead of one
> > which seems unneeded (but not worth mentioning)
> > to me.
>
> Okay then.

Patch split and applied.

Carl Eugen


More information about the ffmpeg-devel mailing list