> On Nov 16, 2017, at 7:22 PM, Derek Buitenhuis <derek.buitenh...@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).

Devin

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to