Re: [FFmpeg-devel] [PATCH 2/3] ffplay: remove manual bug option handling code

2014-10-13 Thread Michael Niedermayer
On Mon, Oct 13, 2014 at 11:41:42PM +0200, Marton Balint wrote: > It is supported as a codec option, and those are already parsed. > > Signed-off-by: Marton Balint > --- > doc/ffplay.texi | 2 -- > ffplay.c| 3 --- > 2 files changed, 5 deletions(-) LGTM [...] -- Michael GnuPG finge

Re: [FFmpeg-devel] [PATCH] doc/encoders: add basic libx265 documentation

2014-10-13 Thread Michael Niedermayer
On Thu, Oct 02, 2014 at 03:19:21PM -0800, Lou Logan wrote: > Fixes ticket #3944. > > Signed-off-by: Lou Logan > --- > doc/encoders.texi | 28 > 1 file changed, 28 insertions(+) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB W

Re: [FFmpeg-devel] [PATCH] doc/filters: update link to OpenCV docs

2014-10-13 Thread Michael Niedermayer
On Mon, Oct 13, 2014 at 04:42:05PM -0800, Lou Logan wrote: > Signed-off-by: Lou Logan > --- > doc/filters.texi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/doc/filters.texi b/doc/filters.texi > index b9575f4..e9c1a24 100644 > --- a/doc/filters.texi > +++ b/doc/filters

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

2014-10-13 Thread Michael Niedermayer
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 fate-exif-image-embedded [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Brea

Re: [FFmpeg-devel] [PATCH 3/3] ffplay: update documentation

2014-10-13 Thread compn
On Mon, 13 Oct 2014 14:44:02 -0800 Lou Logan wrote: > On Mon, 13 Oct 2014 23:41:43 +0200, Marton Balint wrote: > > +@item -framedrop > > +Drop video frames if video is out of sync. Set by default, if sync > > mode is not +video. Set this option to enable frame dropping for > > all sync modes, set

Re: [FFmpeg-devel] [PATCH] doc/encoders: add basic libx265 documentation

2014-10-13 Thread Lou Logan
On Thu, 2 Oct 2014 15:19:21 -0800, Lou Logan wrote: > Fixes ticket #3944. > > Signed-off-by: Lou Logan > --- > doc/encoders.texi | 28 > 1 file changed, 28 insertions(+) Ping. ___ ffmpeg-devel mailing list ffmpeg-devel@f

[FFmpeg-devel] [PATCH] doc/filters: update link to OpenCV docs

2014-10-13 Thread Lou Logan
Signed-off-by: Lou Logan --- doc/filters.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/filters.texi b/doc/filters.texi index b9575f4..e9c1a24 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@ -6226,7 +6226,7 @@ values are assumed. Refer to the official libo

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

2014-10-13 Thread Carl Eugen Hoyos
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 10sec in the file Yes, I have no idea how to fix this. The artefacts do not appear with fa

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

2014-10-13 Thread Michael Niedermayer
On Mon, Oct 13, 2014 at 10:38:58PM +0200, Carl Eugen Hoyos wrote: > Hi! > > 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 10sec in the file [...] -- Michael GnuPG fingerprint: 9FF2128B147

Re: [FFmpeg-devel] [PATCH 3/3] ffplay: update documentation

2014-10-13 Thread Lou Logan
Hi, Just some nits. On Mon, 13 Oct 2014 23:41:43 +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > doc/ffplay.texi | 24 +++- > 1 file changed, 19 insertions(+), 5 deletions(-) > > diff --git a/doc/ffplay.texi b/doc/ffplay.texi > index 9bb040e..4c01e6c 100

Re: [FFmpeg-devel] [PATCH 3/3] ffplay: update documentation

2014-10-13 Thread Timothy Gu
On Mon, Oct 13, 2014 at 2:41 PM, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > doc/ffplay.texi | 24 +++- > 1 file changed, 19 insertions(+), 5 deletions(-) > > diff --git a/doc/ffplay.texi b/doc/ffplay.texi > index 9bb040e..4c01e6c 100644 > --- a/doc/ffplay.tex

Re: [FFmpeg-devel] opw@ffmpeg mail alias

2014-10-13 Thread Michael Niedermayer
On Mon, Oct 13, 2014 at 09:34:55PM +0200, Nicolas George wrote: > Le duodi 22 vendémiaire, an CCXXIII, Giorgio Vazzana a écrit : > > Sure, add me, thanks. > > Me2 please. both added Thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Let us carefully observ

Re: [FFmpeg-devel] gnome-opw 2015

2014-10-13 Thread compn
On Tue, 14 Oct 2014 01:37:32 +0530 Nikita Mantri wrote: > Hi, > For the generic CMAC module, can I reuse the code of md5 and sha from > libavutil, or should I implement this hash function again. please reuse it. we do not want to duplicate code. :) -compn ___

[FFmpeg-devel] [PATCH 3/3] ffplay: update documentation

2014-10-13 Thread Marton Balint
Signed-off-by: Marton Balint --- doc/ffplay.texi | 24 +++- 1 file changed, 19 insertions(+), 5 deletions(-) diff --git a/doc/ffplay.texi b/doc/ffplay.texi index 9bb040e..4c01e6c 100644 --- a/doc/ffplay.texi +++ b/doc/ffplay.texi @@ -37,10 +37,14 @@ Force displayed height. S

[FFmpeg-devel] [PATCH 2/3] ffplay: remove manual bug option handling code

2014-10-13 Thread Marton Balint
It is supported as a codec option, and those are already parsed. Signed-off-by: Marton Balint --- doc/ffplay.texi | 2 -- ffplay.c| 3 --- 2 files changed, 5 deletions(-) diff --git a/doc/ffplay.texi b/doc/ffplay.texi index 203085c..9bb040e 100644 --- a/doc/ffplay.texi +++ b/doc/ffplay.

[FFmpeg-devel] [PATCH 1/3] ffplay: remove delays when paused from video and subtitle thread

2014-10-13 Thread Marton Balint
Once the frame queue is full, we will wait anyway. Signed-off-by: Marton Balint --- ffplay.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/ffplay.c b/ffplay.c index 37983e9..960fdc8 100644 --- a/ffplay.c +++ b/ffplay.c @@ -2122,9 +2122,6 @@ static int video_thread(void *arg) #endif

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

2014-10-13 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #1614 for me (and passes fate-aac). Please review, Carl Eugen diff --git a/libavcodec/aacdec.c b/libavcodec/aacdec.c index ef820d2..2c20020 100644 --- a/libavcodec/aacdec.c +++ b/libavcodec/aacdec.c @@ -656,7 +656,7 @@ static ChannelElement *get_che(AACContext *ac,

Re: [FFmpeg-devel] gnome-opw 2015

2014-10-13 Thread Giorgio Vazzana
Hi, 2014-10-13 22:07 GMT+02:00 Nikita Mantri : > Hi, > For the generic CMAC module, can I reuse the code of md5 and sha from > libavutil, or should I implement this hash function again. You can reuse the existing code, of course. ___ ffmpeg-devel mailin

Re: [FFmpeg-devel] [PATCH] avformat/movenc: add support for syncframes concatenation for E-AC-3.

2014-10-13 Thread Michael Niedermayer
On Mon, Oct 13, 2014 at 09:35:11AM +0200, Benoit Fouet wrote: > E-AC-3 samples should contain 6 audio blocks, so concatenate syncframes > in order to achieve this. > --- > libavformat/movenc.c | 43 +-- > 1 file changed, 37 insertions(+), 6 deletions(-) app

Re: [FFmpeg-devel] gnome-opw 2015

2014-10-13 Thread Nikita Mantri
Hi, For the generic CMAC module, can I reuse the code of md5 and sha from libavutil, or should I implement this hash function again. Thanks, Nikita Mantri On 13 October 2014 05:49, Michael Niedermayer wrote: > On Mon, Oct 13, 2014 at 05:43:28AM +0530, Nikita Mantri wrote: > > Hi, > > Should I s

Re: [FFmpeg-devel] opw@ffmpeg mail alias

2014-10-13 Thread Nicolas George
Le duodi 22 vendémiaire, an CCXXIII, Giorgio Vazzana a écrit : > Sure, add me, thanks. Me2 please. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mail

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

2014-10-13 Thread Michael Chen
Hi, Oh, NO, it is easy that Compatible with older Android versions, like 4.1.2 or 4.0.4 AOSP On Mon, Oct 13, 2014 at 6:24 PM, Fabio Fumi wrote: > Hi, > > I'd like to ask for your generic advice in back-porting your work to my > build, which is a slightly customized 4.1.2 AOSP (found here: >

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

2014-10-13 Thread Fabio Fumi
Hi, I'd like to ask for your generic advice in back-porting your work to my build, which is a slightly customized 4.1.2 AOSP (found here: https://github.com/Renesas-EMEV2). I think the pieces are all there already, but I'm wondering whether there's something preventing that, or making it too c

Re: [FFmpeg-devel] opw@ffmpeg mail alias

2014-10-13 Thread Giorgio Vazzana
2014-10-13 3:49 GMT+02:00 Michael Niedermayer : > Hi all > > does any mentor want to be added to the opw@ffmpeg alias > if so, tell me and ill add you Sure, add me, thanks. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/l

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

2014-10-13 Thread Michael Niedermayer
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, could help if they are ever wrong tha

Re: [FFmpeg-devel] Eager to work on the MPEG-4 Audio Lossless Coding (ALS) Encoder

2014-10-13 Thread Thilo Borgmann
Hi, > I am Ngassa Amalia Finjap ( amalia on irc ) - a Management major who codes in > C > and C++ and interested in working on the MPEG-4 Audio Lossless Coding (ALS) > Encoder under the Outreach programs For Women (OPW) program.I am currently > working on my OPW project proposal and some qualific

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mov: fix mix of declaration and statement

2014-10-13 Thread Michael Niedermayer
On Mon, Oct 13, 2014 at 01:45:19PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/mov.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Awnsering whe

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

2014-10-13 Thread Benoit Fouet
Fixes ticket #4003 --- libavformat/id3v2.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/libavformat/id3v2.c b/libavformat/id3v2.c index 5469e0a..d3dc647 100644 --- a/libavformat/id3v2.c +++ b/libavformat/id3v2.c @@ -732,10 +732,7 @@ static void id3v2_parse(AVIOContext *p

[FFmpeg-devel] Eager to work on the MPEG-4 Audio Lossless Coding (ALS) Encoder

2014-10-13 Thread Ngassa Finjap
Hello everyone, I am Ngassa Amalia Finjap ( amalia on irc ) - a Management major who codes in C and C++ and interested in working on the MPEG-4 Audio Lossless Coding (ALS) Encoder under the Outreach programs For Women (OPW) program.I am currently working on my OPW project proposal and some qualifi

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

2014-10-13 Thread Benoit Fouet
Fixes ticket #3930 --- libavcodec/aacdec.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/libavcodec/aacdec.c b/libavcodec/aacdec.c index 10c509b..ef820d2 100644 --- a/libavcodec/aacdec.c +++ b/libavcodec/aacdec.c @@ -630,6 +630,16 @@ static ChannelElement *get_che(AACContext *ac,

Re: [FFmpeg-devel] [PATCH] mov.c: allow reading fragment start dts/pts from fragmented mp4

2014-10-13 Thread Michael Niedermayer
On Mon, Oct 13, 2014 at 12:36:30PM +0200, Michael Niedermayer wrote: > On Mon, Oct 13, 2014 at 08:54:31AM +0300, Mika Raento wrote: > > On 11 October 2014 23:02, Michael Niedermayer wrote: > > > On Sat, Oct 11, 2014 at 08:38:14PM +0300, Mika Raento wrote: > > >> On 11 October 2014 20:20, Michael N

[FFmpeg-devel] [PATCH 2/2] avformat/mov: fix mix of declaration and statement

2014-10-13 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/mov.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 294b38a..17551cd 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -780,9 +780,9 @@ static int mov_read_moof(MOVConte

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

2014-10-13 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/isom.h |1 + libavformat/mov.c | 29 +++-- 2 files changed, 28 insertions(+), 2 deletions(-) diff --git a/libavformat/isom.h b/libavformat/isom.h index 343cdd3..52da1a3 100644 --- a/libavformat/isom.h +++ b/libavfo

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

2014-10-13 Thread Nicolas George
Le duodi 22 vendémiaire, an CCXXIII, Reimar Döffinger a écrit : > Uh, wouldn't that be undefined behaviour? > You're not allowed to compare pointers from different allocations as far > as I know, even if it usually works. The spec says: # Two pointers compare equal if and only if both are null po

Re: [FFmpeg-devel] [PATCH] mov.c: allow reading fragment start dts/pts from fragmented mp4

2014-10-13 Thread Michael Niedermayer
On Mon, Oct 13, 2014 at 08:54:31AM +0300, Mika Raento wrote: > On 11 October 2014 23:02, Michael Niedermayer wrote: > > On Sat, Oct 11, 2014 at 08:38:14PM +0300, Mika Raento wrote: > >> On 11 October 2014 20:20, Michael Niedermayer wrote: > >> > On Sat, Oct 11, 2014 at 06:43:48PM +0300, Mika Raen

Re: [FFmpeg-devel] opw@ffmpeg mail alias

2014-10-13 Thread Michael Niedermayer
On Mon, Oct 13, 2014 at 03:49:07AM +0200, Michael Niedermayer wrote: > Hi all > > does any mentor want to be added to the opw@ffmpeg alias > if so, tell me and ill add you reimar, thilo, tomas added thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB He wh

[FFmpeg-devel] re-submit: patch: fix buf in configure to ensure the HAVE_VSX is only enabled when HAVE_ALTIVECT is enabled

2014-10-13 Thread rongyan
Hi, We add some comments in the file 'configure'. The new patch is attached. Thanks. Rong Yan -- The world has enough for everyone's need, but not enough for everyone's greed. -- Original -- From: "rongyan";; Date: Mo

Re: [FFmpeg-devel] opw@ffmpeg mail alias

2014-10-13 Thread tomas . hardin
On 2014-10-13 03:49, Michael Niedermayer wrote: Hi all does any mentor want to be added to the opw@ffmpeg alias if so, tell me and ill add you thanks Add me too. I had an applicant PM me on IRC (shreshtha), but I haven't been able to reply to them since they're only online sporadically. /T

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

2014-10-13 Thread rongyan
Reimar, HAVE_VSX is used to make PPC ALTIVEC code successfully running in LE. Yes, we can add some comment to the configure. Rong Yan -- The world has enough for everyone's need, but not enough for everyone's greed. -- Original

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

2014-10-13 Thread Thilo Borgmann
Am 13.10.14 09:57, schrieb Reimar Döffinger: > On 11.10.2014, at 16:19, Nicolas George wrote: >>> +if (str_size > sizeof(str)-1) { // free buffer for long data field >>> +av_freep(&pstr); >> >> I think "if (pstr != str)" would be more robust. > > Uh, wouldn't that be undefined

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

2014-10-13 Thread Reimar Döffinger
On 13.10.2014, at 09:04, "rongyan" wrote: > Hi, > We present this patch to fix the bug in 'configure' file to make sure that > the HAVE_VSX is only enabled when HAVE_ALTIVEC is enabled on POWER LE, and > disabled when HAVE_ALTIVEC is disabled. Sorry if my question makes no sense (I am fairly cl

Re: [FFmpeg-devel] opw@ffmpeg mail alias

2014-10-13 Thread Reimar Döffinger
On 13.10.2014, at 09:22, Thilo Borgmann wrote: > Am 13.10.14 03:49, schrieb Michael Niedermayer: >> Hi all >> >> does any mentor want to be added to the opw@ffmpeg alias if so, tell me >> and ill add you > > add me please - thanks! Me too, I got a few questions though I have not yet answered al

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

2014-10-13 Thread Reimar Döffinger
On 11.10.2014, at 16:19, Nicolas George wrote: >> +if (str_size > sizeof(str)-1) { // free buffer for long data field >> +av_freep(&pstr); > > I think "if (pstr != str)" would be more robust. Uh, wouldn't that be undefined behaviour? You're not allowed to compare pointers fro

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

2014-10-13 Thread Thilo Borgmann
Am 11.10.14 16:19, schrieb Nicolas George: > [...] all remarks applied. -Thilo >From 5a14ef97ffc7d82dea5644c736e6dc2de2079e89 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Mon, 13 Oct 2014 09:36:17 +0200 Subject: [PATCH] lavf/mov.c: Allocate buffer in case of long metadata entries. ---

[FFmpeg-devel] [PATCH] avformat/movenc: add support for syncframes concatenation for E-AC-3.

2014-10-13 Thread Benoit Fouet
E-AC-3 samples should contain 6 audio blocks, so concatenate syncframes in order to achieve this. --- libavformat/movenc.c | 43 +-- 1 file changed, 37 insertions(+), 6 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 7b1554e..d8b

Re: [FFmpeg-devel] opw@ffmpeg mail alias

2014-10-13 Thread Thilo Borgmann
Am 13.10.14 03:49, schrieb Michael Niedermayer: > Hi all > > does any mentor want to be added to the opw@ffmpeg alias if so, tell me > and ill add you add me please - thanks! -Thilo ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.o

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

2014-10-13 Thread rongyan
Hi, We present this patch to fix the bug in 'configure' file to make sure that the HAVE_VSX is only enabled when HAVE_ALTIVEC is enabled on POWER LE, and disabled when HAVE_ALTIVEC is disabled. The fate test result is attached here: The patch is attached. Thanks. Rong Yan