Re: [FFmpeg-devel] [PATCH 01/11] libavdevice/decklink: Add support for EIA-708 output over SDI

2018-01-12 Thread Marton Balint
On Mon, 8 Jan 2018, Devin Heitmueller wrote: Hook in libklvanc and use it for output of EIA-708 captions over SDI. The bulk of this patch is just general support for ancillary data for the Decklink SDI module - the real work for construction of the EIA-708 CDP and VANC line construction is do

Re: [FFmpeg-devel] [PATCH 01/11] libavdevice/decklink: Add support for EIA-708 output over SDI

2018-01-10 Thread Moritz Barsnick
On Wed, Jan 10, 2018 at 09:11:35 -0500, Devin Heitmueller wrote: > For loop declaration has been part of the C standard since C99. I'm just quoting other similar discussions on this list. This has been a topic several (hundred? ;-)) times the last few months. It peaked in this discussion which was

Re: [FFmpeg-devel] [PATCH 01/11] libavdevice/decklink: Add support for EIA-708 output over SDI

2018-01-10 Thread Devin Heitmueller
Hello Moritz, > On Wed, Jan 10, 2018 at 14:29:43 +0100, Moritz Barsnick wrote: >>> +for (size_t i = 0; i < cc_count; i++) { >> Declare i outside the clause. >>> +for (int i = 0; i < vanc_lines.num_lines; i++) { >> Same here. > > Sorry, this rule may not apply to C++ files, as it's par

Re: [FFmpeg-devel] [PATCH 01/11] libavdevice/decklink: Add support for EIA-708 output over SDI

2018-01-10 Thread Moritz Barsnick
On Wed, Jan 10, 2018 at 14:29:43 +0100, Moritz Barsnick wrote: > > +for (size_t i = 0; i < cc_count; i++) { > Declare i outside the clause. > > +for (int i = 0; i < vanc_lines.num_lines; i++) { > Same here. Sorry, this rule may not apply to C++ files, as it's part of all C++ standards

Re: [FFmpeg-devel] [PATCH 01/11] libavdevice/decklink: Add support for EIA-708 output over SDI

2018-01-10 Thread Moritz Barsnick
On Mon, Jan 08, 2018 at 20:16:48 -0500, Devin Heitmueller wrote: > +if (ctx->supports_vanc == 0 || > ctx->dlo->DoesSupportVideoMode(ctx->bmd_mode, ctx->raw_format, In other places you (or the DeckLink code) use if (!ctx->supports_vanc || [...] which I believe is preferred. > +if

[FFmpeg-devel] [PATCH 01/11] libavdevice/decklink: Add support for EIA-708 output over SDI

2018-01-08 Thread Devin Heitmueller
Hook in libklvanc and use it for output of EIA-708 captions over SDI. The bulk of this patch is just general support for ancillary data for the Decklink SDI module - the real work for construction of the EIA-708 CDP and VANC line construction is done by libklvanc. Libklvanc can be found at: https