Re: [FFmpeg-devel] [PATCH] avformat/id3v2: only use syncsafe for extended header size.

2014-10-14 Thread Benoit Fouet
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hi, Le 14/10/2014 04:37, Michael Niedermayer a écrit : > On Mon, Oct 13, 2014 at 03:53:32PM +0200, Benoit Fouet wrote: >> Fixes ticket #4003 >> --- >> libavformat/id3v2.c | 5 + >> 1 file changed, 1 insertion(+), 4 deletions(-) > > this breaks

Re: [FFmpeg-devel] [PATCH] avformat/id3v2: only use syncsafe for extended header size.

2014-10-14 Thread Carl Eugen Hoyos
Benoit Fouet free.fr> writes: > Another approach could be to check the next > tag to see if the size is correct, and if > not, fallback to 2.3 sizes for 2.4 tags. > Should I try that? Please do that. Do you already know what other software does? (It doesn't matter imo if checking the next ta

[FFmpeg-devel] [PATCH]Fix initialiser in dct-test.c

2014-10-14 Thread Carl Eugen Hoyos
Hi! Attached patch fixes a (fate) compilation warning on unusual platforms, I did not really check if the warning is correct though. CC libavcodec/dct-test.o libavcodec/dct-test.c:101:21: warning: missing braces around initializer [-Wmissing-braces] libavcodec/dct-test.c:101:21: warning:

Re: [FFmpeg-devel] [PATCH] avformat/id3v2: only use syncsafe for extended header size.

2014-10-14 Thread Benoit Fouet
Hi, Le 14/10/2014 10:51, Carl Eugen Hoyos a écrit : > Benoit Fouet free.fr> writes: > >> Another approach could be to check the next >> tag to see if the size is correct, and if >> not, fallback to 2.3 sizes for 2.4 tags. >> Should I try that? > Please do that. > OK. > Do you already know wha

Re: [FFmpeg-devel] [PATCH]Fix initialiser in dct-test.c

2014-10-14 Thread Michael Niedermayer
On Tue, Oct 14, 2014 at 10:59:10AM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes a (fate) compilation warning on > unusual platforms, I did not really check if the > warning is correct though. > > CC libavcodec/dct-test.o > libavcodec/dct-test.c:101:21: warning: missing bra

Re: [FFmpeg-devel] [PATCH]Fix decoding of SCE stereo aac streams

2014-10-14 Thread Michael Niedermayer
Hi On Tue, Oct 14, 2014 at 12:05:19AM +, Carl Eugen Hoyos wrote: > Michael Niedermayer gmx.at> writes: > > > > Attached patch fixes ticket #1614 for me (and > > > passes fate-aac). > > > > > > Please review, Carl Eugen > > > > i can still hear some rather loud artifacts at > > around 10s

[FFmpeg-devel] Any ffmpeg plugin for image stabilization that handles walking tilt?

2014-10-14 Thread Fernando Cassia
Hi there. I have begun playing around with using a wearable camera that just clips into your shirt, like w mic clip. It records MJPEG at 720x480 which is enough for my uses, recording daily walks around the city. But there's one issue I had not taken into account: the body (actually torso) where

Re: [FFmpeg-devel] patch: fix buf in configure to ensure theHAVE_VSX is only enabled when HAVE_ALTIVECT is enabled

2014-10-14 Thread Michael Niedermayer
On Mon, Oct 13, 2014 at 04:14:57PM +0800, rongyan wrote: > Reimar, > HAVE_VSX is used to make PPC ALTIVEC code successfully running in LE. > Yes, we can add some comment to the configure. i belive the question reimar had was why VSX is not enabled when altivec is disabled ? or the same from ano

Re: [FFmpeg-devel] Any ffmpeg plugin for image stabilization that handles walking tilt?

2014-10-14 Thread compn
On Tue, 14 Oct 2014 12:07:16 -0300 Fernando Cassia wrote: > In short: I´d like to know if there´s any FOSS filters -both for > FFMPEG and third party ones for ie Virtualdub or Avidemux, that > represents the state-of-the-art in image stabilization > post-processing wrt to walking recordings... a

Re: [FFmpeg-devel] Use ffmpeg as an Android stagefright plugin

2014-10-14 Thread Fabio Fumi
Thanks for confirming that. Could you please explain a bit this step, how it could be applied to the AOSP case: "you should merge my android_frameworks_native(branch: cm_maguro-10.1) and android_frameworks_av(branch: cm_maguro-10.1) code" thanks Fabio On Mon, Oct 13, 2014 at 4:42 PM, Michael

Re: [FFmpeg-devel] [PATCH 2/7] fate: fix some subtitles tests dependencies

2014-10-14 Thread James Darnley
On 2014-10-11 19:59, Clément Bœsch wrote: > --- > tests/fate/subtitles.mak | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Looks okay. signature.asc Description: OpenPGP digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg

Re: [FFmpeg-devel] [PATCH 3/7] avformat/assenc: support remuxing old V4 ASS form (SSA)

2014-10-14 Thread James Darnley
On 2014-10-11 19:59, Clément Bœsch wrote: > --- > libavformat/assenc.c | 11 +-- > 1 file changed, 9 insertions(+), 2 deletions(-) The diff looks fine. So if the code does what you want it to do, push. Just one comment below. > +ass->ssa_mode = !strstr(avctx->extradata, "\n[V4+

Re: [FFmpeg-devel] [PATCH 3/7] avformat/assenc: support remuxing old V4 ASS form (SSA)

2014-10-14 Thread Clément Bœsch
On Tue, Oct 14, 2014 at 08:12:38PM +0200, James Darnley wrote: > On 2014-10-11 19:59, Clément Bœsch wrote: > > --- > > libavformat/assenc.c | 11 +-- > > 1 file changed, 9 insertions(+), 2 deletions(-) > > The diff looks fine. So if the code does what you want it to do, push. > Just one

Re: [FFmpeg-devel] [PATCH 4/7] avcodec/ass: output missing fields in AVSubtitles and output files

2014-10-14 Thread James Darnley
On 2014-10-11 19:59, Clément Bœsch wrote: > Fixes Ticket #3207 > --- > libavcodec/ass.c| 29 +-- > libavcodec/version.h| 2 +- > tests/ref/fate/sub-aqtitle | 72 +-- > tests/ref/fate/sub-charenc | 106 >

Re: [FFmpeg-devel] [PATCH 4/7] avcodec/ass: output missing fields in AVSubtitles and output files

2014-10-14 Thread Clément Bœsch
On Tue, Oct 14, 2014 at 08:35:51PM +0200, James Darnley wrote: > On 2014-10-11 19:59, Clément Bœsch wrote: > > Fixes Ticket #3207 > > --- > > libavcodec/ass.c| 29 +-- > > libavcodec/version.h| 2 +- > > tests/ref/fate/sub-aqtitle | 72 +

Re: [FFmpeg-devel] [PATCH 5/7] avcodec/ass_split: extend recognized fields in ASS splitter

2014-10-14 Thread James Darnley
On 2014-10-11 19:59, Clément Bœsch wrote: > diff --git a/libavcodec/ass_split.h b/libavcodec/ass_split.h > index 06c1ce3..c912252 100644 > --- a/libavcodec/ass_split.h > +++ b/libavcodec/ass_split.h > @@ -41,13 +41,28 @@ typedef struct { > char *font_name; /**< font face (case sensitive)

Re: [FFmpeg-devel] [PATCH 5/7] avcodec/ass_split: extend recognized fields in ASS splitter

2014-10-14 Thread Clément Bœsch
On Tue, Oct 14, 2014 at 08:43:06PM +0200, James Darnley wrote: > On 2014-10-11 19:59, Clément Bœsch wrote: > > > diff --git a/libavcodec/ass_split.h b/libavcodec/ass_split.h > > index 06c1ce3..c912252 100644 > > --- a/libavcodec/ass_split.h > > +++ b/libavcodec/ass_split.h > > @@ -41,13 +41,28 @@

Re: [FFmpeg-devel] [PATCH] Add channel label and bitmap values to metadata

2014-10-14 Thread Billy Shambrook
On 10 October 2014 11:21, Billy Shambrook wrote: > --- > libavformat/mov_chan.c | 76 > +- > 1 file changed, 75 insertions(+), 1 deletion(-) > > diff --git a/libavformat/mov_chan.c b/libavformat/mov_chan.c > index 3b91ed7..aa369f2 100644 > --- a/li

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mov: auodetect "use_mfra_for"

2014-10-14 Thread Michael Niedermayer
On Mon, Oct 13, 2014 at 01:45:18PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/isom.h |1 + > libavformat/mov.c | 29 +++-- > 2 files changed, 28 insertions(+), 2 deletions(-) mika, any comments ? does this look ok to

Re: [FFmpeg-devel] [PATCH] Add channel label and bitmap values to metadata

2014-10-14 Thread Michael Niedermayer
On Fri, Oct 10, 2014 at 11:21:23AM +0100, Billy Shambrook wrote: > --- > libavformat/mov_chan.c | 76 > +- > 1 file changed, 75 insertions(+), 1 deletion(-) > > diff --git a/libavformat/mov_chan.c b/libavformat/mov_chan.c > index 3b91ed7..aa369f2 1

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mov: auodetect "use_mfra_for"

2014-10-14 Thread Mika Raento
On 14 October 2014 23:48, Michael Niedermayer wrote: > On Mon, Oct 13, 2014 at 01:45:18PM +0200, Michael Niedermayer wrote: >> Signed-off-by: Michael Niedermayer >> --- >> libavformat/isom.h |1 + >> libavformat/mov.c | 29 +++-- >> 2 files changed, 28 insertions(+

[FFmpeg-devel] [PATCH] build: mov_muxer requires ac3_parser

2014-10-14 Thread Yu Xiaolei
--- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index cda463d..3e7143c 100755 --- a/configure +++ b/configure @@ -2413,7 +2413,7 @@ matroska_muxer_select="riffenc" mmf_muxer_select="riffenc" mov_demuxer_select="riffdec" mov_demuxer_suggest

Re: [FFmpeg-devel] [PATCH] build: mov_muxer requires ac3_parser

2014-10-14 Thread Benoit Fouet
Hi, On 15 October 2014 07:33:39 CEST, Yu Xiaolei wrote: >--- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/configure b/configure >index cda463d..3e7143c 100755 >--- a/configure >+++ b/configure >@@ -2413,7 +2413,7 @@ matroska_muxer_select="riffenc" > mmf_muxer