[FFmpeg-devel] [PATCH 1/4] avdevice/decklink_dec: Added VANC search for all resolutions

Jeyapal, Karthick kjeyapal at akamai.com
Tue Sep 5 13:32:46 EEST 2017

>On 9/3/17, 4:48 PM, "Marton Balint" <cus at passwd.hu<mailto:cus at passwd.hu>> wrote:

>That may well be true, but SMPTE 334M recommands to use lines "from
>the second line after the line specified for switching to the last line
>preceding active video, inclusive", so I'd skip at least the lines up to
>the switching line, in order not to decode stuff which in fact belongs to
>another source... And since we already keeping track of VANC line
>numbers for each mode, it's almost no extra code to do this properly.

>Instead of all the getter functions, I think it is simpler to create
>struct for a mode/field combination, and use an array to store the mode
>line settings. Then you can loop through the array and find your settings.

Hi Marton,

Thanks for your comments. Please find the patch attached, incorporating both the above suggestions.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-avdevice-decklink_dec-Added-VANC-search-for-all-reso.patch
Type: application/octet-stream
Size: 4800 bytes
Desc: 0001-avdevice-decklink_dec-Added-VANC-search-for-all-reso.patch
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170905/e1773f3e/attachment.obj>

More information about the ffmpeg-devel mailing list