[FFmpeg-devel] [PATCH] mov: Support mdcv and clli boxes for mastering display an color light level

Vittorio Giovara vittorio.giovara at gmail.com
Mon Nov 27 23:26:57 EET 2017


>* On 11/27/2017 5:20 PM, James Almer wrote:*> Pointless duplicate atoms :/ At least these don't use fixed point
> values, so they are nicer.

indeed

> I assume no file will have both smdm and mdcv, or coll and clli, so
> reusing the MOVStreamContext fields should be ok, but maybe free the
> pointers first?
that's very unlikely as one set is defined for iso files (.mp4) and
the other one for qtff files (.mov).
if you think that is still needed, I'll go ahead and add that to a
separate patch

> Can't comment about the implementation as i don't have access to the spec.

if it helps, mediainfo got these properties too
https://github.com/MediaArea/MediaInfoLib/commit/818f2fd4fdfadf2dbb8cc010b1c35e8cf3ada504
-- 
Vittorio


More information about the ffmpeg-devel mailing list