[FFmpeg-devel] [PATCH 2/6] decklink: Introduce support for capture of multiple audio streams

Devin Heitmueller dheitmueller at ltnglobal.com
Fri Nov 17 02:32:59 EET 2017

> On Nov 16, 2017, at 7:22 PM, Derek Buitenhuis <derek.buitenhuis at gmail.com> wrote:
> On 11/16/2017 6:34 PM, Devin Heitmueller wrote:
>> +                uint8_t *audio_in = ((uint8_t *) audioFrameBytes) + audio_offset;
>> +                for (int x = 0; x < pkt.size; x += sample_size) {
> I realize this is C++, but I'm not sure if we still try to stick
> to our C style (aka no mixed variable decls) in our C++ too.

I don’t have strong feelings either way.  I’m happy to jam this into a subsequent cleanup patch if nobody has an objection (it’s just much easier since I have about 15 commits after this one in my Git tree).


More information about the ffmpeg-devel mailing list