[FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

Marton Balint cus at passwd.hu
Fri Apr 14 01:40:37 EEST 2017


On Thu, 13 Apr 2017, Aaron Levinson wrote:

> On 4/13/2017 2:12 AM, Hendrik Leppkes wrote:
>> On Thu, Apr 13, 2017 at 10:36 AM, Aaron Levinson <alevinsn at aracnet.com> 
> wrote:
>> Give it some time for the other changes to be reviewed by the people
>> that actually know decklink itself, you can include that in any new
>> versions of the patch then, no need to send one for that right now.
>>
>> - Hendrik
>
> Actually, the coding changes made to the decklink source files in this 
> patch have pretty much nothing to do with decklink itself, and anyone 
> with familiarity with semaphores and pthread could review those changes.
>  In fact, all I really did was use already existing approaches for 
> replacing a semaphore with the combination of a condition variable, 
> mutex, and counter, and there are plenty of examples of how to do this 
> on the Web.
>

Yeah, the changes look fine, please send an updated patch, I will apply it 
after some final testing.

Thanks,
Marton


More information about the ffmpeg-devel mailing list