Re: [FFmpeg-devel] [PATCH] -- Add input mode autodetect to the decklink module.

2016-05-12 Thread Felt, Patrick
Ah… I see a bug. I should either be setting ctx->video in the if block or I should be breaking instead of continuing. Good catch! On 5/12/16, 1:32 PM, "ffmpeg-devel on behalf of Matthias Hunstock" wrote: >Am 12.05.2016 um 19:16 schrieb Felt, Patrick: >> +while (!ctx->video) { >> +

Re: [FFmpeg-devel] [PATCH] -- Add input mode autodetect to the decklink module.

2016-05-12 Thread Felt, Patrick
Ah. Didn’t realize we had that. Will do. On 5/12/16, 1:37 PM, "ffmpeg-devel on behalf of Marton Balint" wrote: > > >On Thu, 12 May 2016, Matthias Hunstock wrote: > >> Am 12.05.2016 um 19:16 schrieb Felt, Patrick: >>> +while (!ctx->video) { >>> +if (autodetect_delay--) { >>

Re: [FFmpeg-devel] [PATCH] -- Add input mode autodetect to the decklink module.

2016-05-12 Thread Matthias Hunstock
Am 12.05.2016 um 19:16 schrieb Felt, Patrick: > +while (!ctx->video) { > +if (autodetect_delay--) { > +/* this could indicate we are in the right mode. let's > assume so */ > +continue; > +} > +sleep(1); > +} I d

Re: [FFmpeg-devel] [PATCH] -- Add input mode autodetect to the decklink module.

2016-05-12 Thread Marton Balint
On Thu, 12 May 2016, Matthias Hunstock wrote: Am 12.05.2016 um 19:16 schrieb Felt, Patrick: +while (!ctx->video) { +if (autodetect_delay--) { +/* this could indicate we are in the right mode. let's assume so */ +continue; +} +

[FFmpeg-devel] [PATCH] -- Add input mode autodetect to the decklink module.

2016-05-12 Thread Felt, Patrick
Previously users had to supply the input format like this 'DeckLink Device@modenum'. This patch allows users to either leave it off completely, or supply 0 or negative number to indicate autodetect is requested. Autodetect only works the first time so if the mode changes mid stream you'll die.