Re: [FFmpeg-devel] [PATCH] lavfi/pan: allow negative gain parameters also for other inputs than the first named

2016-11-23 Thread Michael Niedermayer
On Sat, Oct 29, 2016 at 03:03:58PM +0200, Nicolas George wrote: > Le septidi 7 brumaire, an CCXXV, Moritz Barsnick a écrit : > > Expands the parser to also accept the separator '-' in addition to > > '+', and take the negative sign into consideration. > > > > The optional sign for the first factor

Re: [FFmpeg-devel] [PATCH] lavfi/pan: allow negative gain parameters also for other inputs than the first named

2016-10-29 Thread Nicolas George
Le septidi 7 brumaire, an CCXXV, Moritz Barsnick a écrit : > Expands the parser to also accept the separator '-' in addition to > '+', and take the negative sign into consideration. > > The optional sign for the first factor in the expression is already > covered by parsing for an integer. > > Si

Re: [FFmpeg-devel] [PATCH] lavfi/pan: allow negative gain parameters also for other inputs than the first named

2016-10-28 Thread Moritz Barsnick
On Thu, Oct 13, 2016 at 11:31:22 +0200, Nicolas George wrote: > Nit: inconsistent placement of the else clause. Not blocking. Fixed, and ping. Thanks, Moritz ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg

[FFmpeg-devel] [PATCH] lavfi/pan: allow negative gain parameters also for other inputs than the first named

2016-10-28 Thread Moritz Barsnick
Expands the parser to also accept the separator '-' in addition to '+', and take the negative sign into consideration. The optional sign for the first factor in the expression is already covered by parsing for an integer. Signed-off-by: Moritz Barsnick --- doc/filters.texi | 2 +- libavfil

Re: [FFmpeg-devel] [PATCH] lavfi/pan: allow negative gain parameters also for other inputs than the first named

2016-10-13 Thread Nicolas George
Le decadi 20 vendémiaire, an CCXXV, Moritz Barsnick a écrit : > Expands the parser to also accept the separator '-' in addition to > '+', and take the negative sign into consideration. > > Signed-off-by: Moritz Barsnick > --- > doc/filters.texi | 2 +- > libavfilter/af_pan.c | 11 --

[FFmpeg-devel] [PATCH] lavfi/pan: allow negative gain parameters also for other inputs than the first named

2016-10-11 Thread Moritz Barsnick
Expands the parser to also accept the separator '-' in addition to '+', and take the negative sign into consideration. Signed-off-by: Moritz Barsnick --- doc/filters.texi | 2 +- libavfilter/af_pan.c | 11 --- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/doc/filter

Re: [FFmpeg-devel] [PATCH] lavfi/pan: allow negative gain parameters also for other inputs than the first named

2016-10-11 Thread Moritz Barsnick
On Mon, Oct 10, 2016 at 19:09:34 +0200, Nicolas George wrote: > Maybe I am missing something, but I do not see where sign is reset to 1 > before the next run of the loop. You are correct. Too stupid me, I must pay more attention. And improve my testing. ;-) New patch coming. Thanks for checking,

Re: [FFmpeg-devel] [PATCH] lavfi/pan: allow negative gain parameters also for other inputs than the first named

2016-10-10 Thread Nicolas George
L'octidi 18 vendémiaire, an CCXXV, Moritz Barsnick a écrit : > Expands the parser to also accept the separator '-' in addition to > '+', and take the negative sign into consideration. > > Signed-off-by: Moritz Barsnick > --- > doc/filters.texi | 2 +- > libavfilter/af_pan.c | 8 +--- > 2

[FFmpeg-devel] [PATCH] lavfi/pan: allow negative gain parameters also for other inputs than the first named

2016-10-08 Thread Moritz Barsnick
Expands the parser to also accept the separator '-' in addition to '+', and take the negative sign into consideration. Signed-off-by: Moritz Barsnick --- doc/filters.texi | 2 +- libavfilter/af_pan.c | 8 +--- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/doc/filters.tex