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

Marton Balint cus at passwd.hu
Sat Apr 15 13:41:35 EEST 2017


On Thu, 13 Apr 2017, Aaron Levinson wrote:

> On 4/13/2017 3:40 PM, Marton Balint wrote:
>> 
>> 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
>
> Here's a new version of the patch with the pthreads dependency replaced with threads.
>

Thanks, applied.

Regards,
Marton


More information about the ffmpeg-devel mailing list