Re: [FFmpeg-devel] [PATCH] avcodec: avutil: use avutil/thread.h instead of redundant conditional includes

2015-12-07 Thread Clément Bœsch
On Mon, Dec 07, 2015 at 03:12:19PM +0100, Clément Bœsch wrote: [...] Applied -- Clément B. signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec: avutil: use avutil/thread.h instead of redundant conditional includes

2015-12-07 Thread Ganesh Ajjanagadde
On Mon, Dec 7, 2015 at 10:57 AM, Clément Bœsch wrote: > On Mon, Dec 07, 2015 at 10:53:58AM -0500, Ganesh Ajjanagadde wrote: >> On Mon, Dec 7, 2015 at 10:25 AM, Clément Bœsch wrote: >> > On Mon, Dec 07, 2015 at 10:22:10AM -0500, Ronald S. Bultje wrote: >> >> Hi, >> >> >> >> On Mon, Dec 7, 2015 at

Re: [FFmpeg-devel] [PATCH] avcodec: avutil: use avutil/thread.h instead of redundant conditional includes

2015-12-07 Thread Clément Bœsch
On Mon, Dec 07, 2015 at 10:53:58AM -0500, Ganesh Ajjanagadde wrote: > On Mon, Dec 7, 2015 at 10:25 AM, Clément Bœsch wrote: > > On Mon, Dec 07, 2015 at 10:22:10AM -0500, Ronald S. Bultje wrote: > >> Hi, > >> > >> On Mon, Dec 7, 2015 at 10:15 AM, Clément Bœsch wrote: > >> > >> > On Mon, Dec 07, 20

Re: [FFmpeg-devel] [PATCH] avcodec: avutil: use avutil/thread.h instead of redundant conditional includes

2015-12-07 Thread Ganesh Ajjanagadde
On Mon, Dec 7, 2015 at 10:25 AM, Clément Bœsch wrote: > On Mon, Dec 07, 2015 at 10:22:10AM -0500, Ronald S. Bultje wrote: >> Hi, >> >> On Mon, Dec 7, 2015 at 10:15 AM, Clément Bœsch wrote: >> >> > On Mon, Dec 07, 2015 at 04:13:11PM +0100, Nicolas George wrote: >> > > Le septidi 17 frimaire, an CC

Re: [FFmpeg-devel] [PATCH] avcodec: avutil: use avutil/thread.h instead of redundant conditional includes

2015-12-07 Thread Clément Bœsch
On Mon, Dec 07, 2015 at 10:22:10AM -0500, Ronald S. Bultje wrote: > Hi, > > On Mon, Dec 7, 2015 at 10:15 AM, Clément Bœsch wrote: > > > On Mon, Dec 07, 2015 at 04:13:11PM +0100, Nicolas George wrote: > > > Le septidi 17 frimaire, an CCXXIV, Clement Boesch a écrit : > > > > From: Clément Bœsch >

Re: [FFmpeg-devel] [PATCH] avcodec: avutil: use avutil/thread.h instead of redundant conditional includes

2015-12-07 Thread Ronald S. Bultje
Hi, On Mon, Dec 7, 2015 at 10:15 AM, Clément Bœsch wrote: > On Mon, Dec 07, 2015 at 04:13:11PM +0100, Nicolas George wrote: > > Le septidi 17 frimaire, an CCXXIV, Clement Boesch a écrit : > > > From: Clément Bœsch > > > > > > --- > > > libavcodec/frame_thread_encoder.c | 9 + > > > li

Re: [FFmpeg-devel] [PATCH] avcodec: avutil: use avutil/thread.h instead of redundant conditional includes

2015-12-07 Thread Nicolas George
Le septidi 17 frimaire, an CCXXIV, Clement Boesch a écrit : > Will push soon Note: I only maintain one of the files you modify. > BTW, i think we should decide on av{util,codec,...} vs lav[ufc...] and > stick with it. This inconsistency is triggering my mild OCD. Indeed. Regards, -- Nicolas

Re: [FFmpeg-devel] [PATCH] avcodec: avutil: use avutil/thread.h instead of redundant conditional includes

2015-12-07 Thread Clément Bœsch
On Mon, Dec 07, 2015 at 04:13:11PM +0100, Nicolas George wrote: > Le septidi 17 frimaire, an CCXXIV, Clement Boesch a écrit : > > From: Clément Bœsch > > > > --- > > libavcodec/frame_thread_encoder.c | 9 + > > libavcodec/pthread_frame.c| 10 ++ > > libavcodec/utils.c

Re: [FFmpeg-devel] [PATCH] avcodec: avutil: use avutil/thread.h instead of redundant conditional includes

2015-12-07 Thread Nicolas George
Le septidi 17 frimaire, an CCXXIV, Clement Boesch a écrit : > From: Clément Bœsch > > --- > libavcodec/frame_thread_encoder.c | 9 + > libavcodec/pthread_frame.c| 10 ++ > libavcodec/utils.c| 9 + > libavcodec/vp8.h | 9 +---

[FFmpeg-devel] [PATCH] avcodec: avutil: use avutil/thread.h instead of redundant conditional includes

2015-12-07 Thread Clément Bœsch
From: Clément Bœsch --- libavcodec/frame_thread_encoder.c | 9 + libavcodec/pthread_frame.c| 10 ++ libavcodec/utils.c| 9 + libavcodec/vp8.h | 9 + libavutil/opencl.c| 10 +- libavutil/threadmess