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

2017-02-22 Thread wm4
On Wed, 22 Feb 2017 14:55:23 -0800 Kyle Schwarz wrote: > DeckLink depends on pthread, and is silently disabled if pthread is missing. > > Also fixes w32pthreads to support C++. > --- > > -sem_post(&ctx->semaphore); > +pthread_mutex_unlock(&ctx->mutex); > > return S

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

2017-02-22 Thread Kyle Schwarz
On Wed, Feb 22, 2017 at 6:55 PM, Marton Balint wrote: > > On Wed, 22 Feb 2017, Kyle Schwarz wrote: > >> DeckLink depends on pthread, and is silently disabled if pthread is >> missing. >> >> Also fixes w32pthreads to support C++. > > > You should split this into two patches, one the w32pthreads C++

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

2017-02-22 Thread Marton Balint
On Wed, 22 Feb 2017, Kyle Schwarz wrote: DeckLink depends on pthread, and is silently disabled if pthread is missing. Also fixes w32pthreads to support C++. You should split this into two patches, one the w32pthreads C++ support, other the decklink patch. Also, I checked the decklink part

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

2017-02-22 Thread Kyle Schwarz
Indeed, an unintended change. 644 is preferred. On Wed, Feb 22, 2017 at 6:26 PM, Ricardo Constantino wrote: > On 22 February 2017 at 22:55, Kyle Schwarz wrote: > > > DeckLink depends on pthread, and is silently disabled if pthread is > > missing. > > > > Also fixes w32pthreads to support C++. >

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

2017-02-22 Thread Carl Eugen Hoyos
2017-02-22 23:55 GMT+01:00 Kyle Schwarz : > DeckLink depends on pthread, and is silently disabled if pthread is missing. > Also fixes w32pthreads to support C++. Please make this a separate patch. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffm

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

2017-02-22 Thread Ricardo Constantino
On 22 February 2017 at 22:55, Kyle Schwarz wrote: > DeckLink depends on pthread, and is silently disabled if pthread is > missing. > > Also fixes w32pthreads to support C++. > --- > compat/w32pthreads.h| 31 +++ > configure | 4 ++--

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

2017-02-22 Thread Kyle Schwarz
DeckLink depends on pthread, and is silently disabled if pthread is missing. Also fixes w32pthreads to support C++. --- compat/w32pthreads.h| 31 +++ configure | 4 ++-- libavdevice/decklink_common.cpp | 3 --- libavdevice/decklink_c