[FFmpeg-devel] [PATCH] avformat/movenc: write the colr atom by default

[Michael Bradshaw]

The write_colr flag has been marked as experimental for over 5 years. I
propose we do the following (as implemented by the attached patch):

   - 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
   - Remove the experimental notice from the write_colr flag.
   - 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

Please review.
--Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-avformat-movenc-write-the-colr-atom-by-default.patch
Type: application/octet-stream
Size: 3440 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200330/7d16e109/attachment.obj>