[FFmpeg-devel] [PATCH 3/4] avformat/movenc: force colr atom for uncompressed yuv in mov

[Derek Buitenhuis]

On 11/22/2017 1:33 PM, Kevin Wheatley wrote:
> The guess work comes from this list:
> 
> https://developer.apple.com/library/content/technotes/tn2162/_index.html#//apple_ref/doc/uid/DTS40013070-CH1-TNTAG10-SAMPLE__COLR__SETTINGS

It's a mistake to conflate resolutions with the standards listed there. It does not
list any heuristics, just a set of standards and associated values. No info falls
under "Anything Else", IMO.

> and I agree is the source of a lot of chaos, but it is what several
> tools have done in the absence of any specified values.

"Other tools use heuristics too" is a bad rationale for creating such files by default.

> My usage of the feature is to write the atom when I need to by
> explicitly enabling it on the command line and I also explicitly set
> the metadata either from the input file, or by overriding on the
> command line.

This use-case is preserved if we leave the heuristics as an option, which I'm
totally OK with.

- Derek