Re: [FFmpeg-devel] [PATCH 2/2] avfilter: add hstack vstack filter

2015-08-26 Thread Ganesh Ajjanagadde
On Wed, Aug 26, 2015 at 2:21 PM, Paul B Mahol one...@gmail.com wrote: On 8/25/15, Timothy Gu timothyg...@gmail.com wrote: On Tue, Aug 25, 2015 at 7:37 AM Paul B Mahol one...@gmail.com wrote: +@section hstack +Stack streams horizontally. + +All streams must be of same pixel format and of

Re: [FFmpeg-devel] [PATCH 3/5] lavfi/vf_idet: reindent after last commit.

2015-08-26 Thread Paul B Mahol
On 8/26/15, Nicolas George geo...@nsup.org wrote: Signed-off-by: Nicolas George geo...@nsup.org --- libavfilter/vf_idet.c | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) lgtm ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 2/2] avfilter: add hstack vstack filter

2015-08-26 Thread Paul B Mahol
On 8/25/15, Timothy Gu timothyg...@gmail.com wrote: On Tue, Aug 25, 2015 at 7:37 AM Paul B Mahol one...@gmail.com wrote: +@section hstack +Stack streams horizontally. + +All streams must be of same pixel format and of same height. Add a new line here Done. +Note: this filter is

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

2015-08-26 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavfilter/af_apad.c | 42 +- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/libavfilter/af_apad.c b/libavfilter/af_apad.c index eafc705..0a2d420 100644 ---

Re: [FFmpeg-devel] [PATCHv3] configure: warn if GCC 4.2 is being used

2015-08-26 Thread Carl Eugen Hoyos
Ganesh Ajjanagadde gajjanagadde at gmail.com writes: +case $gcc_basever in +4.2*) +warn gcc 4.2 is outdated and may miscompile FFmpeg. Please use a newer compiler. ;; I believe this is a good idea and will push if nobody objects. Carl Eugen

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

2015-08-26 Thread Nicolas George
Le nonidi 9 fructidor, an CCXXIII, Ganesh Ajjanagadde a écrit : Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavfilter/af_amerge.c | 104 1 file changed, 52 insertions(+), 52 deletions(-) Ok. Regards, -- Nicolas George

Re: [FFmpeg-devel] [PATCH 4/5] lavfi: drop the requirement that request_frame returns a frame.

2015-08-26 Thread Paul B Mahol
On 8/26/15, Nicolas George geo...@nsup.org wrote: Le nonidi 9 fructidor, an CCXXIII, Paul B Mahol a ecrit : I'm interested. Glad to read it. So here it is: Foremost, I want to remove the recursiveness. Currently, when a frame is requested on buffersink, it recursively calls request_frame()

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

2015-08-26 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavfilter/af_amerge.c | 104 1 file changed, 52 insertions(+), 52 deletions(-) diff --git a/libavfilter/af_amerge.c b/libavfilter/af_amerge.c index 62a11f7..fb46ec3 100644 ---

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

2015-08-26 Thread Paul B Mahol
On 8/26/15, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: On Wed, Aug 26, 2015 at 12:53 PM, Ganesh Ajjanagadde gajjanaga...@gmail.com wrote: Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- libavfilter/af_amerge.c | 104 1

[FFmpeg-devel] [PATCH] api-h264-test: rename and expand

2015-08-26 Thread Ludmila Glinskih
Add support of floating point decoders. Add support of audio decoders. --- tests/api/Makefile | 2 +- tests/api/api-decode-test.c| 368 + tests/api/api-h264-test.c | 166 --- tests/fate/api.mak | 12 +-

Re: [FFmpeg-devel] [libav-devel] [PATCH 20/20] Bump major versions of all libraries

2015-08-26 Thread Vittorio Giovara
On Wed, Aug 26, 2015 at 12:47 AM, Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: On 25.08.2015 11:03, Vittorio Giovara wrote: There is, consensus does not need to be unanimous, and so far only you have been expressing concerns (multiple times). That's not true. wm4, James and

Re: [FFmpeg-devel] [PATCH] configure: do not fork off grep subprocess while testing for whitespace

2015-08-26 Thread Michael Niedermayer
On Tue, Aug 25, 2015 at 11:22:49PM -0400, Ganesh Ajjanagadde wrote: grep is not necessary for the functionality. This avoids an unnecessary fork. Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- configure | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) applied

[FFmpeg-devel] Connection timeout to HTTP resource

2015-08-26 Thread Przemysław Sobala
Hi I wanted to configure tcp connection timeout while connecting to eg. http://google.com:81. ffmpeg -i http://google.com:81?timeout=100 doesn't pass timeout param to tcp proto, as tcp proto doesn't support URL query string ffmpeg -timeout 100 -i http://google.com:81 doesn't pass

[FFmpeg-devel] [PATCH 5/5] lavfi: remove FF_LINK_FLAG_REQUEST_LOOP.

2015-08-26 Thread Nicolas George
It has no longer any effect. Signed-off-by: Nicolas George geo...@nsup.org --- libavfilter/af_afade.c | 1 - libavfilter/af_asetnsamples.c | 1 - libavfilter/af_atempo.c| 2 -- libavfilter/af_compand.c | 1 - libavfilter/af_dynaudnorm.c| 7 ---

[FFmpeg-devel] [PATCH 4/5] lavfi: drop the requirement that request_frame returns a frame.

2015-08-26 Thread Nicolas George
It requires a loop in filters or the framework, that makes the scheduling less efficient and more complex. This is purely an internal change since the loop is now present in buffersink. Note that no filter except buffersink did rely on the requirement. Signed-off-by: Nicolas George

[FFmpeg-devel] [PATCH 1/5] lavfi/buffersink: loop over ff_request_frame().

2015-08-26 Thread Nicolas George
Do not assume that ff_request_frame() returning success implies a frame has arrived in the FIFO. Instead, just loop until a frame is in the FIFO. It does not change anything since the same loop is present in ff_request_frame(), confirmed by an assertion. Signed-off-by: Nicolas George

[FFmpeg-devel] [PATCH 2/5] lavfi/vf_idet: remove the loop in request_frame().

2015-08-26 Thread Nicolas George
It is not necessary due to the use of FF_LINK_FLAG_REQUEST_LOOP. Signed-off-by: Nicolas George geo...@nsup.org --- libavfilter/vf_idet.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/libavfilter/vf_idet.c b/libavfilter/vf_idet.c index eb1303a..113e8ec 100644 ---

[FFmpeg-devel] [PATCH 3/5] lavfi/vf_idet: reindent after last commit.

2015-08-26 Thread Nicolas George
Signed-off-by: Nicolas George geo...@nsup.org --- libavfilter/vf_idet.c | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) diff --git a/libavfilter/vf_idet.c b/libavfilter/vf_idet.c index 113e8ec..b6411a4 100644 --- a/libavfilter/vf_idet.c +++

Re: [FFmpeg-devel] [PATCH 4/5] lavfi: drop the requirement that request_frame returns a frame.

2015-08-26 Thread Paul B Mahol
On 8/26/15, Nicolas George geo...@nsup.org wrote: It requires a loop in filters or the framework, that makes the scheduling less efficient and more complex. This is purely an internal change since the loop is now present in buffersink. Note that no filter except buffersink did rely on the

[FFmpeg-devel] [PATCH] avfilter/vf_histogram: 9 and 10 bit depth support

2015-08-26 Thread Paul B Mahol
Signed-off-by: Paul B Mahol one...@gmail.com --- libavfilter/vf_histogram.c | 136 +++-- 1 file changed, 106 insertions(+), 30 deletions(-) diff --git a/libavfilter/vf_histogram.c b/libavfilter/vf_histogram.c index ece0b13..f95dae6 100644 ---

[FFmpeg-devel] [PATCHv3] configure: warn if GCC 4.2 is being used

2015-08-26 Thread Ganesh Ajjanagadde
The wiki, Ticket1464, and Ticket3970 warn about the usage of GCC 4.2. This fixes Ticket3970. Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- configure | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 6449018..5e49705 100755 ---

Re: [FFmpeg-devel] [PATCH] api-h264-test: rename and expand

2015-08-26 Thread Michael Niedermayer
On Thu, Aug 27, 2015 at 02:03:55AM +0300, Ludmila Glinskih wrote: Add support of floating point decoders. Add support of audio decoders. --- tests/api/Makefile | 2 +- tests/api/api-decode-test.c| 368 + tests/api/api-h264-test.c

Re: [FFmpeg-devel] [PATCH] api-h264-test: rename and expand

2015-08-26 Thread Ludmila Glinskih
Hi, Thank you for the comment! I'm not sure if I fixed it right =/ Kind regards, Ludmila Glinskih ср, 26 авг. 2015 г. в 3:52, Michael Niedermayer mich...@niedermayer.cc: On Tue, Aug 25, 2015 at 11:00:40PM +0300, Ludmila Glinskih wrote: Add support of floating point decoders. Add support of

Re: [FFmpeg-devel] [PATCH 4/5] lavfi: drop the requirement that request_frame returns a frame.

2015-08-26 Thread Michael Niedermayer
On Wed, Aug 26, 2015 at 04:46:21PM +0200, Nicolas George wrote: Le nonidi 9 fructidor, an CCXXIII, Paul B Mahol a écrit : I'm interested. Glad to read it. So here it is: Foremost, I want to remove the recursiveness. Currently, when a frame is requested on buffersink, it recursively calls

[FFmpeg-devel] [PATCH] configure: remove unused apply() function

2015-08-26 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde gajjanaga...@gmail.com --- configure | 6 -- 1 file changed, 6 deletions(-) diff --git a/configure b/configure index 5e49705..e8b38b4 100755 --- a/configure +++ b/configure @@ -1312,12 +1312,6 @@ check_host_cpp_condition(){ EOF } -apply(){ -file=$1 -

Re: [FFmpeg-devel] [PATCH]lavf/mov: Support alac extradata in wave atom v2

2015-08-26 Thread Carl Eugen Hoyos
Michael Niedermayer michael at niedermayer.cc writes: feel free to push the variant you tested Done, thank you! Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: Add -hwaccels option that lists all supported hwaccels

2015-08-26 Thread Ganesh Ajjanagadde
On Wed, Aug 26, 2015 at 9:47 PM, Timothy Gu timothyg...@gmail.com wrote: On Tue, Aug 25, 2015 at 09:25:14PM -0400, Ganesh Ajjanagadde wrote: Ok. Please place the function outside of all the opt_* stuff; since this is not setting options, it should not be in the middle of them. Otherwise, patch