Re: [FFmpeg-devel] [PATCH 2/2] lavf/http: Add error codes 301 and 503 and make replies more customizable

2015-08-20 Thread Nicolas George
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : Signed-off-by: Stephan Holljes klaxa1...@googlemail.com --- libavformat/http.c | 44 +++- 1 file changed, 39 insertions(+), 5 deletions(-) diff --git a/libavformat/http.c

Re: [FFmpeg-devel] [PATCH] avfilter/vf_dejudder: use the name 's' for the pointer to the private context

2015-08-20 Thread Paul B Mahol
On 8/20/15, Ganesh Ajjanagadde gajja...@mit.edu wrote: On Thu, Aug 20, 2015 at 11:24 AM, Paul B Mahol one...@gmail.com wrote: This is shorter and consistent across filters. Signed-off-by: Paul B Mahol one...@gmail.com --- libavfilter/vf_dejudder.c | 66

[FFmpeg-devel] [PATCH v9] Add support for Audible AA files

2015-08-20 Thread Vesselin Bontchev
This patch revision (v9) adds me to the MAINTAINERS file. VesselinFrom a9457dead0dac503c9aeca8d49bc89d5c9b607d9 Mon Sep 17 00:00:00 2001 From: Vesselin Bontchev vesselin.bontc...@yandex.com Date: Sun, 19 Jul 2015 23:16:36 +0200 Subject: [PATCH] Add support for Audible AA files

Re: [FFmpeg-devel] [PATCH] avfilter/vf_dejudder: use the name 's' for the pointer to the private context

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 12:59 PM, Paul B Mahol one...@gmail.com wrote: On 8/20/15, Ganesh Ajjanagadde gajja...@mit.edu wrote: On Thu, Aug 20, 2015 at 11:24 AM, Paul B Mahol one...@gmail.com wrote: This is shorter and consistent across filters. Signed-off-by: Paul B Mahol one...@gmail.com ---

Re: [FFmpeg-devel] [PATCHv2] avformat/webmdashenc: fix uninitialized variable

2015-08-20 Thread Ronald S. Bultje
Hi, On Thu, Aug 20, 2015 at 11:18 AM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: Fixes -Wsometimes-uninitialized from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com ---

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/metasound_data: remove unused variable

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: Fixes -Wunused-const-variable from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com ---

Re: [FFmpeg-devel] [PATCH v8] Add support for Audible AA files

2015-08-20 Thread Michael Niedermayer
On Thu, Aug 20, 2015 at 05:34:22AM +0300, Vesselin Bontchev wrote: Hi! It would be awesome to have this patch merged in FFmpeg. Vesselin doc/demuxers.texi|6 doc/general.texi |2 libavformat/Makefile |1 libavformat/aadec.c | 303

[FFmpeg-devel] [PATCH 3/4] avformat/ftp: use correct enum type

2015-08-20 Thread Ganesh Ajjanagadde
Fixes -Wenum-conversion from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavformat/ftp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/ftp.c

[FFmpeg-devel] [PATCH 4/4] avformat/webmdashenc: fix uninitialized variable

2015-08-20 Thread Ganesh Ajjanagadde
Fixes -Wsometimes-uninitialized from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavformat/webmdashenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [FFmpeg-devel] [PATCH 1/2] lavf/http: Parse and/or expose various client data.

2015-08-20 Thread Nicolas George
Thanks for the patch. A few remarks, specific to the stand-alone-commit version or that I had not yet spotted. Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : Signed-off-by: Stephan Holljes klaxa1...@googlemail.com --- libavformat/http.c | 40

Re: [FFmpeg-devel] [PATCH 2/2] lavf/file: check for dirent.h support

2015-08-20 Thread Mariusz Szczepańczyk
On Thu, Jun 25, 2015 at 12:09 PM, Michael Niedermayer michae...@gmx.at wrote: On Wed, Jun 24, 2015 at 03:25:18AM +0200, Mariusz Szczepańczyk wrote: On Tue, Jun 23, 2015 at 8:34 PM, Michael Niedermayer michae...@gmx.at wrote: On Mon, Jun 22, 2015 at 12:01:33AM +0200, Mariusz Szczepańczyk

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/metasound_data: remove unused variable

2015-08-20 Thread Michael Niedermayer
On Thu, Aug 20, 2015 at 04:34:19AM -0400, Ganesh Ajjanagadde wrote: On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: Fixes -Wunused-const-variable from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by:

[FFmpeg-devel] [RFC]fps filter

2015-08-20 Thread Carl Eugen Hoyos
Hi! I finally realized why -vf fps=3/1001,fieldmatch,decimate does not always work / produces ugly output. The issue is not related to fieldmatch and/or decimate but to the fps filter: It (sometimes) drops frames even if the framerate is increased. So the question is: Is it a good idea

Re: [FFmpeg-devel] [PATCH 3/4] avformat/ftp: use correct enum type

2015-08-20 Thread Michael Niedermayer
On Thu, Aug 20, 2015 at 04:29:12AM -0400, Ganesh Ajjanagadde wrote: Fixes -Wenum-conversion from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavformat/ftp.c | 2 +- 1 file changed, 1

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/jpeg2000: comment out unused variable

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 7:28 PM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: Fixes -Wunused-const-variable from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavcodec/jpeg2000.c | 2

Re: [FFmpeg-devel] [PATCH] lavf/http: implement directory listing callbacks for Apache

2015-08-20 Thread Mariusz Szczepańczyk
On Thu, Aug 20, 2015 at 11:59 PM, Ganesh Ajjanagadde gajja...@mit.edu wrote: On Thu, Aug 20, 2015 at 5:48 PM, Mariusz Szczepańczyk mszczepanc...@gmail.com wrote: On Thu, Aug 20, 2015 at 11:38 PM, Ganesh Ajjanagadde gajja...@mit.edu wrote: On Thu, Aug 20, 2015 at 5:32 PM, Mariusz

Re: [FFmpeg-devel] [PATCH] ffserver: cast PID to int64_t before printing

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 10:03 PM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: On Thu, Aug 20, 2015 at 9:29 PM, Timothy Gu timothyg...@gmail.com wrote: On Thu, Aug 20, 2015 at 6:03 PM Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: -avio_printf(pb, Running as pid %d.\n,

[FFmpeg-devel] [PATCH] all: use { 0 } for zero initializing compound data structures

2015-08-20 Thread Ganesh Ajjanagadde
C Standard guarantees that { 0 } will do a correct zero initialization. We use this for consistency across the project. Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- ffplay.c | 2 +- ffserver.c| 2 +- libavcodec/aacdec_template.c | 2 +-

[FFmpeg-devel] [PATCH 3/3] avcodec/jpeg2000: comment out unused variable

2015-08-20 Thread Ganesh Ajjanagadde
Fixes -Wunused-const-variable from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavcodec/jpeg2000.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/jpeg2000.c

[FFmpeg-devel] [PATCH 1/3] avformat/hls: correct comment for ensure_playlist()

2015-08-20 Thread Ganesh Ajjanagadde
Comment was previously slightly incorrect. Also, it was placed in the wrong location. Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavformat/hls.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index

[FFmpeg-devel] [PATCH 2/3] avformat/hls: comment out unused function

2015-08-20 Thread Ganesh Ajjanagadde
Fixes -Wunused-function from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavformat/hls.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/hls.c

[FFmpeg-devel] [PATCH] ffserver: cast PID to int64_t before printing

2015-08-20 Thread Ganesh Ajjanagadde
Unfortunately, there is no portable format specifier for PID's. Furthermore, it is not safe to assume pid_t = 32 bit in size, see e.g http://unix.derkeiler.com/Mailing-Lists/AIX-L/2010-08/msg8.html. Right now, it is ok to assume pid_t = 32 bit in size, but this may change in the future. Also,

Re: [FFmpeg-devel] [PATCH] ffserver: cast PID to int64_t before printing

2015-08-20 Thread Timothy Gu
On Thu, Aug 20, 2015 at 6:03 PM Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: -avio_printf(pb, Running as pid %d.\n, stream-pid); +avio_printf(pb, Running as pid %ld.\n, (int64_t) stream-pid); You need `% PRIi64` as the format specifier. Timothy

Re: [FFmpeg-devel] [PATCH] ffserver: cast PID to int64_t before printing

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 9:29 PM, Timothy Gu timothyg...@gmail.com wrote: On Thu, Aug 20, 2015 at 6:03 PM Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: -avio_printf(pb, Running as pid %d.\n, stream-pid); +avio_printf(pb, Running as pid %ld.\n, (int64_t) stream-pid);

[FFmpeg-devel] [PATCHv2] ffserver: cast PID to int64_t before printing

2015-08-20 Thread Ganesh Ajjanagadde
Unfortunately, there is no portable format specifier for PID's. Furthermore, it is not safe to assume pid_t = 32 bit in size, see e.g http://unix.derkeiler.com/Mailing-Lists/AIX-L/2010-08/msg8.html. Right now, it is ok to assume pid_t = 32 bit in size, but this may change in the future. Also,

Re: [FFmpeg-devel] [PATCH] avfilter/vf_dejudder: use the name 's' for the pointer to the private context

2015-08-20 Thread Michael Niedermayer
On Thu, Aug 20, 2015 at 02:06:18PM -0400, Ganesh Ajjanagadde wrote: On Thu, Aug 20, 2015 at 1:53 PM, Clément Bœsch u...@pkh.me wrote: On Thu, Aug 20, 2015 at 01:10:43PM -0400, Ganesh Ajjanagadde wrote: [...] I know what s is doing, I just fail to see the connection between the name s and

[FFmpeg-devel] [PATCH] lavf/http: Export headers as AVDictionary

2015-08-20 Thread Stephan Holljes
Signed-off-by: Stephan Holljes klaxa1...@googlemail.com --- libavformat/http.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavformat/http.c b/libavformat/http.c index 3fd0496..2c51491 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -48,6 +48,7 @@ #define

Re: [FFmpeg-devel] [PATCH 1/2] lavf/http: Parse and/or expose various client data.

2015-08-20 Thread Stephan Holljes
On Thu, Aug 20, 2015 at 6:51 PM, Nicolas George geo...@nsup.org wrote: Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : In ffserver the User-Agent is used to decide how to deal with a client. (WMP being a special case.) Ok, but that is special case for a specific application.

Re: [FFmpeg-devel] [PATCH] avfilter/vf_dejudder: use the name 's' for the pointer to the private context

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 1:53 PM, Clément Bœsch u...@pkh.me wrote: On Thu, Aug 20, 2015 at 01:10:43PM -0400, Ganesh Ajjanagadde wrote: [...] I know what s is doing, I just fail to see the connection between the name s and filter private context. s originally stands for struct iirc If that is

Re: [FFmpeg-devel] [PATCH] avfilter/vf_dejudder: use the name 's' for the pointer to the private context

2015-08-20 Thread Clément Bœsch
On Thu, Aug 20, 2015 at 01:10:43PM -0400, Ganesh Ajjanagadde wrote: [...] I know what s is doing, I just fail to see the connection between the name s and filter private context. s originally stands for struct iirc -- Clément B. pgperHJDLxk01.pgp Description: PGP signature

Re: [FFmpeg-devel] [PATCH] lavf/http: Export headers as AVDictionary

2015-08-20 Thread Stephan Holljes
Actually I just noticed that Pragma-headers may appear multiple times. What would be a good way to handle that? I thought about expanding the header tag from Pragma to Pragma-%02d with either an incrementing number or just using s-nb_headers. Is there a better solution? Regards, Stephan

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/aacsbr_fixed: remove unused variable

2015-08-20 Thread Ronald S. Bultje
Hi, On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: Fixes -Wunused-const-variable from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com ---

Re: [FFmpeg-devel] [PATCH 4/4] avformat/webmdashenc: fix uninitialized variable

2015-08-20 Thread Ronald S. Bultje
Hi, On Thu, Aug 20, 2015 at 4:29 AM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: Fixes -Wsometimes-uninitialized from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com ---

Re: [FFmpeg-devel] [PATCH 02/13] avfilter: add missing FF_API_AVFILTERPAD_PUBLIC guard

2015-08-20 Thread Andreas Cadhalpun
On 20.08.2015 01:33, Michael Niedermayer wrote: On Tue, Aug 18, 2015 at 11:47:45PM +0200, Andreas Cadhalpun wrote: graph2dot.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 560d5f42580115baf305ba8ff985202b248b4864 0001-graph2dot-use-avfilter_pad_get_name-accessor-functio.patch

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/aacsbr_fixed: remove unused variable

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 3:31 PM, Michael Niedermayer mich...@niedermayer.cc wrote: On Thu, Aug 20, 2015 at 09:33:31AM -0400, Ronald S. Bultje wrote: Hi, On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: Fixes -Wunused-const-variable from

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/aacsbr_fixed: remove unused variable

2015-08-20 Thread Michael Niedermayer
On Thu, Aug 20, 2015 at 04:44:17PM -0400, Ganesh Ajjanagadde wrote: On Thu, Aug 20, 2015 at 3:31 PM, Michael Niedermayer mich...@niedermayer.cc wrote: On Thu, Aug 20, 2015 at 09:33:31AM -0400, Ronald S. Bultje wrote: Hi, On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/metasound_data: remove unused variable

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 6:37 AM, Michael Niedermayer mich...@niedermayer.cc wrote: On Thu, Aug 20, 2015 at 04:34:19AM -0400, Ganesh Ajjanagadde wrote: On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: Fixes -Wunused-const-variable from

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/metasound_data: remove unused variable

2015-08-20 Thread Michael Niedermayer
On Thu, Aug 20, 2015 at 05:12:40PM -0400, Ganesh Ajjanagadde wrote: On Thu, Aug 20, 2015 at 6:37 AM, Michael Niedermayer mich...@niedermayer.cc wrote: On Thu, Aug 20, 2015 at 04:34:19AM -0400, Ganesh Ajjanagadde wrote: On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH] ffserver: use -b instead of -ab for setting audio bitrate.

2015-08-20 Thread Paul B Mahol
On 8/18/15, Ronald S. Bultje rsbul...@gmail.com wrote: --- ffserver_config.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ffserver_config.c b/ffserver_config.c index 06bd8ac..de8a454 100644 --- a/ffserver_config.c +++ b/ffserver_config.c @@ -230,9 +230,9 @@

Re: [FFmpeg-devel] [PATCH] avfilter/vf_dejudder: use the name 's' for the pointer to the private context

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 2:59 PM, Michael Niedermayer mich...@niedermayer.cc wrote: On Thu, Aug 20, 2015 at 02:06:18PM -0400, Ganesh Ajjanagadde wrote: On Thu, Aug 20, 2015 at 1:53 PM, Clément Bœsch u...@pkh.me wrote: On Thu, Aug 20, 2015 at 01:10:43PM -0400, Ganesh Ajjanagadde wrote: [...]

[FFmpeg-devel] [PATCH] avfilter/vf_dejudder: use the name 's' for the pointer to the private context

2015-08-20 Thread Paul B Mahol
This is shorter and consistent across filters. Signed-off-by: Paul B Mahol one...@gmail.com --- libavfilter/vf_dejudder.c | 66 +++ 1 file changed, 33 insertions(+), 33 deletions(-) diff --git a/libavfilter/vf_dejudder.c b/libavfilter/vf_dejudder.c

Re: [FFmpeg-devel] [PATCH 1/2] lavf/http: Parse and/or expose various client data.

2015-08-20 Thread Stephan Holljes
On Thu, Aug 20, 2015 at 5:15 PM, Nicolas George geo...@nsup.org wrote: Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : Added to all while(!av_isspace(p)) loops in the server code. You may want to send this part in a stand-alone patch, since it is actually a bug in the current

Re: [FFmpeg-devel] [PATCH 2/2] lavf/http: Add error codes 301 and 503 and make replies more customizable

2015-08-20 Thread Stephan Holljes
On Thu, Aug 20, 2015 at 11:08 AM, Nicolas George geo...@nsup.org wrote: Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : Signed-off-by: Stephan Holljes klaxa1...@googlemail.com --- libavformat/http.c | 44 +++- 1 file changed, 39

Re: [FFmpeg-devel] [PATCH 2/2] lavf/http: Add error codes 301 and 503 and make replies more customizable

2015-08-20 Thread Nicolas George
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : It's the opposite. When the request was HEAD, the location header is not set, otherwise it is. Indeed, but still, why? I just checked with a random Apache server, and HEAD also returns the Location header. Regards, -- Nicolas

Re: [FFmpeg-devel] [PATCH 2/2] lavf/http: Add error codes 301 and 503 and make replies more customizable

2015-08-20 Thread Stephan Holljes
On Thu, Aug 20, 2015 at 5:17 PM, Nicolas George geo...@nsup.org wrote: Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : It's the opposite. When the request was HEAD, the location header is not set, otherwise it is. Indeed, but still, why? I just checked with a random Apache server,

Re: [FFmpeg-devel] [PATCH 1/2] lavf/http: Parse and/or expose various client data.

2015-08-20 Thread Stephan Holljes
On Thu, Aug 20, 2015 at 11:02 AM, Nicolas George geo...@nsup.org wrote: Thanks for the patch. A few remarks, specific to the stand-alone-commit version or that I had not yet spotted. Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : Signed-off-by: Stephan Holljes

Re: [FFmpeg-devel] [PATCH 1/2] lavf/http: Parse and/or expose various client data.

2015-08-20 Thread Nicolas George
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : Added to all while(!av_isspace(p)) loops in the server code. You may want to send this part in a stand-alone patch, since it is actually a bug in the current code. This is more or less temporal, since the user-agent field is/will be

[FFmpeg-devel] [PATCHv2] avformat/webmdashenc: fix uninitialized variable

2015-08-20 Thread Ganesh Ajjanagadde
Fixes -Wsometimes-uninitialized from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavformat/webmdashenc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH 1/2] lavf/http: Parse and/or expose various client data.

2015-08-20 Thread Stephan Holljes
On Thu, Aug 20, 2015 at 5:32 PM, Nicolas George geo...@nsup.org wrote: Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : Should the same be done for the Host-parameter sent by the client too? It can be discussed. On one hand, the Host field has a special meaning for the protocol,

Re: [FFmpeg-devel] [PATCHv2] avformat/webmdashenc: fix uninitialized variable

2015-08-20 Thread Michael Niedermayer
On Thu, Aug 20, 2015 at 01:14:10PM -0400, Ronald S. Bultje wrote: Hi, On Thu, Aug 20, 2015 at 11:18 AM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: Fixes -Wsometimes-uninitialized from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1

Re: [FFmpeg-devel] [PATCH] api-band-test: first version

2015-08-20 Thread Michael Niedermayer
Hi ludmila did you see my reply on the ML (its also below)? as you didnt reply ... On Tue, Aug 11, 2015 at 03:31:05AM +0200, Michael Niedermayer wrote: On Tue, Jul 28, 2015 at 09:54:06PM +0300, Ludmila Glinskih wrote: Works only for flv, h263 and huffyuv decoders, for video with yuv420p pixel

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/aacsbr_fixed: remove unused variable

2015-08-20 Thread Michael Niedermayer
On Thu, Aug 20, 2015 at 09:33:31AM -0400, Ronald S. Bultje wrote: Hi, On Thu, Aug 20, 2015 at 4:28 AM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: Fixes -Wunused-const-variable from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1

Re: [FFmpeg-devel] [PATCH] api-seek-test: first version

2015-08-20 Thread Michael Niedermayer
On Fri, Jul 31, 2015 at 04:01:43PM +0300, Ludmila Glinskih wrote: Works only with video stream. First pass without seeking -- counts crcs of a frames and store it in an array. After that it seeks a lot in different places and checks if crcs of these frames and crcs of frames in array are

[FFmpeg-devel] [PATCH] avcodec/metasound_data: comment out unused variable

2015-08-20 Thread Ganesh Ajjanagadde
Fixes -Wunused-const-variable from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavcodec/metasound_data.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [FFmpeg-devel] [PATCH] lavf/http: implement directory listing callbacks for Apache

2015-08-20 Thread Mariusz Szczepańczyk
On Thu, Aug 20, 2015 at 11:38 PM, Ganesh Ajjanagadde gajja...@mit.edu wrote: On Thu, Aug 20, 2015 at 5:32 PM, Mariusz Szczepańczyk mszczepanc...@gmail.com wrote: --- configure | 3 + libavformat/http.c | 194 + 2 files

Re: [FFmpeg-devel] [PATCH] lavf/http: implement directory listing callbacks for Apache

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 5:48 PM, Mariusz Szczepańczyk mszczepanc...@gmail.com wrote: On Thu, Aug 20, 2015 at 11:38 PM, Ganesh Ajjanagadde gajja...@mit.edu wrote: On Thu, Aug 20, 2015 at 5:32 PM, Mariusz Szczepańczyk mszczepanc...@gmail.com wrote: --- configure | 3 +

[FFmpeg-devel] [PATCH] lavf/http: implement directory listing callbacks for Apache

2015-08-20 Thread Mariusz Szczepańczyk
--- configure | 3 + libavformat/http.c | 194 + 2 files changed, 197 insertions(+) diff --git a/configure b/configure index e67ddf6..401e041 100755 --- a/configure +++ b/configure @@ -265,6 +265,7 @@ External library support:

Re: [FFmpeg-devel] [PATCH] lavf/http: implement directory listing callbacks for Apache

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 5:32 PM, Mariusz Szczepańczyk mszczepanc...@gmail.com wrote: --- configure | 3 + libavformat/http.c | 194 + 2 files changed, 197 insertions(+) diff --git a/configure b/configure index

Re: [FFmpeg-devel] [PATCH] avcodec/metasound_data: comment out unused variable

2015-08-20 Thread Michael Niedermayer
On Thu, Aug 20, 2015 at 05:34:52PM -0400, Ganesh Ajjanagadde wrote: Fixes -Wunused-const-variable from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavcodec/metasound_data.c | 3 ++- 1

[FFmpeg-devel] [PATCH 2/4] avcodec/aacsbr_fixed: remove unused variable

2015-08-20 Thread Ganesh Ajjanagadde
Fixes -Wunused-const-variable from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavcodec/aacsbr_fixed.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/aacsbr_fixed.c

[FFmpeg-devel] [PATCH 1/4] avcodec/metasound_data: remove unused variable

2015-08-20 Thread Ganesh Ajjanagadde
Fixes -Wunused-const-variable from http://fate.ffmpeg.org/report.cgi?time=20150820031140slot=arm64-darwin-clang-apple-5.1 Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavcodec/metasound_data.c | 4 1 file changed, 4 deletions(-) diff --git a/libavcodec/metasound_data.c

Re: [FFmpeg-devel] [PATCH] avfilter/vf_dejudder: use the name 's' for the pointer to the private context

2015-08-20 Thread Ganesh Ajjanagadde
On Thu, Aug 20, 2015 at 11:24 AM, Paul B Mahol one...@gmail.com wrote: This is shorter and consistent across filters. Signed-off-by: Paul B Mahol one...@gmail.com --- libavfilter/vf_dejudder.c | 66 +++ 1 file changed, 33 insertions(+), 33

Re: [FFmpeg-devel] [PATCH] lavf/http: Fix parsing http request data to not read over '\0'.

2015-08-20 Thread Nicolas George
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : Signed-off-by: Stephan Holljes klaxa1...@googlemail.com --- libavformat/http.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) LGTM, thanks. Note to Hendrik and the others who took over the merging: Until recently,

Re: [FFmpeg-devel] [PATCH 1/2] lavf/http: Parse and/or expose various client data.

2015-08-20 Thread Nicolas George
Le tridi 3 fructidor, an CCXXIII, Stephan Holljes a écrit : In ffserver the User-Agent is used to decide how to deal with a client. (WMP being a special case.) Ok, but that is special case for a specific application. That sounds pretty optimistic. Maybe it is trivial for you, but from my

[FFmpeg-devel] [PATCH] lavf/http: Fix parsing http request data to not read over '\0'.

2015-08-20 Thread Stephan Holljes
Signed-off-by: Stephan Holljes klaxa1...@googlemail.com --- libavformat/http.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/http.c b/libavformat/http.c index a136918..18df34e 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -716,7 +716,7 @@ static

[FFmpeg-devel] [PATCH 1/2] lavf/http: Parse and/or expose various client data.

2015-08-20 Thread Stephan Holljes
Introduce the ability to set reply body and reply text and read the url-query, http-version and host parameter. Signed-off-by: Stephan Holljes klaxa1...@googlemail.com --- libavformat/http.c | 32 ++-- 1 file changed, 30 insertions(+), 2 deletions(-) Does not export

[FFmpeg-devel] [PATCH 2/2] lavf/http: Rework http_write_reply()

2015-08-20 Thread Stephan Holljes
Add error codes 301 and 503, make replies more customizable and use AVBPrint for strings. Signed-off-by: Stephan Holljes klaxa1...@googlemail.com --- libavformat/http.c | 75 +- 1 file changed, 63 insertions(+), 12 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] QSV : Added look ahead rate control mode

2015-08-20 Thread Michael Niedermayer
On Thu, Aug 20, 2015 at 02:34:22PM +0100, Sven Dueking wrote: From: Sven Dueking s...@nablet.com --- libavcodec/qsvenc.c | 36 +--- libavcodec/qsvenc.h | 12 libavcodec/qsvenc_h264.c | 12 3 files changed, 57