Re: [FFmpeg-devel] [PATCH] transcode_aac: fix const return value

2014-08-05 Thread Michael Niedermayer
On Tue, Aug 05, 2014 at 05:59:15PM -0700, Timothy Gu wrote: > Fixes Trac #3613. > > Signed-off-by: Timothy Gu > --- > doc/examples/transcode_aac.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC78704

[FFmpeg-devel] [PATCH] transcode_aac: fix const return value

2014-08-05 Thread Timothy Gu
Fixes Trac #3613. Signed-off-by: Timothy Gu --- doc/examples/transcode_aac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/examples/transcode_aac.c b/doc/examples/transcode_aac.c index bf0128f..cee447f 100644 --- a/doc/examples/transcode_aac.c +++ b/doc/examples/transco

Re: [FFmpeg-devel] [PATCH]Make loas probe more similar to mp3 probe

2014-08-05 Thread Michael Niedermayer
On Tue, Aug 05, 2014 at 10:28:38PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch copies one line from mp3dec.c into loasdec.c to force probing > more data and fixes ticket #3821. An alternative is to remove the offending > line from mp3dec.c (it was added in 2006 or earlier). > > Plea

Re: [FFmpeg-devel] [PATCH] avutil/cpu: output cpu data to stdout

2014-08-05 Thread Michael Niedermayer
On Tue, Aug 05, 2014 at 08:43:58PM -0300, James Almer wrote: > On 05/08/14 8:16 PM, Michael Niedermayer wrote: > > Errors go to stderr, but the cpu stats are non error output for cputest > > > > This fixes echoing the cpu test results > > > > Signed-off-by: Michael Niedermayer > > --- > > libav

Re: [FFmpeg-devel] [PATCH] tests/fate/libavutil: switch cpu-test back to runecho so its results are vissible

2014-08-05 Thread Michael Niedermayer
On Tue, Aug 05, 2014 at 08:44:16PM -0300, James Almer wrote: > On 05/08/14 8:17 PM, Michael Niedermayer wrote: > > (needs next commt too) > > > > Signed-off-by: Michael Niedermayer > > --- > > tests/fate/libavutil.mak |2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --g

Re: [FFmpeg-devel] [PATCH] tests/fate/libavutil: switch cpu-test back to runecho so its results are vissible

2014-08-05 Thread James Almer
On 05/08/14 8:17 PM, Michael Niedermayer wrote: > (needs next commt too) > > Signed-off-by: Michael Niedermayer > --- > tests/fate/libavutil.mak |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/fate/libavutil.mak b/tests/fate/libavutil.mak > index 9f676a8..579bd

Re: [FFmpeg-devel] [PATCH] avutil/cpu: output cpu data to stdout

2014-08-05 Thread James Almer
On 05/08/14 8:16 PM, Michael Niedermayer wrote: > Errors go to stderr, but the cpu stats are non error output for cputest > > This fixes echoing the cpu test results > > Signed-off-by: Michael Niedermayer > --- > libavutil/cpu.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(

[FFmpeg-devel] [PATCH] tests/fate/libavutil: switch cpu-test back to runecho so its results are vissible

2014-08-05 Thread Michael Niedermayer
(needs next commt too) Signed-off-by: Michael Niedermayer --- tests/fate/libavutil.mak |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/fate/libavutil.mak b/tests/fate/libavutil.mak index 9f676a8..579bdfb 100644 --- a/tests/fate/libavutil.mak +++ b/tests/fate/libavuti

[FFmpeg-devel] [PATCH] avutil/cpu: output cpu data to stdout

2014-08-05 Thread Michael Niedermayer
Errors go to stderr, but the cpu stats are non error output for cputest This fixes echoing the cpu test results Signed-off-by: Michael Niedermayer --- libavutil/cpu.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavutil/cpu.c b/libavutil/cpu.c index 03f89cb

[FFmpeg-devel] [PATCH]Make loas probe more similar to mp3 probe

2014-08-05 Thread Carl Eugen Hoyos
Hi! Attached patch copies one line from mp3dec.c into loasdec.c to force probing more data and fixes ticket #3821. An alternative is to remove the offending line from mp3dec.c (it was added in 2006 or earlier). Please review, Carl Eugen diff --git a/libavformat/loasdec.c b/libavformat/loasdec.c

Re: [FFmpeg-devel] [PATCH]Do not select a default subtitle output stream with incorrect subtitle type

2014-08-05 Thread Carl Eugen Hoyos
On Friday 01 August 2014 10:04:17 am Clément Bœsch wrote: > On Fri, Aug 01, 2014 at 01:54:57AM +0200, Carl Eugen Hoyos wrote: > > When transcoding an input file with subtitles to mkv, ffmpeg by default > > tries to encode the subtitles even if the input contains bitmap > > subtitles. Attached patch

Re: [FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

2014-08-05 Thread Clément Bœsch
On Sun, Aug 03, 2014 at 12:36:19AM +0200, Michael Niedermayer wrote: > On Sat, Aug 02, 2014 at 11:34:07PM +0200, Clément Bœsch wrote: [...] > > +#ifdef TEST > > +#define W1 320 > > +#define H1 240 > > +#define W2 640 > > +#define H2 480 > > +int main(void) > > +{ > > +int i, a, ret = 0; > > +

Re: [FFmpeg-devel] [PATCH] [RFC] avcodec: export MB information in frame side data

2014-08-05 Thread Michael Niedermayer
On Tue, Aug 05, 2014 at 04:23:14PM +0200, Clément Bœsch wrote: > On Tue, Jul 22, 2014 at 01:51:46AM +0200, Michael Niedermayer wrote: > [...] > > > +typedef struct AVMBInfo_MB { > > > > > +int8_t source; /* -1/+1 XXX: set exact relative ref frame > > instead of "direction" */ > > > > i s

Re: [FFmpeg-devel] [PATCH] [RFC] avcodec: export MB information in frame side data

2014-08-05 Thread Michael Niedermayer
On Tue, Aug 05, 2014 at 04:27:53PM +0200, wm4 wrote: > On Tue, 22 Jul 2014 01:51:46 +0200 > Michael Niedermayer wrote: > > > On Wed, Jul 16, 2014 at 05:06:27PM +0200, Clément Bœsch wrote: > > > Also add an example exporting the MB information as a CSV stream. > > > > > > --- > > > > > > A bunch

[FFmpeg-devel] [PATCH] avdevice/lavfi: Simplify non-AV stream check

2014-08-05 Thread Michael Niedermayer
Fixes a clang warning Signed-off-by: Michael Niedermayer --- libavdevice/lavfi.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c index f1c88ac..4cfe804 100644 --- a/libavdevice/lavfi.c +++ b/libavdevice/lavfi.c @@ -184,1

Re: [FFmpeg-devel] [recentering] Reintroducing FFmpeg to Debian

2014-08-05 Thread Niv Sardi
I feel the debate is going a bit on a tangent in this thread, so I'd like to take an opportunity to recenter it a tad.​ ​We have many issues that were risen in this thread, ​ ​but ​ I believe that the cut has to be made by the people that we have in special roles for; -security for security concer

Re: [FFmpeg-devel] [PATCH] [RFC] avcodec: export MB information in frame side data

2014-08-05 Thread Clément Bœsch
On Tue, Jul 22, 2014 at 01:51:46AM +0200, Michael Niedermayer wrote: [...] > > +typedef struct AVMBInfo_MB { > > > +int8_t source; /* -1/+1 XXX: set exact relative ref frame instead > of "direction" */ > > i suspect this is too small for long term references Would signed 16-bit enough?

Re: [FFmpeg-devel] [PATCH] [RFC] avcodec: export MB information in frame side data

2014-08-05 Thread wm4
On Tue, 22 Jul 2014 01:51:46 +0200 Michael Niedermayer wrote: > On Wed, Jul 16, 2014 at 05:06:27PM +0200, Clément Bœsch wrote: > > Also add an example exporting the MB information as a CSV stream. > > > > --- > > > > A bunch of questions stand: > > > > - Ideally the "source" for a given macrob

Re: [FFmpeg-devel] [PATCH] New p2p mode for showwaves filter

2014-08-05 Thread Michael Niedermayer
On Tue, Aug 05, 2014 at 08:13:18AM +0200, Paul B Mahol wrote: > On Mon, Aug 4, 2014 at 11:40 AM, mrskman wrote: > > > New patch attached. > > > > Thank you. > > > > --- > > doc/filters.texi|3 +++ > > libavfilter/avf_showwaves.c | 30 +- > > 2 files