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

[Jeyapal, Karthick]

>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.

Regards,
Karthick
-------------- 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>