[FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread Rodger Combs
This fixes https://trac.ffmpeg.org/ticket/3934 https://trac.ffmpeg.org/ticket/3934, but I'm not sure if there was a good reason for this to be here to begin with. Perhaps a protection against infinite recursion (though I believe EBML_MAX_DEPTH serves that purpose to some degree)?

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread Rodger Combs
On Oct 17, 2014, at 01:16, Rodger Combs rodger.co...@gmail.com wrote: This fixes https://trac.ffmpeg.org/ticket/3934 https://trac.ffmpeg.org/ticket/3934, but I'm not sure if there was a good reason for this to be here to begin with. Perhaps a protection against infinite recursion

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread Rodger Combs
On Oct 17, 2014, at 01:52, Rodger Combs rodger.co...@gmail.com wrote: On Oct 17, 2014, at 01:16, Rodger Combs rodger.co...@gmail.com mailto:rodger.co...@gmail.com wrote: This fixes https://trac.ffmpeg.org/ticket/3934 https://trac.ffmpeg.org/ticket/3934, but I'm not sure if there was

Re: [FFmpeg-devel] PATCH: Make -copyts a tri-state

2014-10-17 Thread Rodger Combs
On Oct 15, 2014, at 06:00, Rodger Combs rodger.co...@gmail.com wrote: On Oct 15, 2014, at 05:37, Nicolas George geo...@nsup.org wrote: Le quartidi 24 vendémiaire, an CCXXIII, Rodger Combs a écrit : Discussed this briefly on IRC; decided this was a good idea. I'm not sure if the commit

[FFmpeg-devel] [PATCH] avformat/id3v2: support buggy id3v2.3 tag length in id3v2.4

2014-10-17 Thread Benoit Fouet
Some encoders do not use syncsafe sizes in v2.4 id3 tags. Check the next tag to try to choose between the two. Fixes ticket #4003 --- libavformat/id3v2.c | 64 +++-- 1 file changed, 62 insertions(+), 2 deletions(-) diff --git a/libavformat/id3v2.c

Re: [FFmpeg-devel] [PATCH] avcodec/aacdec: map LFE[0] to SCE[1] for 4.0 audio.

2014-10-17 Thread Benoit Fouet
Hi, - Mail original - On Mon, Oct 13, 2014 at 02:42:55PM +0200, Benoit Fouet wrote: Fixes ticket #3930 --- libavcodec/aacdec.c | 10 ++ 1 file changed, 10 insertions(+) applied maybe these special case mappings should print some warning or some debug information,

[FFmpeg-devel] [PATCH] gitignore: add test_copy.ffmeta

2014-10-17 Thread Benoit Fouet
--- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 480fbe0..793d33a 100644 --- a/.gitignore +++ b/.gitignore @@ -64,6 +64,7 @@ /tests/data/ /tests/pixfmts.mak /tests/rotozoom +/tests/test_copy.ffmeta /tests/tiny_psnr /tests/tiny_ssim

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread wm4
On Fri, 17 Oct 2014 01:55:35 -0500 Rodger Combs rodger.co...@gmail.com wrote: On Oct 17, 2014, at 01:52, Rodger Combs rodger.co...@gmail.com wrote: On Oct 17, 2014, at 01:16, Rodger Combs rodger.co...@gmail.com mailto:rodger.co...@gmail.com wrote: This fixes

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Allocate buffer in case of long metadata entries.

2014-10-17 Thread Thilo Borgmann
Am 16.10.14 04:47, schrieb Michael Niedermayer: On Mon, Oct 13, 2014 at 09:40:42AM +0200, Thilo Borgmann wrote: Am 11.10.14 16:19, schrieb Nicolas George: [...] all remarks applied. -Thilo mov.c | 16 1 file changed, 12 insertions(+), 4 deletions(-)

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 01:55:35AM -0500, Rodger Combs wrote: On Oct 17, 2014, at 01:52, Rodger Combs rodger.co...@gmail.com wrote: On Oct 17, 2014, at 01:16, Rodger Combs rodger.co...@gmail.com mailto:rodger.co...@gmail.com wrote: This fixes https://trac.ffmpeg.org/ticket/3934

Re: [FFmpeg-devel] [PATCH] gitignore: add test_copy.ffmeta

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 11:50:53AM +0200, Benoit Fouet wrote: --- .gitignore | 1 + 1 file changed, 1 insertion(+) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Let us carefully observe those good qualities wherein our enemies excel us and

Re: [FFmpeg-devel] [PATCH] avformat/id3v2: support buggy id3v2.3 tag length in id3v2.4

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 10:56:59AM +0200, Benoit Fouet wrote: Some encoders do not use syncsafe sizes in v2.4 id3 tags. Check the next tag to try to choose between the two. Fixes ticket #4003 --- libavformat/id3v2.c | 64 +++-- 1 file

[FFmpeg-devel] [PATCH] configure: add pkg-config support for libx264

2014-10-17 Thread Benoit Fouet
--- configure | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 3e7143c..5b16af9 100755 --- a/configure +++ b/configure @@ -4880,7 +4880,10 @@ enabled libvpx { enabled libvpx_vp9_encoder { check_lib2 vpx/vpx_encoder.h vpx/vp8cx.h

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Allocate buffer in case of long metadata entries.

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 02:33:27PM +0200, Thilo Borgmann wrote: Am 16.10.14 04:47, schrieb Michael Niedermayer: On Mon, Oct 13, 2014 at 09:40:42AM +0200, Thilo Borgmann wrote: Am 11.10.14 16:19, schrieb Nicolas George: [...] all remarks applied. -Thilo mov.c | 16

Re: [FFmpeg-devel] Fix bug for POWER LE: libswscale/ppc/swscale_altivec.c

2014-10-17 Thread Michael Niedermayer
On Thu, Oct 16, 2014 at 10:48:14AM +0800, rongyan wrote: Hi, I created a patch to fix the bug in file libswscale/ppc/swscale_altivec.c for POWER LE. The fixed functions including 'hScale_altivec_real()', 'yuv2planeX_16_altivec()', and 'yuv2planeX_8()'. The fate test result can be found on

[FFmpeg-devel] Fw: [FFmpeg-user] How to create trapezoid videos with ffmpeg?

2014-10-17 Thread Nicholas Robbins
On Thursday, October 16, 2014 11:59 AM, Moritz Barsnick barsn...@gmx.net wrote: Hi Zenny, On Wed, Oct 15, 2014 at 10:55:40 +0200, Zenny wrote: New to ffmpeg. How can one create a trapezoid shaped video as a overlay to the background as seen at

Re: [FFmpeg-devel] [PATCH] hlsenc.c, segment.c: propagate defaults to mpegts

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 07:28:47PM +0300, Mika Raento wrote: This fixes the abnormally high ts overhead in the files produced by the HLS and segments muxers. See https://trac.ffmpeg.org/ticket/2857 . For example makes it much more likely that it can produces streams that fit under the 64kb App

[FFmpeg-devel] [PATCH 1/5] Introduce new error codes for 4XX and 5XX replies from remote servers

2014-10-17 Thread Andrey Utkin
--- libavutil/error.c | 6 ++ libavutil/error.h | 7 +++ 2 files changed, 13 insertions(+) diff --git a/libavutil/error.c b/libavutil/error.c index 9be78bc..d2868ac 100644 --- a/libavutil/error.c +++ b/libavutil/error.c @@ -53,6 +53,12 @@ static const struct error_entry error_entries[] =

[FFmpeg-devel] [PATCH 2/5] Introduce ff_http_averror()

2014-10-17 Thread Andrey Utkin
int ff_http_averror(int status_code, int default_averror) This helper function returns AVERROR_ value from 3-digit HTTP status code. Second argument, default_averror, is used if no specific AVERROR_ is available. It is introduced because in different places of code different return codes are

[FFmpeg-devel] [PATCH 5/5] Use ff_rtsp_averror()

2014-10-17 Thread Andrey Utkin
--- libavformat/rtsp.c| 4 ++-- libavformat/rtspdec.c | 27 ++- libavformat/rtspenc.c | 4 ++-- 3 files changed, 18 insertions(+), 17 deletions(-) diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index 77f03ba..1682db8 100644 --- a/libavformat/rtsp.c +++

[FFmpeg-devel] [PATCH 3/5] Use ff_http_averror()

2014-10-17 Thread Andrey Utkin
--- libavformat/http.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/http.c b/libavformat/http.c index 1958a1b..d12dcaa 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -250,7 +250,7 @@ redo: fail: if (s-hd) ffurl_closep(s-hd); -

[FFmpeg-devel] [PATCH 4/5] Introduce ff_rtsp_averror()

2014-10-17 Thread Andrey Utkin
Currently this is another name for ff_http_averror() --- libavformat/rtspcodes.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavformat/rtspcodes.h b/libavformat/rtspcodes.h index d4ae0a8..0ae490a 100644 --- a/libavformat/rtspcodes.h +++ b/libavformat/rtspcodes.h @@ -25,6 +25,7 @@

Re: [FFmpeg-devel] [FFmpeg-user] How to create trapezoid videos with ffmpeg?

2014-10-17 Thread Nicholas Robbins
On Friday, October 17, 2014 2:30 PM, Nicholas Robbins nickrobb...@yahoo.com wrote: snip actual request Ok, it seems like this should be easier. If we had a filter that did the same thing as vf_perspective but with inverse options this would be easy. So instead telling it what points

Re: [FFmpeg-devel] [PATCH 0/5] Add HTTP/RTSP specific AVERROR codes

2014-10-17 Thread Andrey Utkin
On Sat, Oct 18, 2014 at 1:19 AM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Andrey Utkin andrey.utkin at corp.bluecherry.net writes: Any comments and testing are appreciated. I have made a little amount of testing, but RTSP, HTTP and HTTPS 401 and 404 work as expected. Is the patchset related

Re: [FFmpeg-devel] [PATCH] matroskadec: execute seekheads recursively

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 03:27:49PM +0200, Michael Niedermayer wrote: On Fri, Oct 17, 2014 at 01:55:35AM -0500, Rodger Combs wrote: On Oct 17, 2014, at 01:52, Rodger Combs rodger.co...@gmail.com wrote: On Oct 17, 2014, at 01:16, Rodger Combs rodger.co...@gmail.com

Re: [FFmpeg-devel] [FFmpeg-user] How to create trapezoid videos with ffmpeg?

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 02:09:49PM -0700, Nicholas Robbins wrote: On Friday, October 17, 2014 2:30 PM, Nicholas Robbins nickrobb...@yahoo.com wrote: snip actual request Ok, it seems like this should be easier. If we had a filter that did the same thing as vf_perspective but

[FFmpeg-devel] [PATCH] Add support for Opus in MPEG-TS

2014-10-17 Thread Kieran Kunhya
--- libavcodec/opus.c| 11 +--- libavcodec/opus.h|9 +++ libavcodec/opus_parser.c | 139 +- libavformat/mpegts.c | 54 +- 4 files changed, 191 insertions(+), 22 deletions(-) diff --git a/libavcodec/opus.c

Re: [FFmpeg-devel] [PATCHv2] ffplay: update documentation

2014-10-17 Thread Michael Niedermayer
On Sat, Oct 18, 2014 at 01:22:03AM +0200, Marton Balint wrote: On Thu, 16 Oct 2014, Lou Logan wrote: On Wed, 15 Oct 2014 23:46:21 +0200, Marton Balint wrote: Thank you for all your feedback, apparently there was some confusion because I have used set both as a synonym to enabled and as

Re: [FFmpeg-devel] [PATCH] configure: add pkg-config support for libx264

2014-10-17 Thread Carl Eugen Hoyos
Benoit Fouet benoit.fouet at free.fr writes: +enabled libx264{ { check_pkg_config x264 stdint.h x264.h x264_encoder_encode + require_pkg_config x264 stdint.h x264.h x264_encoder_encode; } || + { require libx264

Re: [FFmpeg-devel] [PATCH] configure: add pkg-config support for libx264

2014-10-17 Thread Carl Eugen Hoyos
Benoit Fouet benoit.fouet at free.fr writes: +enabled libx264{ { check_pkg_config x264 I forgot: You could add a line to the Changelog pkg-config is now preferred over --extra-*flags for x264 detection. Carl Eugen ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] configure: add pkg-config support for libx264

2014-10-17 Thread James Almer
On 17/10/14 8:49 PM, Carl Eugen Hoyos wrote: Benoit Fouet benoit.fouet at free.fr writes: +enabled libx264{ { check_pkg_config x264 I forgot: You could add a line to the Changelog pkg-config is now preferred over --extra-*flags for x264 detection. No need to mention it's

Re: [FFmpeg-devel] [PATCH] configure: add pkg-config support for libx264

2014-10-17 Thread Carl Eugen Hoyos
James Almer jamrial at gmail.com writes: You could add a line to the Changelog pkg-config is now preferred over --extra-*flags for x264 detection. A line saying something like x264 may now also be detected with pkg-config is enough. I disagree (this would be the change that I

Re: [FFmpeg-devel] Patch for Cast128 algorithm implementation

2014-10-17 Thread Michael Niedermayer
On Sat, Oct 18, 2014 at 12:00:28AM +0530, supraja reddy wrote: Hello , Updated the changes in the patch . Please let me know if any further changes required . [...] Changelog|1 libavutil/Makefile |3 libavutil/cast5.c| 541

Re: [FFmpeg-devel] [PATCH] configure: add pkg-config support for libx264

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 11:48:12PM +, Carl Eugen Hoyos wrote: Benoit Fouet benoit.fouet at free.fr writes: +enabled libx264{ { check_pkg_config x264 stdint.h x264.h x264_encoder_encode + require_pkg_config x264 stdint.h x264.h