On 14/05/2020 18:19, Thierry Foucu wrote:
> Looking at
> https://developer.apple.com/library/archive/documentation/QuickTime/QTFF/Metadata/Metadata.html
> The spec does not seem to say 1 or more. But because the `keys` atom is a
> list of indexes used by the `ilst` and the spec for `keys` said:
> Indexes into the metadata item keys atom are 1-based (1…entry_count).
> 
> having 2+ `keys` atoms will conflict with this, as which one will be index 1
> 
> I'm assuming that only `keys` could only work.
> instead of skipping the consecutive `keys` atom, I could change the code to
> free the keys and re-alloc it for each `keys` atom found. So you will get
> only the data from the last `keys` atom.
> 
> so, it is either the 1st `keys` or the last `keys`. Up to you.

LGTM as-is then.

- Derek
_______________________________________________
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