Re: [FFmpeg-devel] [PATCH 1/2] avutil/threadmessage: add av_thread_message_flush()

2015-12-01 Thread Clément Bœsch
On Tue, Dec 01, 2015 at 05:11:06PM +0100, Nicolas George wrote: > Le decadi 10 frimaire, an CCXXIV, Clement Boesch a écrit : > > From: Clément Bœsch <clem...@stupeflix.com> > > > > --- > > libavutil/threadmessage.c | 37 ++--- &g

Re: [FFmpeg-devel] [PATCH 5/5] lavfi: make request_frame() non-recursive.

2015-12-01 Thread Clément Bœsch
On Tue, Dec 01, 2015 at 05:26:23PM +0100, Nicolas George wrote: > Le decadi 10 frimaire, an CCXXIV, Hendrik Leppkes a écrit : > > We generally just use ints for boolean properties, any particular > > reason this uses unsigned instead? > > Just a matter of personal habit, justified below. But as I

Re: [FFmpeg-devel] [PATCH 1/2] avutil/threadmessage: add av_thread_message_flush()

2015-12-01 Thread Clément Bœsch
On Tue, Dec 01, 2015 at 05:34:39PM +0100, Nicolas George wrote: > Le primidi 11 frimaire, an CCXXIV, Clement Boesch a écrit : > > Your suggestion is twice smaller. > > I guess I was not clear enough: I was not asking about merits of this > implementations over yours, but about the usefulness of

[FFmpeg-devel] [PATCH 2/2] fate/api: test threadmessage

2015-11-30 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- tests/api/Makefile | 1 + tests/api/api-threadmessage-test.c | 202 + tests/fate/api.mak | 6 ++ 3 files changed, 209 insertions(+) create mode 100644 tests/a

[FFmpeg-devel] [PATCH 1/2] avutil/threadmessage: add av_thread_message_flush()

2015-11-30 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- libavutil/threadmessage.c | 37 ++--- libavutil/threadmessage.h | 21 ++--- 2 files changed, 52 insertions(+), 6 deletions(-) diff --git a/libavutil/threadmessage.c b/libavutil/threadmessage.c

[FFmpeg-devel] threadmessage improvements

2015-11-30 Thread Clément Bœsch
So I've been playing with threadmessage a bit. Since I'm trying to extend the API for (currently) external usage, I'm also adding an API test. I've been triggering deadlocks thoughm, typically by setting queue_size=1. I haven't investigated but I think the issue lies in the fact that there is a

Re: [FFmpeg-devel] [PATCH] avfilter: add audio pulsator filter

2015-11-29 Thread Clément Bœsch
On Sun, Nov 29, 2015 at 06:39:02PM +0100, Moritz Barsnick wrote: > Hi, > > On Sat, Nov 28, 2015 at 23:26:44 +0100, Paul B Mahol wrote: > > In addition to Ganesh's comments: > > > +@item amount > > +Set modulation. Define how much of original signal is affected by the LFO. > [...] > > +@item

Re: [FFmpeg-devel] [PATCH] avfilter: add audio pulsator filter

2015-11-29 Thread Clément Bœsch
On Sun, Nov 29, 2015 at 01:11:07PM -0500, Ganesh Ajjanagadde wrote: > On Sun, Nov 29, 2015 at 12:45 PM, Clément Bœsch <u...@pkh.me> wrote: > > On Sun, Nov 29, 2015 at 06:39:02PM +0100, Moritz Barsnick wrote: > >> Hi, > >> > >> On Sat, Nov 28,

Re: [FFmpeg-devel] [PATCH 1/5] lavf: add FFERROR_REDO to let demuxer return no packet.

2015-11-28 Thread Clément Bœsch
On Sat, Nov 28, 2015 at 09:08:52AM +0100, Nicolas George wrote: > Le septidi 7 frimaire, an CCXXIV, Clement Boesch a écrit : > > "returned internally by demuxers" maybe > > That sounds to me like an oxymoron. IMHO, being in internal.h and named > FFERROR instead of AVERROR should be enough (even

Re: [FFmpeg-devel] [PATCH 1/5] lavf: add FFERROR_REDO to let demuxer return no packet.

2015-11-27 Thread Clément Bœsch
On Fri, Nov 27, 2015 at 07:16:06PM +0100, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > libavformat/internal.h | 6 ++ > libavformat/utils.c| 5 + > 2 files changed, 11 insertions(+) > > > Apparently, the most unhappyness was caused by the error

Re: [FFmpeg-devel] [PATCH 2/3] lavf/utils: handle AVERROR_REDO.

2015-11-27 Thread Clément Bœsch
On Fri, Nov 27, 2015 at 01:10:29PM +0100, Nicolas George wrote: > Le septidi 7 frimaire, an CCXXIV, Clement Boesch a écrit : > > But then it's still exposed by the API, and someone looking at handling > > every error code might be wondering how to handle it. > > « Handling every error code »

Re: [FFmpeg-devel] [PATCHv3 1/3] avutil/tablegen: add tablegen compatibility hacks

2015-11-27 Thread Clément Bœsch
On Fri, Nov 27, 2015 at 06:32:40AM -0500, Ganesh Ajjanagadde wrote: > On Fri, Nov 27, 2015 at 4:46 AM, Clément Bœsch <u...@pkh.me> wrote: > > On Thu, Nov 26, 2015 at 11:58:10AM -0500, Ganesh Ajjanagadde wrote: > >> Reviewed-by: Ronald S. Bultje <rsbul...@gmail.com&

Re: [FFmpeg-devel] [PATCHv3 1/3] avutil/tablegen: add tablegen compatibility hacks

2015-11-27 Thread Clément Bœsch
On Thu, Nov 26, 2015 at 11:58:10AM -0500, Ganesh Ajjanagadde wrote: > Reviewed-by: Ronald S. Bultje > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/tablegen.h | 33 + > 1 file changed, 33 insertions(+) > create

Re: [FFmpeg-devel] [PATCH 2/3] lavf/utils: handle AVERROR_REDO.

2015-11-27 Thread Clément Bœsch
On Thu, Nov 26, 2015 at 11:42:03PM +0100, Nicolas George wrote: > Le sextidi 6 frimaire, an CCXXIV, Clement Boesch a écrit : > > > An option can be added later to grant applications fine-grained control on > > > the looping, but it can not be the default as it would be an API change, > > > and >

Re: [FFmpeg-devel] [PATCH 2/3] lavf/utils: handle AVERROR_REDO.

2015-11-26 Thread Clément Bœsch
On Thu, Nov 26, 2015 at 07:47:53PM +0100, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > libavformat/utils.c | 2 ++ > 1 file changed, 2 insertions(+) > > > An option can be added later to grant applications fine-grained control on > the looping, but it can not

Re: [FFmpeg-devel] [PATCH 1/5] avcodec: use AV_OPT_TYPE_BOOL in a bunch of places

2015-11-25 Thread Clément Bœsch
On Sat, Nov 21, 2015 at 10:17:50PM +0100, Clément Bœsch wrote: > From: Clément Bœsch <clem...@stupeflix.com> > > --- > libavcodec/ac3dec_fixed.c | 2 +- > libavcodec/ac3dec_float.c | 2 +- > libavcodec/ac3enc_opts_template.c | 4 ++-- > libavcodec/

Re: [FFmpeg-devel] [PATCH] avutil/WIP: add AVAsync API

2015-11-24 Thread Clément Bœsch
On Fri, Nov 13, 2015 at 03:57:22PM +0100, Nicolas George wrote: > Le primidi 21 brumaire, an CCXXIV, Clement Boesch a écrit : > > You construct something like this: > > > > AVAsyncContext( > > AVAsyncReader( > > AVAsyncDecoder(...), > > AVAsyncDecoder(...), > >

Re: [FFmpeg-devel] [PATCH] lavd/avfoundation: overhaul

2015-11-24 Thread Clément Bœsch
On Tue, Nov 24, 2015 at 07:27:57PM +0100, Thilo Borgmann wrote: > Hi, > > this patchset fixes most if not all bugs reported to trac for me. > Thanks. BTW, did you see http://lists.libav.org/pipermail/libav-devel/2015-November/073162.html ? Are you in contact with the author or something? Any

Re: [FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-23 Thread Clément Bœsch
On Sun, Nov 22, 2015 at 04:27:02PM +0100, Michael Niedermayer wrote: [...] > > > IIUC src_x/y are only an approximation, as the exact motion vector > > > isnt an integer in fullpel units so it would be mostly cosmetic > > > if x + 4 >> 8, or /8 is used > > > > yes > > > > mpeg used x >> n, i

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo: use constants for rc_strategy

2015-11-23 Thread Clément Bœsch
On Sat, Nov 21, 2015 at 10:26:17PM +0100, Michael Niedermayer wrote: > On Sat, Nov 21, 2015 at 09:28:31PM +0100, Clément Bœsch wrote: > > --- > > libavcodec/mpegvideo.h | 10 +- > > libavcodec/ratecontrol.c | 10 +++--- > > 2 files changed, 16

Re: [FFmpeg-devel] [PATCH] swscale/arm: add ff_nv{12, 21}_to_{argb, rgba, abgr, bgra}_neon

2015-11-23 Thread Clément Bœsch
On Thu, Nov 19, 2015 at 11:48:53AM +0100, Clément Bœsch wrote: > From: Matthieu Bouron <matthieu.bou...@stupeflix.com> > > Signed-off-by: Matthieu Bouron <matthieu.bou...@stupeflix.com> > Signed-off-by: Clément Bœsch <clem...@stupeflix.com> > > --- > The

Re: [FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-23 Thread Clément Bœsch
On Mon, Nov 23, 2015 at 03:54:19PM +0100, Michael Niedermayer wrote: > On Mon, Nov 23, 2015 at 11:01:02AM +0100, Clément Bœsch wrote: > > On Sun, Nov 22, 2015 at 04:27:02PM +0100, Michael Niedermayer wrote: > > [...] > > > > > IIUC src_x/y are only an approximati

Re: [FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-22 Thread Clément Bœsch
On Sun, Nov 22, 2015 at 03:24:55AM +0100, Michael Niedermayer wrote: [...] > > diff --git a/libavcodec/snowdec.c b/libavcodec/snowdec.c > > index 1b288dd..5eb775a 100644 > > --- a/libavcodec/snowdec.c > > +++ b/libavcodec/snowdec.c > > @@ -104,8 +104,11 @@ static av_always_inline void > >

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/dct32_template: use M_SQRT1_2 instead of actual literal

2015-11-22 Thread Clément Bœsch
On Sun, Nov 22, 2015 at 10:24:49PM +0100, Ganesh Ajjanagadde wrote: > ffmpeg | branch: master | Ganesh Ajjanagadde | Tue > Nov 10 22:26:25 2015 -0500| [ccf3c694032bd84c7672979fa89f437980d717ba] | > committer: Ganesh Ajjanagadde > > avcodec/dct32_template: use M_SQRT1_2

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/dct32_template: use M_SQRT1_2 instead of actual literal

2015-11-22 Thread Clément Bœsch
On Sun, Nov 22, 2015 at 10:38:35PM +0100, Clément Bœsch wrote: > On Sun, Nov 22, 2015 at 10:24:49PM +0100, Ganesh Ajjanagadde wrote: > > ffmpeg | branch: master | Ganesh Ajjanagadde <gajjanaga...@gmail.com> | Tue > > Nov 10 22:26:25 2015 -0500| [ccf3c694032bd84c767

Re: [FFmpeg-devel] [PATCH] avfilter: add deflash filter

2015-11-21 Thread Clément Bœsch
On Fri, Nov 20, 2015 at 08:13:11PM +0100, Paul B Mahol wrote: [...] > +for (y = 0; y < h; y++) { > +for (x = 0; x < w; x++) { > +int diff = next[x] - src[x]; > +int t = 0; > + > +for (z = 0; z < FF_BUFQUEUE_SIZE - 1; z++) { > +

[FFmpeg-devel] [PATCH 3/5] avformat: use AV_OPT_TYPE_BOOL in a bunch of places

2015-11-21 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- libavformat/adtsenc.c| 4 ++-- libavformat/aiffenc.c| 2 +- libavformat/apngdec.c| 2 +- libavformat/asfdec_f.c | 4 ++-- libavformat/assenc.c | 2 +- libavformat/av

[FFmpeg-devel] [PATCH 5/5] swscale: use AV_OPT_TYPE_BOOL for {src, dst}_range options

2015-11-21 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- libswscale/options.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libswscale/options.c b/libswscale/options.c index e666e5a..c1ea336 100644 --- a/libswscale/options.c +++ b/libswscale/options.c @@ -59,8 +59,8 @@

[FFmpeg-devel] [PATCH 4/5] swresample: use AV_OPT_TYPE_BOOL for linear_interp and cheby options

2015-11-21 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- libswresample/options.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libswresample/options.c b/libswresample/options.c index 654102a..eac1a7c 100644 --- a/libswresample/options.c +++ b/libswresample/options.c @@

[FFmpeg-devel] [PATCH 1/5] avcodec: use AV_OPT_TYPE_BOOL in a bunch of places

2015-11-21 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- libavcodec/ac3dec_fixed.c | 2 +- libavcodec/ac3dec_float.c | 2 +- libavcodec/ac3enc_opts_template.c | 4 ++-- libavcodec/alac.c | 2 +- libavcodec/cinepakenc.c | 2 +- libavcodec/dc

[FFmpeg-devel] [PATCH 2/5] avfilter: use AV_OPT_TYPE_BOOL in a bunch of places

2015-11-21 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- libavfilter/avf_showwaves.c | 2 +- libavfilter/vf_pullup.c | 2 +- libavfilter/vf_scale.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libavfilter/avf_showwaves.c b/libavfilter/avf_showwaves.c index 7

Re: [FFmpeg-devel] [PATCH] avfilter/WIP: add streamselect filter

2015-11-20 Thread Clément Bœsch
On Thu, Nov 19, 2015 at 06:47:07PM +0100, Nicolas George wrote: > Le nonidi 29 brumaire, an CCXXIV, Clement Boesch a écrit : > > - I always receive frames from the first input link... I suppose I need > > to write a "smart" request_frame callback, but things changed recently > > so maybe there

Re: [FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-20 Thread Clément Bœsch
On Fri, Nov 20, 2015 at 10:36:36AM +0100, Clément Bœsch wrote: > On Fri, Nov 20, 2015 at 10:33:04AM +0100, Clément Bœsch wrote: > [...] > > > i think this doesnt support libavcodec/tpeldsp.c as used in svq3 > > > that would need a motion_xy / 3 > > > &

Re: [FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-20 Thread Clément Bœsch
On Fri, Nov 20, 2015 at 10:33:04AM +0100, Clément Bœsch wrote: [...] > > i think this doesnt support libavcodec/tpeldsp.c as used in svq3 > > that would need a motion_xy / 3 > > > > i might be missing something but > > > > * Motion vector > >

Re: [FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-20 Thread Clément Bœsch
On Mon, Nov 16, 2015 at 02:49:05AM +0100, Michael Niedermayer wrote: > On Thu, Nov 12, 2015 at 03:03:33PM +0100, Clément Bœsch wrote: > > --- > > libavcodec/mpegvideo.c | 36 +--- > > libavcodec/snowdec.c | 4 > > liba

[FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-20 Thread Clément Bœsch
--- FATE test changes because of the replacement of >>n with /(1<

[FFmpeg-devel] [PATCH] swscale/arm: add ff_nv{12, 21}_to_{argb, rgba, abgr, bgra}_neon

2015-11-19 Thread Clément Bœsch
From: Matthieu Bouron <matthieu.bou...@stupeflix.com> Signed-off-by: Matthieu Bouron <matthieu.bou...@stupeflix.com> Signed-off-by: Clément Bœsch <clem...@stupeflix.com> --- The function takes about 29ms with a 1080p source (testsrc2) on a cortex-a8. Though, 16ms (more

Re: [FFmpeg-devel] [PATCH] ffmpeg: enable echoing with command and debug modes

2015-11-19 Thread Clément Bœsch
On Thu, Nov 19, 2015 at 11:16:46PM +0100, Michael Niedermayer wrote: > On Thu, Nov 19, 2015 at 10:54:45PM +0100, Clément Bœsch wrote: > > Allow seeing text when pressing 'c' or 'd'. > > --- > > ffmpeg.c | 18 ++ > > 1 file changed, 18 insertion

[FFmpeg-devel] [PATCH] ffmpeg: enable echoing with command and debug modes

2015-11-19 Thread Clément Bœsch
Allow seeing text when pressing 'c' or 'd'. --- ffmpeg.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/ffmpeg.c b/ffmpeg.c index c4e9280..8d558e3 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -3395,6 +3395,18 @@ static OutputStream *choose_output(void) return ost_min; }

Re: [FFmpeg-devel] [PATCH] swscale/arm: add ff_nv{12, 21}_to_{argb, rgba, abgr, bgra}_neon

2015-11-19 Thread Clément Bœsch
On Thu, Nov 19, 2015 at 11:48:53AM +0100, Clément Bœsch wrote: [...] > +vst2.32 {q6, q7}, [\dst,:128]! Sorry, this is supposed to be vst4.8 -- Clément B. signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffm

[FFmpeg-devel] [PATCH] avfilter/WIP: add streamselect filter

2015-11-19 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- No documentation so the concept is simply to map N streams to M other streams, with the possibility to live configure the filter (process_command). For example, let's say you have 2 a/v inputs and you want to live switch between the tw

Re: [FFmpeg-devel] [PATCH] swscale/arm: add ff_nv{12, 21}_to_{argb, rgba, abgr, bgra}_neon

2015-11-19 Thread Clément Bœsch
On Thu, Nov 19, 2015 at 04:50:54PM +0100, Michael Niedermayer wrote: > On Thu, Nov 19, 2015 at 11:48:53AM +0100, Clément Bœsch wrote: > > From: Matthieu Bouron <matthieu.bou...@stupeflix.com> > > > > Signed-off-by: Matthieu Bouron <matthieu.bou...@stupeflix.com>

[FFmpeg-devel] [SURVEY] Summary of first 2015 survey

2015-11-15 Thread Clément Bœsch
Hi folks, So the October/November 2015 FFmpeg survey ended, so as promised here is a summary. It was planed for yesterday but due to some unfortunate national issues it was kind of postponed to today... We got 123 answers. I was expecting more but that's a start. 35,54% (43) A developer using

Re: [FFmpeg-devel] [PATCH] avutil: Move av_rint64_clip_* to internal.h

2015-11-15 Thread Clément Bœsch
On Sun, Nov 15, 2015 at 10:24:52AM -0500, Ganesh Ajjanagadde wrote: [...] > >> > IMO rename it to avpriv_rint64_clip() or even ff_rint64_clip() since > >> > it's inlined > >> > and not public/exported. > >> > >> Just noticed an issue: Ronald mentioned to me that ffserver and other > >> such

Re: [FFmpeg-devel] [PATCH] avutil/WIP: add AVAsync API

2015-11-12 Thread Clément Bœsch
On Thu, Nov 12, 2015 at 10:15:16PM +0100, wm4 wrote: > On Wed, 11 Nov 2015 08:45:52 +0100 > Hendrik Leppkes <h.lepp...@gmail.com> wrote: > > > On Wed, Nov 11, 2015 at 1:27 AM, Clément Bœsch <u...@pkh.me> wrote: > > > From: Clément Bœsch <clem...@stupe

[FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-12 Thread Clément Bœsch
--- libavcodec/mpegvideo.c | 36 +--- libavcodec/snowdec.c | 4 libavfilter/vf_codecview.c | 7 +-- libavutil/motion_vector.h | 8 libavutil/version.h| 2 +- 5 files changed, 39 insertions(+), 18 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] all: use predefined mathematics macros

2015-11-11 Thread Clément Bœsch
On Tue, Nov 10, 2015 at 10:35:23PM -0500, Ganesh Ajjanagadde wrote: > This uses M_SQRT2, M_PI, and M_E instead of the actual literals. > Benefits include: > 1. Reduced scope for copy/paste errors and improved readability. > 2. Consistency across the codebase. > 3. Actually fixes an incorrect

[FFmpeg-devel] [PATCH] avutil/WIP: add AVAsync API

2015-11-10 Thread Clément Bœsch
From: Clément Bœsch <clem...@stupeflix.com> --- So here is a first prototype of a higher level API following an asynchronous model to (for now) simply use the current synchronous API. I suggest to look at libavutil/async.h (sorry no doxy yet) and doc/examples/async_demuxing_deco

Re: [FFmpeg-devel] [PATCHv2] avfilter: add anoisesrc

2015-11-05 Thread Clément Bœsch
On Thu, Nov 05, 2015 at 09:49:32PM +0100, Nicolas George wrote: > Le quintidi 15 brumaire, an CCXXIV, Paul B Mahol a écrit : > > Hmm, why not simple enum/int? > > Did you miss the fact that AV_OPT_TYPE_FLAG is exactly for that? > AV_OPT_TYPE_FLAG is for concatenated "bool" flags (foo+bar+baz),

Re: [FFmpeg-devel] [PATCHv3 1/4] fate: add concat demuxer tests

2015-11-05 Thread Clément Bœsch
On Wed, Nov 04, 2015 at 03:33:16PM +0100, Nicolas George wrote: > Le quartidi 14 brumaire, an CCXXIV, Marton Balint a écrit : > > + run ffprobe${PROGSUF} -show_streams -show_packets -v 0 -fflags > > keepside -safe 0 $extra_args $concatfile > $packetfile > > Maybe -of compact, that makes the

Re: [FFmpeg-devel] [PATCH 2/2] all: use FFDIFFSIGN to resolve possible undefined behavior in comparators

2015-11-03 Thread Clément Bœsch
On Sun, Nov 01, 2015 at 07:29:36PM -0500, Ganesh Ajjanagadde wrote: [...] > I believe vf_palettegen and libavformat/subtitles are the only ones > lacking review, but may be mistaken. > Those are probably fine. -- Clément B. signature.asc Description: PGP signature

Re: [FFmpeg-devel] [PATCH] swresample/resample: improve bessel function accuracy

2015-11-03 Thread Clément Bœsch
On Tue, Nov 03, 2015 at 07:20:39AM -0500, Ganesh Ajjanagadde wrote: [...] > This is an opinion, so I will state mine here: if you are developing > use ccache + GCC > ccache + clang > clang = gcc. Reason for the first > is due to the terrible interaction ccache has with clang. I'm curious; what

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Clément Bœsch
On Sat, Oct 31, 2015 at 12:33:54AM +0100, Michael Niedermayer wrote: > On Sat, Oct 31, 2015 at 12:14:44AM +0100, Hendrik Leppkes wrote: > > On Fri, Oct 30, 2015 at 11:35 PM, Michael Niedermayer > > wrote: > > > From: Michael Niedermayer > > > > > > This

Re: [FFmpeg-devel] [PATCH] swresample/resample: improve bessel function accuracy

2015-11-03 Thread Clément Bœsch
On Tue, Nov 03, 2015 at 08:41:33AM -0500, Ganesh Ajjanagadde wrote: > On Tue, Nov 3, 2015 at 8:36 AM, Clément Bœsch <u...@pkh.me> wrote: > > On Tue, Nov 03, 2015 at 07:20:39AM -0500, Ganesh Ajjanagadde wrote: > > [...] > >> This is an opinion, so I will state mi

Re: [FFmpeg-devel] [PATCH] configure+libm.h: add fmin/fmax/fminf/fmaxf emulation

2015-11-03 Thread Clément Bœsch
On Tue, Nov 03, 2015 at 02:42:55PM +0100, Clément Bœsch wrote: [...] > > > Wasn't there a concern that these emulations don't behave identical to > > > the C library versions in regards to NaN handling? > > > I faintly remember something in the previous discussion. &

Re: [FFmpeg-devel] [PATCH] swresample/resample: improve bessel function accuracy

2015-11-02 Thread Clément Bœsch
On Mon, Nov 02, 2015 at 10:10:33PM -0500, Ganesh Ajjanagadde wrote: > On Mon, Nov 2, 2015 at 9:32 PM, Ganesh Ajjanagadde wrote: > > On Mon, Nov 2, 2015 at 6:49 PM, Ganesh Ajjanagadde wrote: > >> On Mon, Nov 2, 2015 at 6:37 PM, wm4

Re: [FFmpeg-devel] [PATCH] avfilter/vf_deshake: replace qsort with AV_QSORT

2015-10-31 Thread Clément Bœsch
On Sat, Oct 31, 2015 at 08:47:37AM -0400, Ganesh Ajjanagadde wrote: > On Sat, Oct 31, 2015 at 6:19 AM, Nicolas George wrote: > > Le nonidi 9 brumaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > >> Sample benchmark (x86-64, Haswell, GNU/Linux): > >> File: original from

Re: [FFmpeg-devel] possible farewell; dev policy clarification

2015-10-30 Thread Clément Bœsch
On Fri, Oct 30, 2015 at 07:34:29AM -0400, Ganesh Ajjanagadde wrote: [...] TL;DR Reviews are not meant to encourage, reviews are meant to find problems. A good review is a review where problems are found and pointed out. Sometimes, reviews mean suggesting way to find issues (which is then

Re: [FFmpeg-devel] [PATCH] avutil/mathematics: correct documentation for av_gcd

2015-10-30 Thread Clément Bœsch
On Thu, Oct 29, 2015 at 07:49:44PM -0400, Ganesh Ajjanagadde wrote: [...] > although you did explain to me where optimizations could be useful at > an algorithmic level privately, I take this as an opportunity to > request for general comments on places that people think could benefit > from

Re: [FFmpeg-devel] [PATCH 3/3] WIP: lavf/utils: try to avoid decoding a frame to get the codec parameters

2015-10-29 Thread Clément Bœsch
On Wed, Oct 28, 2015 at 06:34:06PM +0100, Hendrik Leppkes wrote: > On Sat, Oct 17, 2015 at 10:34 PM, Matthieu Bouron > wrote: > > From: Matthieu Bouron > > > > Avoid decoding twice images such as jpeg and png, once in the > >

Re: [FFmpeg-devel] [PATCH 3/3] WIP: lavf/utils: try to avoid decoding a frame to get the codec parameters

2015-10-29 Thread Clément Bœsch
On Thu, Oct 29, 2015 at 08:34:42AM +0100, Clément Bœsch wrote: > On Wed, Oct 28, 2015 at 06:34:06PM +0100, Hendrik Leppkes wrote: > > On Sat, Oct 17, 2015 at 10:34 PM, Matthieu Bouron > > <matthieu.bou...@gmail.com> wrote: > > > From: Matthieu Bouron

Re: [FFmpeg-devel] [PATCHv2] all: fix enum definition for large values

2015-10-28 Thread Clément Bœsch
On Wed, Oct 28, 2015 at 10:34:37AM -0400, Ronald S. Bultje wrote: [...] > You could do the same for the last 4 values of the cinething changes, they > are clearly not enums, but flags. Flags are always unsigned, so if enums > are unsigned and that causes an issue (as it does here), it makes sense

Re: [FFmpeg-devel] [PATCH 2/2] lavfi: add testsrc2 test source.

2015-10-27 Thread Clément Bœsch
On Mon, Oct 26, 2015 at 11:31:58PM +0100, Nicolas George wrote: > Le quintidi 5 brumaire, an CCXXIV, Clement Boesch a écrit : > > Same. I also think the noise could get some improvement with a wider > > color spectrum. > > I agree, but unfortunately, there is no drawutils API to blend a colored >

Re: [FFmpeg-devel] [DECISION] Include more developers in the voting committee [#2]

2015-10-26 Thread Clément Bœsch
d generated locally at around 1 o clock local time > >3691 Commit: Michael Niedermayer > 334 Commit: Paul B Mahol > 242 Commit: Clément Bœsch > 205 Commit: Carl Eugen Hoyos > 203 Commit: James Almer > 154 Commit: Ronald S. Bultje &

Re: [FFmpeg-devel] [PATCH] lavu/opt: enhance printing durations.

2015-10-26 Thread Clément Bœsch
On Mon, Oct 26, 2015 at 09:56:37PM +0100, Nicolas George wrote: > Le quintidi 5 brumaire, an CCXXIV, Clement Boesch a écrit : > > > Note: While I am touching the options code, I have this crazy idea about > > > replacing AV_OPT_TYPE_* that would allow to get out of escaping hell > > > completely.

Re: [FFmpeg-devel] [DECISION] Include more developers in the voting committee

2015-10-26 Thread Clément Bœsch
On Mon, Oct 26, 2015 at 09:55:05PM +0100, Paul B Mahol wrote: > On 10/26/15, Clement Boesch wrote: > > On Sun, Sep 20, 2015 at 12:55:11PM +0200, Clement Boesch wrote: > >> Hi Voting Committee > >> > >> This mail is an attempt to try out the process described on > >>

Re: [FFmpeg-devel] [DECISION] Include more developers in the voting committee [#2]

2015-10-26 Thread Clément Bœsch
On Sun, Oct 25, 2015 at 02:23:21PM +0100, Nicolas George wrote: > Le quartidi 4 brumaire, an CCXXIV, Michael Niedermayer a écrit : > > Heres another suggested addition to the voting committee > > also off topic but i wonder if "[DECISSION]" in the subject is the best > > choice for this > >

Re: [FFmpeg-devel] [PATCH] lavu/opt: enhance printing durations.

2015-10-26 Thread Clément Bœsch
On Mon, Oct 26, 2015 at 09:19:58PM +0100, Nicolas George wrote: > Trim unneeded leading components and trailing zeros. > Move the formating code in a separate function. > Use the function also to format the default value, it was currently > printed as plain integer, inconsistent to the way it is

Re: [FFmpeg-devel] [DECISION] Include more developers in the voting committee

2015-10-26 Thread Clément Bœsch
On Sun, Sep 20, 2015 at 12:55:11PM +0200, Clément Bœsch wrote: > Hi Voting Committee > > This mail is an attempt to try out the process described on > http://lists.ffmpeg.org/pipermail/ffmpeg-devel/2015-September/178858.html > > This thread is a call for the following peo

Re: [FFmpeg-devel] [PATCH 2/2] lavfi: add testsrc2 test source.

2015-10-26 Thread Clément Bœsch
On Sun, Oct 25, 2015 at 04:06:15PM +, Paul B Mahol wrote: > On 10/25/15, Nicolas George wrote: > > Le quartidi 4 brumaire, an CCXXIV, Nicolas George a ecrit : > >> Similar to testsrc, but using drawutils and therefore > >> supporting a lot of pixel formats instead of just

Re: [FFmpeg-devel] [PATCH] avutil/opt: print more meaningful default flags values

2015-10-25 Thread Clément Bœsch
On Sat, Oct 24, 2015 at 07:02:50PM +0200, Michael Niedermayer wrote: > On Sat, Oct 24, 2015 at 03:14:33PM +0200, Clément Bœsch wrote: > > Example: > > % ./ffmpeg -h encoder=gif > > [...] > > GIF encoder AVOptions: > > -gifflagsE..V set G

[FFmpeg-devel] [PATCH] avutil/opt: print more meaningful default flags values

2015-10-24 Thread Clément Bœsch
Example: % ./ffmpeg -h encoder=gif [...] GIF encoder AVOptions: -gifflagsE..V set GIF flags (default offsetting+transdiff) offsetting E..V enable picture offsetting transdiffE..V enable transparency detection between

Re: [FFmpeg-devel] [PATCH] swresample: Add support for clipping float/double to -1.0..1.0 range

2015-10-24 Thread Clément Bœsch
On Fri, Oct 23, 2015 at 12:41:18PM -0400, Ronald S. Bultje wrote: [...] > ffmpeg -h full, searching to -sws_flags, gives this: > -sws_flags E..V scaler flags (default 4) > fast_bilinearE..V fast bilinear > bilinear E..V

Re: [FFmpeg-devel] [PATCH] swresample: Add support for clipping float/double to -1.0..1.0 range

2015-10-24 Thread Clément Bœsch
On Sat, Oct 24, 2015 at 03:16:15PM +0200, Clément Bœsch wrote: > On Fri, Oct 23, 2015 at 12:41:18PM -0400, Ronald S. Bultje wrote: > [...] > > ffmpeg -h full, searching to -sws_flags, gives this: > > -sws_flags E..V scaler flags (default 4) >

Re: [FFmpeg-devel] [PATCH] avutil/qsort: use do while form for AV_QSORT

2015-10-22 Thread Clément Bœsch
On Thu, Oct 22, 2015 at 10:53:10PM -0400, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/qsort.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavutil/qsort.h b/libavutil/qsort.h > index 30edcc8..f079661

Re: [FFmpeg-devel] forcing ints to be 64 bits, possible new FATE client idea

2015-10-21 Thread Clément Bœsch
On Wed, Oct 21, 2015 at 06:00:21AM -0400, Ganesh Ajjanagadde wrote: [...] > why don't you spend 5 minutes trying to outline to beginners like me > what is "actually important" in your view? > According to the first 100 answers of the survey, the majority of the users want... speed optimisation

Re: [FFmpeg-devel] [PATCH] avfilter: add shuffleframes filter

2015-10-20 Thread Clément Bœsch
On Tue, Oct 20, 2015 at 08:10:55PM +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > doc/filters.texi | 20 ++ > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/vf_shuffleframes.c | 159 >

Re: [FFmpeg-devel] [PATCH] avfilter: add shuffleframes filter

2015-10-20 Thread Clément Bœsch
On Tue, Oct 20, 2015 at 12:28:32PM +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/vf_shuffleframes.c | 157 > + doc please

Re: [FFmpeg-devel] [PATCH] avutil/opt: display a better default value for int/int64 options

2015-10-18 Thread Clément Bœsch
On Sat, Oct 17, 2015 at 04:32:17PM +0200, Michael Niedermayer wrote: > On Sat, Oct 17, 2015 at 02:54:31PM +0200, Clément Bœsch wrote: > > Example: > > > > % ./ffmpeg -h encoder=aac > > -aac_coder E...A... Coding algorithm (from -1 to 3) > >

Re: [FFmpeg-devel] [PATCH] avutil/opt: display a better default value for int/int64 options

2015-10-18 Thread Clément Bœsch
On Sat, Oct 17, 2015 at 05:24:03PM +0200, Moritz Barsnick wrote: > On Sat, Oct 17, 2015 at 14:54:31 +0200, Clément Bœsch wrote: > > % ./ffmpeg -h encoder=aac > > -aac_coder E...A... Coding algorithm (from -1 to 3) > > (default twoloop) > > faa

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Clément Bœsch
On Sun, Oct 18, 2015 at 10:47:52AM -0400, Ganesh Ajjanagadde wrote: [...] > diff --git a/libavformat/subtitles.c b/libavformat/subtitles.c > index bb89766..16ab245 100644 > --- a/libavformat/subtitles.c > +++ b/libavformat/subtitles.c > @@ -23,6 +23,7 @@ > #include "avio_internal.h" > #include

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Clément Bœsch
On Sun, Oct 18, 2015 at 11:12:03AM -0400, Ganesh Ajjanagadde wrote: > On Sun, Oct 18, 2015 at 11:03 AM, Clément Bœsch <u...@pkh.me> wrote: > > On Sun, Oct 18, 2015 at 10:47:52AM -0400, Ganesh Ajjanagadde wrote: > > [...] > >> diff --git a/libavformat/subtitl

[FFmpeg-devel] [PATCH] avutil/opt: display a better default value for int/int64 options

2015-10-17 Thread Clément Bœsch
Example: % ./ffmpeg -h encoder=aac -aac_coder E...A... Coding algorithm (from -1 to 3) (default twoloop) faac E...A... FAAC-inspired method anmr E...A... ANMR method twoloop E...A... Two loop

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Clément Bœsch
On Thu, Oct 15, 2015 at 08:18:02AM -0400, Ganesh Ajjanagadde wrote: > It has been demonstrated that using libc provided floating point > functions is beneficial, in the context of fabs() vs FFABS. > > Unfortunately, MSVC 2012 (and earlier) lack the ISO C99 fmax, fmaxf, > fmin, fminf functions.

Re: [FFmpeg-devel] [PATCH] configure, avutil/libm: add fmax, fmaxf, fmin, fminf support

2015-10-15 Thread Clément Bœsch
On Thu, Oct 15, 2015 at 09:10:00AM -0400, Ganesh Ajjanagadde wrote: > On Thu, Oct 15, 2015 at 8:31 AM, Clément Bœsch <u...@pkh.me> wrote: > > On Thu, Oct 15, 2015 at 08:18:02AM -0400, Ganesh Ajjanagadde wrote: > >> It has been demonstrated that using libc provided float

Re: [FFmpeg-devel] [PATCH] avfilter/avf_showcqt: fix frame management

2015-10-13 Thread Clément Bœsch
On Tue, Oct 13, 2015 at 12:18:58PM +0700, Muhammad Faiz wrote: > > From 893d4068adb9d3d2c118186bdc5645056f0ef172 Mon Sep 17 00:00:00 2001 > From: Muhammad Faiz > Date: Tue, 13 Oct 2015 12:06:37 +0700 > Subject: [PATCH] avfilter/avf_showcqt: fix frame management > > follow

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

2015-10-13 Thread Clément Bœsch
On Tue, Oct 13, 2015 at 12:31:10AM -0400, Ganesh Ajjanagadde wrote: > On Tue, Oct 13, 2015 at 12:26 AM, Ganesh Ajjanagadde wrote: > > On Tue, Oct 13, 2015 at 12:16 AM, Carl Eugen Hoyos wrote: > >> Ganesh Ajjanagadde mit.edu> writes: > >> > >>> Bench from

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

2015-10-13 Thread Clément Bœsch
On Tue, Oct 13, 2015 at 09:25:03AM +0200, Paul B Mahol wrote: [...] > What about fmax/FFMAX? Feel free to try that out (it looks OT regarding the patch), but fmax() looks glibc specific -- Clément B. signature.asc Description: PGP signature ___

Re: [FFmpeg-devel] Survey for the community

2015-10-12 Thread Clément Bœsch
On Fri, Oct 09, 2015 at 06:13:05PM +0200, Clément Bœsch wrote: [...] > > > You have 24 hours to make comments. > > > > 24 hours is IMHO too little. What is the hurry? Maybe at least 96 hours? > > > > People had time to comment on the thread already, but sure OK

Re: [FFmpeg-devel] Survey for the community

2015-10-12 Thread Clément Bœsch
On Mon, Oct 12, 2015 at 03:19:33PM -0400, Ganesh Ajjanagadde wrote: > On Mon, Oct 12, 2015 at 3:10 PM, Clément Bœsch <u...@pkh.me> wrote: > > On Fri, Oct 09, 2015 at 06:13:05PM +0200, Clément Bœsch wrote: > > [...] > >> > > You have 24 hours to make comments

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-12 Thread Clément Bœsch
On Sun, Oct 11, 2015 at 05:11:01PM +0100, Ricardo Constantino wrote: > Bare ampersand characters are still accepted, even though out-of-spec. > Also fixes adjacent tags not being parsed. > > Fixes trac #4915 > > Signed-off-by: Ricardo Constantino > --- >

Re: [FFmpeg-devel] [PATCH 2/3] avformat/webvttdec: Don't stop parsing on comments

2015-10-12 Thread Clément Bœsch
On Sun, Oct 11, 2015 at 05:11:02PM +0100, Ricardo Constantino wrote: > Signed-off-by: Ricardo Constantino > --- > libavformat/webvttdec.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavformat/webvttdec.c b/libavformat/webvttdec.c > index

Re: [FFmpeg-devel] [PATCH 3/3] fate/subtitles: Add a new test for WebVTT

2015-10-12 Thread Clément Bœsch
On Sun, Oct 11, 2015 at 05:11:03PM +0100, Ricardo Constantino wrote: > Includes escapes that should now be supported and a few features not yet > fully supported, like comments, regions, classes, ruby, and lang. > > All were tested with https://quuz.org/webvtt/ for validation, except > regions

Re: [FFmpeg-devel] [PATCH] avfilter/WIP: add selectivecolor filter

2015-10-11 Thread Clément Bœsch
On Sat, Aug 15, 2015 at 07:02:11PM +0200, Clément Bœsch wrote: > --- > Only K adjustment remains to fix before it's ready for a first upstream > version. Help figuring out is still welcome. Otherwise, since this is very > close to the final version, review is welcome as well.

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-11 Thread Clément Bœsch
On Fri, Oct 09, 2015 at 04:29:03PM +0100, Ricardo Constantino wrote: > Bare ampersand characters are still accepted, even though out-of-spec. > Also fixes adjacent tags not being parsed. > > Fixes trac #4915 > > Signed-off-by: Ricardo Constantino > --- >

Re: [FFmpeg-devel] setting up a proper Github mirror

2015-10-11 Thread Clément Bœsch
On Sun, Oct 11, 2015 at 02:43:53PM -0400, Ganesh Ajjanagadde wrote: > Hi all, > > I noticed that the Github mirror: https://github.com/FFmpeg/FFmpeg is > over 3 hours out of sync with the main repos, making it unusable as a > fetch url for development. Anyone knows why this is the case? > Isn't

Re: [FFmpeg-devel] [PATCH 1/5] lavu: add JNI support

2015-10-10 Thread Clément Bœsch
On Fri, Oct 09, 2015 at 07:00:44PM +0200, Nicolas George wrote: > L'octidi 18 vendémiaire, an CCXXIV, Jean-Baptiste Kempf a écrit : > > > +HEADERS-$(CONFIG_JNI) += jni.h > > You are going to install a jni.h header? > > I am not very fond of having java-related stuff in FFmpeg,

Re: [FFmpeg-devel] [PATCHv2 1/2] avfilter/all: propagate errors of functions from avfilter/formats

2015-10-10 Thread Clément Bœsch
On Sat, Oct 10, 2015 at 09:34:21AM -0400, Ganesh Ajjanagadde wrote: [...] > diff --git a/libavfilter/vsrc_life.c b/libavfilter/vsrc_life.c > index 314746d..0e31ef7 100644 > --- a/libavfilter/vsrc_life.c > +++ b/libavfilter/vsrc_life.c > @@ -417,6 +417,7 @@ static int query_formats(AVFilterContext

Re: [FFmpeg-devel] [PATCH] build: restore videotoolbox compilation on iOS

2015-10-09 Thread Clément Bœsch
On Fri, Oct 02, 2015 at 12:36:54PM +0200, Clément Bœsch wrote: > On Fri, Oct 02, 2015 at 12:20:10PM +0200, wm4 wrote: > > On Fri, 2 Oct 2015 11:23:59 +0200 > > Clément Bœsch <u...@pkh.me> wrote: > > > > > On Fri, Oct 02,

Re: [FFmpeg-devel] [RFC] Poll for the community

2015-10-09 Thread Clément Bœsch
On Fri, Oct 09, 2015 at 10:45:17AM -0400, Ganesh Ajjanagadde wrote: > On Fri, Oct 9, 2015 at 9:51 AM, Clément Bœsch <u...@pkh.me> wrote: > > On Fri, Sep 25, 2015 at 10:41:44AM +0200, Clément Bœsch wrote: > >> Hi, > >> > >> I'd like to suggest a poll for

<    4   5   6   7   8   9   10   11   12   13   >