Looking at this further, Apple's HDR requirements[1] say for Dolby Vision:

----
The color (‘colr’) atom with these values shall be present.

   - Color Primaries shall be set to 2 (Unspecified).
   - Color Transfer Function Index shall be set to 2 (Unspecified).
   - Color Matrix Index shall be set to 2 (Unspecified).

----

In that case this patch is wrong because it will not write the colr atom.
The original code doesn't work for this either because it will try to guess
the values instead of writing "unspecified".

I'm going to abandon this patch for now.

[1]:
https://developer.apple.com/av-foundation/High-Dynamic-Range-Metadata-for-Apple-Devices.pdf

>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to