[FFmpeg-devel] [PATCH] avformat/movenc: use enum values directly for colr atom

Michael Bradshaw mjbshaw at google.com
Tue Mar 31 19:08:48 EEST 2020


The switch cases were missing:

   - Primaries: bt470m, film, smpte428, and ebu3213.
   - TRCs: gamma22, gamma28, linear, log, log_sqrt, iec61966_2_4, bt1361,
   iec61966_2_1, bt2020_10bit, and bt2020_12bit.
   - Space: rgb, fcc, ycgco, bt2020_cl, smpte2085, chroma-derived-nc,
   chroma-derived-c, and ictcp.

They also annoyingly remapped the following (which are functionally
equivalent but can be treated differently by clients):

  - smpte240m primaries to smpte170m.
  - smpte170m TRC to bt709.
  - bt470bg color space to smpte170m.

The enum values in FFmpeg are the same values as ITU-T H.273 and ISO/IEC
23001-8 so we can just use them directly, which is both simpler and
preserves the user intent.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-avformat-movenc-use-enum-values-directly-for-colr-at.patch
Type: application/octet-stream
Size: 3039 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200331/cc8e25c6/attachment.obj>


More information about the ffmpeg-devel mailing list