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

[Carl Eugen Hoyos]

Am Fr., 27. Sept. 2019 um 09:59 Uhr schrieb Michael Bradshaw
<mjbshaw-at-google.com at ffmpeg.org>:
>
> 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).

I suggest you push if nobody comments for a few days;-)

Carl Eugen