[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