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

Michael Niedermayer michael at niedermayer.cc
Wed Mar 1 15:26:34 EET 2017


On Wed, Mar 01, 2017 at 01:36:26PM +0100, wm4 wrote:
> 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.

You seem to be reading things in the mail that it didnt say.

A reply to "What do you suggest?" is a suggestion not a demand
and
The suggestion does not refer to FFmpeg and it was not intended to
imply to refer to FFmpeg, especially the
"wait till its supported widely" was intended to refer to all software.

Its bad if we generate files by default noone can play back
I dont know how widespread V_FFV1 support is but i assume its not
widespread yet at all


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

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170301/4cd1d5b0/attachment.sig>


More information about the ffmpeg-devel mailing list