Re: [FFmpeg-devel] [PATCH 8/8] avfilter/interlace: use AV_OPT_TYPE_BOOL

2015-09-08 Thread Clement Boesch
On Tue, Sep 08, 2015 at 10:35:34AM +, Paul B Mahol wrote: > On 9/6/15, Clement Boesch <u...@pkh.me> wrote: > > --- > > libavfilter/vf_interlace.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > lgtm applied, thx -- Clément

Re: [FFmpeg-devel] [PATCH 3/8] avdevice/dshow: use AV_OPT_TYPE_BOOL (UNTESTED)

2015-09-08 Thread Clement Boesch
On Tue, Sep 08, 2015 at 10:36:39AM +, Paul B Mahol wrote: > On 9/6/15, Clement Boesch <u...@pkh.me> wrote: > > --- > > libavdevice/dshow.c | 32 > > 1 file changed, 8 insertions(+), 24 deletions(-) > > > > lgtm a

Re: [FFmpeg-devel] [PATCH] lavc/magicyuv: fix undefined behaviour introduced in 8a135a55b

2016-06-19 Thread Clement Boesch
On Sun, Jun 19, 2016 at 07:00:50PM +0200, Paul B Mahol wrote: > On 6/19/16, Clement Boesch <u...@pkh.me> wrote: > > Order of evaluation of parameters in C is not defined. > > --- > > libavcodec/magicyuv.c | 6 -- > > 1 file changed, 4 insertions(+),

Re: [FFmpeg-devel] [PATCH] lavfi/ebur128: relicense to LGPL

2016-11-27 Thread Clement Boesch
On Sun, Nov 27, 2016 at 08:40:43PM +0100, Paul B Mahol wrote: > On 11/26/16, Clement Boesch <u...@pkh.me> wrote: > > All copyright holders have agreed to the relicensing. > > > > Approved-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com> > > Approved

Re: [FFmpeg-devel] [PATCH] lavfi/transpose: add missing const options flags

2016-12-27 Thread Clement Boesch
On Tue, Dec 27, 2016 at 11:31:17AM +0100, Paul B Mahol wrote: > On 12/27/16, Clement Boesch <u...@pkh.me> wrote: > > Fixes ffmpeg -h filter=transpose > > > > --- > > Should we instead ignore the flags for AV_OPT_TYPE_CONST? > > --- > > libavfilter/vf