]> git.sesse.net Git - ffmpeg/commit
avformat/movenc: write the colr atom by default
authorMichael Bradshaw <mjbshaw@google.com>
Mon, 13 Apr 2020 17:11:38 +0000 (11:11 -0600)
committerMichael Bradshaw <mjbshaw@google.com>
Fri, 21 Aug 2020 17:01:58 +0000 (10:01 -0700)
commitc5b20cfe19c3cd1bedf637e4caf0feb245d0ad06
tree6a7b00ce3565113c273bd0cf7d76b8289e3813c5
parent242ba4d74cc95aa78528e4496de7cc63816a877b
avformat/movenc: write the colr atom by default

The write_colr flag has been marked as experimental for over 5 years.
It should be safe to enable its behavior by default as follows:

  - Write the colr atom by default for mp4/mov if any of the following:
     - The primaries/trc/matrix are all specified, OR
     - There is an ICC profile, OR
     - The user specified +write_colr
  - Keep the write_colr flag for situations where the user wants to
    write the colr atom even if the color info is unspecified (e.g.,
    http://ffmpeg.org/pipermail/ffmpeg-devel/2020-March/259334.html)

This fixes https://trac.ffmpeg.org/ticket/7961

Signed-off-by: Michael Bradshaw <mjbshaw@google.com>
libavformat/movenc.c
tests/ref/fate/copy-trac236