The original code requires CID to be 1256 or 1270 if CLV or CLF are set.
This is unnecessarily restrictive. If CLF is 1, then CID must be 1256 or
1270. But if CLF is 0, then CID may be different. If CLV is 1 or 2, then
CID must be in the range 1270-1274 (inclusive).

Thus, the original code would warn on files that were perfectly valid
(e.g., CID == 1271 && CLF == 0 && CLV == 1).

See SMPTE ST 2019-1:2016, section 7.2.5 ("Coding Control B").

Please review (and double check my interpretation of the standard).

Thanks,
--Michael

Attachment: 0001-lavc-dnxhddec-do-not-warn-if-CID-1271-CLF-0-CLV-0.patch
Description: Binary data

_______________________________________________
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