>* 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
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to