[FFmpeg-devel] [FFmpeg-cvslog] fftools/ffmpeg: drop the -async option

[Anton Khirnov]

Quoting Michael Niedermayer (2022-10-04 17:39:09)
> On Tue, Oct 04, 2022 at 09:56:29AM +0000, Anton Khirnov wrote:
> > ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Tue Aug 30 11:14:13 2022 +0200| [3d86a13b47b726e49c2d780c5f723c290e8a36b4] | committer: Anton Khirnov
> > 
> > fftools/ffmpeg: drop the -async option
> > 
> > It has been deprecated in favor of the aresample filter for almost 10
> > years.
> > 
> > Another thing this option can do is drop audio timestamps and have them
> > generated by the encoding code or the muxer, but
> > - for encoding, this can already be done with the setpts filter
> > - for muxing this should almost never be done as timestamp generation by
> >   the muxer is deprecated, but people who really want to do this can use
> >   the setts bitstream filter
> > 
> > > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=3d86a13b47b726e49c2d780c5f723c290e8a36b4
> > ---
> > 
> >  doc/ffmpeg.texi         | 12 ------------
> >  fftools/ffmpeg.c        |  2 +-
> >  fftools/ffmpeg.h        |  1 -
> >  fftools/ffmpeg_filter.c | 10 ----------
> >  fftools/ffmpeg_mux.c    |  3 +--
> >  fftools/ffmpeg_opt.c    |  3 ---
> >  6 files changed, 2 insertions(+), 29 deletions(-)
> 
> This breaks:
> ./ffmpeg -i ~/tickets/507/avdesync_cut.avi -bitexact -r 24000/1001 -t 145 -async 1 file1-old.avi
> 
> The file is no longer AV-synced after this, and iam not sure the user
> will understand why
> its not warning or erroring out either

I suppose we could print a warning when swr options are specified, but
no resample filter is present. But then whether it is or is not present
can change at runtime, so not sure that's a good solution.

-- 
Anton Khirnov