[FFmpeg-devel] [PATCH 3/4] avdevice/decklink_dec: Added Closed caption decode from VANC
kjeyapal at akamai.com
Tue Sep 5 14:27:02 EEST 2017
>On 9/3/17, 8:19 PM, "Marton Balint" <cus at passwd.hu<mailto:cus at passwd.hu>> wrote:
>Why don't use you simply use uint16_t here as well? I think we typically
>try to avoid unsigned as a type.
Done. Changed it to uint16_t.
>This VANC parser does not seem right. VANC header can appear multiple
>times, you seem to parse only the first DID. Also, you should simply
>ignore unknown DID-s, ffmpeg will not be able to parse all types there
>are, there is no need to warn.
Thanks for pointing it out. Have handled it now. Also changed the warning log to debug log.
Please find the patch attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 9768 bytes
More information about the ffmpeg-devel