[FFmpeg-devel] [PATCH] lavc/dnxhddec: do not warn if CID == 1271 && CLF == 0 && CLV > 0

[Michael Bradshaw]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavc-dnxhddec-do-not-warn-if-CID-1271-CLF-0-CLV-0.patch
Type: application/octet-stream
Size: 1867 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190927/284d1122/attachment.obj>