[FFmpeg-devel] [PATCHv2][RFC] mov: Write colr atom by default

[Robert Krüger]

On Tue, Feb 24, 2015 at 6:34 PM, Michael Niedermayer <michaelni at gmx.at>
wrote:

> On Tue, Feb 24, 2015 at 03:11:44PM +0000, Derek Buitenhuis wrote:
> > Some formats require this, such as v210. The consensus seems to be
> > to write it by default.
> >
> > Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
> > ---
> > This is a silent behavior change, and I have updated the API doc
> > accordingly.
> > ---
> >  doc/APIchanges        | 3 +++
> >  libavformat/movenc.c  | 2 +-
> >  libavformat/version.h | 4 ++--
> >  3 files changed, 6 insertions(+), 3 deletions(-)
>
> is this correct for all mov, mp4, 3gp, psp, ... ?
> or should it be writteb by default only for some of them ?
>
> neither am I aware of its use in iso-base formats nor is there a mention
of it in the two spec documents I found, so I guess limiting this to mov
seems at least the safer choice until someone comes up with proof that it
is used there.