[FFmpeg-devel] [PATCH]lavf/matroska: Support codec id V_FFV1 for FFV1.

wm4 nfxjfg at googlemail.com
Wed Mar 1 14:36:26 EET 2017


On Wed, 1 Mar 2017 13:07:12 +0100
Michael Niedermayer <michael at niedermayer.cc> wrote:

> On Wed, Mar 01, 2017 at 09:43:57AM +0100, Carl Eugen Hoyos wrote:
> > 2017-03-01 4:40 GMT+01:00 Michael Niedermayer <michael at niedermayer.cc>:  
> > > On Tue, Feb 28, 2017 at 08:45:44PM +0100, Carl Eugen Hoyos wrote:  
> > >> Hi!
> > >>
> > >> Attached patch fixes ticket #6206 here.
> > >>
> > >> Please comment, Carl Eugen  
> > >
> > > previous ffmpeg versions dont play these files  
> > 
> > What do you suggest?  
> 
> support this at the demuxer side, backport such support then
> wait till its supported widely before switching the default in the
> muxer

So if I understand this right:
- the patch will change the muxer output to use V_FFV1 instead of using
  avi-compat muxing
- you demand that we wait with this because older ffmpeg versions would
  choke on the new version

So what about all the other muxers that exist in the wild and that will
break on this file? Will you fix them too? I think this suggestion of
waiting until FFmpeg can eat the new output and only then changing the
output is possibly slightly self-centric, and also assumes everyone is
going to update their copy of FFmpeg.

If you're afraid that this change could break demuxers, you probably
shouldn't change it at all.


More information about the ffmpeg-devel mailing list