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

2014-10-17 Thread Benoit Fouet
Hi, - Mail original - > On Thu, Oct 16, 2014 at 06:06:39PM +0200, Benoit Fouet wrote: > > Hi, > > > > On 16 October 2014 17:10:38 CEST, Michael Niedermayer > > wrote: > > >On Thu, Oct 16, 2014 at 11:44:47AM +0200, Benoit Fouet wrote: > > >> Some encoders do not use syncsafe sizes in v2.4

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

2014-10-17 Thread Rodger Combs
> On Oct 15, 2014, at 06:00, Rodger Combs wrote: > > >> On Oct 15, 2014, at 05:37, Nicolas George 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 message explains it wel

[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

[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 /tests/video

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

2014-10-17 Thread wm4
On Fri, 17 Oct 2014 01:55:35 -0500 Rodger Combs wrote: > > > On Oct 17, 2014, at 01:52, Rodger Combs wrote: > > > > > >> On Oct 17, 2014, at 01:16, Rodger Combs >> > wrote: > >> > >> This fixes https://trac.ffmpeg.org/ticket/3934 > >>

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 wrote: > > > > > >> On Oct 17, 2014, at 01:16, Rodger Combs >> > wrote: > >> > >> This fixes https://trac.ffmpeg.org/ticket/3934 > >>

Re: [FFmpeg-devel] [PATCH] Added STL demuxer and decoder

2014-10-17 Thread Eejya Singh
Please find the revised patch for the STL demuxer and decoder attached. I am experiencing some network problems while rsyncing from the fate-server so I couldn't add the test .stl file. On Thu, Oct 16, 2014 at 2:46 PM, Clément Bœsch wrote: > On Thu, Oct 16, 2014 at 02:26:30AM +0530, Eejya Singh

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 a

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 fi

[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/vp8

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 > >> > >

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 > foun

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

2014-10-17 Thread Mika Raento
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 store limit. --- libavformat/hlsenc.c | 10 ++ libavfor

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

2014-10-17 Thread supraja reddy
Hello , Updated the changes in the patch . Please let me know if any further changes required . Thanks , Supraja On Fri, Oct 17, 2014 at 9:21 AM, Michael Niedermayer wrote: > On Thu, Oct 16, 2014 at 11:09:27PM +0530, supraja reddy wrote: > > Updated the patch . > > Please let me know if there

[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 > 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 >> http://www.youtube.com/watch?v=_380o5B9MrA

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

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

2014-10-17 Thread Andrey Utkin
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. Andrey Utkin (5): Introduce new error codes for 4XX and 5XX replies from remote servers Introduce ff_http_averror() Use ff_http_averror() Introduce ff_r

[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 used

[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 +++ b/libavf

[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 > wrote: > > > 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 you want to send to the corners, y

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

2014-10-17 Thread Carl Eugen Hoyos
Andrey Utkin 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 to ticket #4039? If yes, please mention it. Thank you, Carl Eugen _

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

2014-10-17 Thread James Darnley
On 2014-10-17 23:09, Nicholas Robbins wrote: >> So is it possible to make a filter that is basically just a wrapper around >> another filter? This hypothetical filter (call it vf_keystone) would just >> call >> vf_perspective with crazy options. Clearly I could just copy the >> vf_perspective

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 wrote: > Andrey Utkin 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 to ticket #4039? > I

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 wrote: > > > > > > > > >> On Oct 17, 2014, at 01:16, Rodger Combs > >> > wrote:

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

2014-10-17 Thread Marton Balint
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 an imperative. I have changed all occurences of it to "enabled" o

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 > > wrote: > > > > > > > > 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 wo

[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 b/l

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

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

2014-10-17 Thread Carl Eugen Hoyos
Benoit Fouet 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 x264.h

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

2014-10-17 Thread Carl Eugen Hoyos
Benoit Fouet 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 mailing list ff

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 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 prefer

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

2014-10-17 Thread Carl Eugen Hoyos
James Almer 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 originall

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 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_e

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

2014-10-17 Thread Michael Niedermayer
On Fri, Oct 17, 2014 at 11:47:38AM +0200, Benoit Fouet wrote: > 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