[FFmpeg-devel] [PATCH 2/3] lavc: bump minor version

2015-10-18 Thread Rostislav Pehlivanov
Due to the previous commit slightly modifying the API. --- libavcodec/version.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/version.h b/libavcodec/version.h index 97f4ae4..b9aab44 100644 --- a/libavcodec/version.h +++ b/libavcodec/version.h @@ -29,7 +29,7 @@ #in

[FFmpeg-devel] [PATCH 3/3] lavf: add oggparsedaala

2015-10-18 Thread Rostislav Pehlivanov
Basic Daala format header reading and parsing. --- libavformat/Makefile| 1 + libavformat/oggdec.c| 1 + libavformat/oggdec.h| 1 + libavformat/oggparsedaala.c | 248 4 files changed, 251 insertions(+) create mode 100644 l

[FFmpeg-devel] [PATCH 1/3] lavc: add AV_CODEC_ID_DAALA and its description

2015-10-18 Thread Rostislav Pehlivanov
--- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 +++ 2 files changed, 8 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 18889ec..8512d31 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -220,6 +220,7 @@ enum AVCodecID { AV_CODEC_

[FFmpeg-devel] [PATCH 0/3] libavformat: add support for parsing of Daala video

2015-10-18 Thread Rostislav Pehlivanov
This commit would be the first step in adding decoding support for the Daala next generation video format. With this commit, FFmpeg would be able to identify the format, duration, version, estimate the bitrate, etc. of a video stream encoded using the current git master of daala. Should the bitstre

Re: [FFmpeg-devel] [PATCH] dca_parser: don't overwrite the sample rate, it may not be correct

2015-10-18 Thread Michael Niedermayer
On Sun, Oct 18, 2015 at 04:49:48PM +0200, Hendrik Leppkes wrote: > On Wed, Sep 30, 2015 at 1:09 PM, Hendrik Leppkes wrote: > > The parser only reads the dca core sample rate, which is limited to a > > maximum of 48000 Hz, while X96 and HD extensions can increase the sample > > rate up to 192000 Hz

Re: [FFmpeg-devel] [PATCHv2 4/5] ffmpeg: exit on corrupt packets or decoded frames if exit_on_error flag is present

2015-10-18 Thread Michael Niedermayer
On Mon, Oct 19, 2015 at 01:57:31AM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > ffmpeg.c | 20 > 1 file changed, 16 insertions(+), 4 deletions(-) should be ok thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No sn

Re: [FFmpeg-devel] [PATCH 5/5] ffmpeg: add abort_on option to allow aborting on empty output

2015-10-18 Thread Marton Balint
On Mon, 19 Oct 2015, Michael Niedermayer wrote: On Sun, Oct 18, 2015 at 12:24:07AM +0200, Marton Balint wrote: Signed-off-by: Marton Balint --- doc/ffmpeg.texi | 8 ffmpeg.c| 7 +++ ffmpeg.h| 3 +++ ffmpeg_opt.c| 21 + 4 files changed,

[FFmpeg-devel] [PATCHv2 4/5] ffmpeg: exit on corrupt packets or decoded frames if exit_on_error flag is present

2015-10-18 Thread Marton Balint
Signed-off-by: Marton Balint --- ffmpeg.c | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index fc29ad5..37a819e 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -1925,13 +1925,20 @@ int guess_input_channel_layout(InputStream *ist) return

[FFmpeg-devel] [PATCH] lavf/http: fix incorrect warning in range requests

2015-10-18 Thread Rodger Combs
--- libavformat/http.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/libavformat/http.c b/libavformat/http.c index 075dda7..9b5346c 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -1155,15 +1155,16 @@ static int http_buf_read(URLContext *h, uint8_t *buf, i

Re: [FFmpeg-devel] [PATCH 5/5] ffmpeg: add abort_on option to allow aborting on empty output

2015-10-18 Thread Michael Niedermayer
On Sun, Oct 18, 2015 at 12:24:07AM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > doc/ffmpeg.texi | 8 > ffmpeg.c| 7 +++ > ffmpeg.h| 3 +++ > ffmpeg_opt.c| 21 + > 4 files changed, 39 insertions(+) LGTM but maybe others

Re: [FFmpeg-devel] [PATCH 4/5] ffmpeg: exit on corrupt packets or decoded frames if exit_on_error flag is present

2015-10-18 Thread Michael Niedermayer
On Sun, Oct 18, 2015 at 12:24:06AM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > ffmpeg.c | 20 > 1 file changed, 16 insertions(+), 4 deletions(-) > > diff --git a/ffmpeg.c b/ffmpeg.c > index 3a36af7..f6947f8 100644 > --- a/ffmpeg.c > +++ b/ffmpeg.c > @@

Re: [FFmpeg-devel] [PATCHv2 1/5] ffmpeg: log failed av_write_trailer

2015-10-18 Thread Michael Niedermayer
On Sun, Oct 18, 2015 at 12:07:34PM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > ffmpeg.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/ffmpeg.c b/ffmpeg.c > index 36a68fb..252bc0d 100644 > --- a/ffmpeg.c > +++ b/ffmpeg.c > @@ -4104,7 +4104,9 @@

Re: [FFmpeg-devel] [PATCH] avutil/opt: display a better default value for int/int64 options

2015-10-18 Thread Clément Bœsch
On Sat, Oct 17, 2015 at 04:32:17PM +0200, Michael Niedermayer wrote: > On Sat, Oct 17, 2015 at 02:54:31PM +0200, Clément Bœsch wrote: > > Example: > > > > % ./ffmpeg -h encoder=aac > > -aac_coder E...A... Coding algorithm (from -1 to 3) > > (default twoloop) > > faac

Re: [FFmpeg-devel] [PATCH] avutil/opt: display a better default value for int/int64 options

2015-10-18 Thread Moritz Barsnick
On Sun, Oct 18, 2015 at 16:14:51 +0200, Clément Bœsch wrote: > On Sat, Oct 17, 2015 at 05:24:03PM +0200, Moritz Barsnick wrote: > > If it could also map the numbers ("from -1 to 3") to the enumerated > > values, it would make it a lot easier to understand: > > I'm not sure this is a good idea; it

[FFmpeg-devel] [PATCH] rtsp protocol : teardown packet not sent

2015-10-18 Thread Nicolas Adenis-Lamarre
The rtsp protocol requires the client to send a packet at the end of the connexion. FFmpeg basic network function check wether the user aborted the communication and don't send the packet in this case. So the protocol is not respected. This commit removes the check. An other possibility would have

Re: [FFmpeg-devel] [PATCH] avfilter/af_flanger: free frame on ENOMEM

2015-10-18 Thread Michael Niedermayer
On Thu, Oct 15, 2015 at 10:08:36AM -0500, Kyle Swanson wrote: > Signed-off-by: Kyle Swanson > --- > libavfilter/af_flanger.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No snowflake

Re: [FFmpeg-devel] [PATCH 2/6] avutil/ripemd: use EINVAL instead of -1 for the return code of av_ripemd_init()

2015-10-18 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 7:39 PM, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/ripemd.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/ripemd.c b/libavutil/ripemd.c > index 0084860..d247fb4 100644 > --- a/libavutil/ripemd.c >

Re: [FFmpeg-devel] [PATCH 1/6] avutil/sha512: use EINVAL instead of -1 for the return code of av_sha512_init()

2015-10-18 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 7:39 PM, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/sha512.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/sha512.c b/libavutil/sha512.c > index 66a864f..e2fc58a 100644 > --- a/libavutil/sha512.c >

Re: [FFmpeg-devel] [PATCH 3/6] avutil/aes: use EINVAL instead of -1 for the return code of av_aes_init()

2015-10-18 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 7:39 PM, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/aes.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/aes.c b/libavutil/aes.c > index 8d4..b59e7de 100644 > --- a/libavutil/aes.c > +++ b/libavu

Re: [FFmpeg-devel] [PATCH 4/6] avutil/twofish: use EINVAL instead of -1 for the return code of av_twofish_init()

2015-10-18 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 7:39 PM, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/twofish.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/twofish.c b/libavutil/twofish.c > index f735a1f..162069b 100644 > --- a/libavutil/twofish.

Re: [FFmpeg-devel] [PATCH 5/6] avutil/rc4: use EINVAL instead of -1 for the return code of av_rc4_init()

2015-10-18 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 7:39 PM, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/rc4.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/rc4.c b/libavutil/rc4.c > index 6bd702c..ffcb112 100644 > --- a/libavutil/rc4.c > +++ b/libavu

Re: [FFmpeg-devel] [PATCH 6/6] avutil/des: use EINVAL instead of -1 for the return code of av_des_init()

2015-10-18 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 7:52 PM, Ganesh Ajjanagadde wrote: > On Sat, Oct 17, 2015 at 7:49 PM, Ronald S. Bultje wrote: >> Hi, >> >> On Sat, Oct 17, 2015 at 7:40 PM, Ganesh Ajjanagadde >> wrote: >>> >>> On Sat, Oct 17, 2015 at 7:39 PM, Ganesh Ajjanagadde >>> wrote: >>> > Signed-off-by: Ganesh Ajj

Re: [FFmpeg-devel] [PATCH 2/3] avutil/sha: use EINVAL instead of -1 for the return code of av_sha_init()

2015-10-18 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 8:06 PM, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/sha.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/sha.c b/libavutil/sha.c > index 9963043..748bb9c 100644 > --- a/libavutil/sha.c > +++ b/libavu

Re: [FFmpeg-devel] [PATCH 1/3] avutil/cast5: use EINVAL instead of -1 for the return code of av_cast5_init()

2015-10-18 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 8:06 PM, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/cast5.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/cast5.c b/libavutil/cast5.c > index 98aa19d..a47697b 100644 > --- a/libavutil/cast5.c > +++

Re: [FFmpeg-devel] [PATCH 3/3] avutil/camellia: use EINVAL instead of -1 for the return code of av_camellia_init()

2015-10-18 Thread Ganesh Ajjanagadde
On Sun, Oct 18, 2015 at 2:21 PM, Michael Niedermayer wrote: > On Sun, Oct 18, 2015 at 11:58:55AM -0400, Ganesh Ajjanagadde wrote: >> On Sat, Oct 17, 2015 at 8:07 PM, Ganesh Ajjanagadde >> wrote: >> > Signed-off-by: Ganesh Ajjanagadde >> > --- >> > libavutil/camellia.c | 2 +- >> > 1 file change

[FFmpeg-devel] [PATCH] avformat/genh: Mark coef_splitted as av_unused

2015-10-18 Thread Michael Niedermayer
From: Michael Niedermayer This avoid "libavformat/genh.c:43:14: warning: variable coef_splitted set but not used" Fewer warnings makes it easier to see new and important warnings Signed-off-by: Michael Niedermayer --- libavformat/genh.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [FFmpeg-devel] [PATCH 1/2] avformat/vag: Remove unused variable pos

2015-10-18 Thread Michael Niedermayer
On Sun, Oct 18, 2015 at 12:40:56AM +0200, Paul B Mahol wrote: > On 10/17/15, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/vag.c |2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/libavformat/vag.c b/l

Re: [FFmpeg-devel] [PATCH 3/3] avutil/camellia: use EINVAL instead of -1 for the return code of av_camellia_init()

2015-10-18 Thread Michael Niedermayer
On Sun, Oct 18, 2015 at 11:58:55AM -0400, Ganesh Ajjanagadde wrote: > On Sat, Oct 17, 2015 at 8:07 PM, Ganesh Ajjanagadde > wrote: > > Signed-off-by: Ganesh Ajjanagadde > > --- > > libavutil/camellia.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavutil/camel

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Ganesh Ajjanagadde
On Sun, Oct 18, 2015 at 1:16 PM, Clément Bœsch wrote: > On Sun, Oct 18, 2015 at 11:12:03AM -0400, Ganesh Ajjanagadde wrote: >> On Sun, Oct 18, 2015 at 11:03 AM, Clément Bœsch wrote: >> > On Sun, Oct 18, 2015 at 10:47:52AM -0400, Ganesh Ajjanagadde wrote: >> > [...] >> >> diff --git a/libavformat/

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Clément Bœsch
On Sun, Oct 18, 2015 at 11:12:03AM -0400, Ganesh Ajjanagadde wrote: > On Sun, Oct 18, 2015 at 11:03 AM, Clément Bœsch wrote: > > On Sun, Oct 18, 2015 at 10:47:52AM -0400, Ganesh Ajjanagadde wrote: > > [...] > >> diff --git a/libavformat/subtitles.c b/libavformat/subtitles.c > >> index bb89766..16a

Re: [FFmpeg-devel] [PATCH 3/3] avutil/camellia: use EINVAL instead of -1 for the return code of av_camellia_init()

2015-10-18 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 8:07 PM, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/camellia.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/camellia.c b/libavutil/camellia.c > index 483eed2..f21ca12 100644 > --- a/libavutil/camel

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Ganesh Ajjanagadde
On Sun, Oct 18, 2015 at 11:45 AM, Carl Eugen Hoyos wrote: > Ganesh Ajjanagadde mit.edu> writes: > >> >> +if (q->sort == SUB_SORT_TS_POS) { >> >> +AV_QSORT(q->subs, q->nb_subs, AVPacket, cmp_pkt_sub_ts_pos); >> >> +} >> >> +else >> >> +AV_QSORT(q->subs, q->nb_subs, AVPa

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Ganesh Ajjanagadde
On Sun, Oct 18, 2015 at 11:38 AM, Henrik Gramner wrote: > On Sun, Oct 18, 2015 at 4:47 PM, Ganesh Ajjanagadde > wrote: >> +++ b/tests/checkasm/checkasm.c > [...] >> #include "libavutil/common.h" >> #include "libavutil/cpu.h" >> #include "libavutil/random_seed.h" >> +#include "libavutil/qsort.h

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Carl Eugen Hoyos
Ganesh Ajjanagadde mit.edu> writes: > >> +if (q->sort == SUB_SORT_TS_POS) { > >> +AV_QSORT(q->subs, q->nb_subs, AVPacket, cmp_pkt_sub_ts_pos); > >> +} > >> +else > >> +AV_QSORT(q->subs, q->nb_subs, AVPacket, cmp_pkt_sub_pos_ts); > >> + > > > > Weird style. > > There w

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Ganesh Ajjanagadde
On Sun, Oct 18, 2015 at 11:25 AM, Ganesh Ajjanagadde wrote: > On Sun, Oct 18, 2015 at 11:17 AM, wm4 wrote: >> On Sun, 18 Oct 2015 11:06:57 -0400 >> Ganesh Ajjanagadde wrote: >> >>> On Sun, Oct 18, 2015 at 11:01 AM, wm4 wrote: >>> > On Sun, 18 Oct 2015 10:47:52 -0400 >>> > Ganesh Ajjanagadde wr

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Henrik Gramner
On Sun, Oct 18, 2015 at 4:47 PM, Ganesh Ajjanagadde wrote: > +++ b/tests/checkasm/checkasm.c [...] > #include "libavutil/common.h" > #include "libavutil/cpu.h" > #include "libavutil/random_seed.h" > +#include "libavutil/qsort.h" Alphabetical order. _

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Ganesh Ajjanagadde
On Sun, Oct 18, 2015 at 11:17 AM, wm4 wrote: > On Sun, 18 Oct 2015 11:06:57 -0400 > Ganesh Ajjanagadde wrote: > >> On Sun, Oct 18, 2015 at 11:01 AM, wm4 wrote: >> > On Sun, 18 Oct 2015 10:47:52 -0400 >> > Ganesh Ajjanagadde wrote: >> > >> >> Commit e11e32686fdb21aded1ccf70202f1fffe87bb6a2 expla

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread wm4
On Sun, 18 Oct 2015 11:06:57 -0400 Ganesh Ajjanagadde wrote: > On Sun, Oct 18, 2015 at 11:01 AM, wm4 wrote: > > On Sun, 18 Oct 2015 10:47:52 -0400 > > Ganesh Ajjanagadde wrote: > > > >> Commit e11e32686fdb21aded1ccf70202f1fffe87bb6a2 explains why replacing > >> qsort with AV_QSORT yields perf

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Ganesh Ajjanagadde
On Sun, Oct 18, 2015 at 11:03 AM, Clément Bœsch wrote: > On Sun, Oct 18, 2015 at 10:47:52AM -0400, Ganesh Ajjanagadde wrote: > [...] >> diff --git a/libavformat/subtitles.c b/libavformat/subtitles.c >> index bb89766..16ab245 100644 >> --- a/libavformat/subtitles.c >> +++ b/libavformat/subtitles.c

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Ganesh Ajjanagadde
On Sun, Oct 18, 2015 at 11:01 AM, wm4 wrote: > On Sun, 18 Oct 2015 10:47:52 -0400 > Ganesh Ajjanagadde wrote: > >> Commit e11e32686fdb21aded1ccf70202f1fffe87bb6a2 explains why replacing >> qsort with AV_QSORT yields performance improvements. >> >> This replaces all existing uses of libc's qsort w

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Ganesh Ajjanagadde
On Sun, Oct 18, 2015 at 10:57 AM, Hendrik Leppkes wrote: > On Sun, Oct 18, 2015 at 4:47 PM, Ganesh Ajjanagadde > wrote: >> Commit e11e32686fdb21aded1ccf70202f1fffe87bb6a2 explains why replacing >> qsort with AV_QSORT yields performance improvements. >> >> This replaces all existing uses of libc's

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Clément Bœsch
On Sun, Oct 18, 2015 at 10:47:52AM -0400, Ganesh Ajjanagadde wrote: [...] > diff --git a/libavformat/subtitles.c b/libavformat/subtitles.c > index bb89766..16ab245 100644 > --- a/libavformat/subtitles.c > +++ b/libavformat/subtitles.c > @@ -23,6 +23,7 @@ > #include "avio_internal.h" > #include "l

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread wm4
On Sun, 18 Oct 2015 10:47:52 -0400 Ganesh Ajjanagadde wrote: > Commit e11e32686fdb21aded1ccf70202f1fffe87bb6a2 explains why replacing > qsort with AV_QSORT yields performance improvements. > > This replaces all existing uses of libc's qsort with AV_QSORT. > > Benchmarks deemed unnecessary due t

Re: [FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Hendrik Leppkes
On Sun, Oct 18, 2015 at 4:47 PM, Ganesh Ajjanagadde wrote: > Commit e11e32686fdb21aded1ccf70202f1fffe87bb6a2 explains why replacing > qsort with AV_QSORT yields performance improvements. > > This replaces all existing uses of libc's qsort with AV_QSORT. > > Benchmarks deemed unnecessary due to exi

Re: [FFmpeg-devel] [PATCH] dca_parser: don't overwrite the sample rate, it may not be correct

2015-10-18 Thread Hendrik Leppkes
On Wed, Sep 30, 2015 at 1:09 PM, Hendrik Leppkes wrote: > The parser only reads the dca core sample rate, which is limited to a > maximum of 48000 Hz, while X96 and HD extensions can increase the sample > rate up to 192000 Hz. > > This change prevents the parser and decoder fighting over the sampl

[FFmpeg-devel] [PATCH] all: replace qsort with AV_QSORT

2015-10-18 Thread Ganesh Ajjanagadde
Commit e11e32686fdb21aded1ccf70202f1fffe87bb6a2 explains why replacing qsort with AV_QSORT yields performance improvements. This replaces all existing uses of libc's qsort with AV_QSORT. Benchmarks deemed unnecessary due to existing claims about AV_QSORT. Tested with FATE. Signed-off-by: Ganesh

Re: [FFmpeg-devel] [PATCH]lavf/ingenient: Add a probe function

2015-10-18 Thread Michael Niedermayer
On Sun, Oct 18, 2015 at 02:12:16PM +0200, Carl Eugen Hoyos wrote: > Hi! > > In the past, the ingenient demuxer was the reason for many bug reports. > This has changed since mjpeg autodetection exists, but since ingenient > auto-detection is easily possible, it should be added. > Unfortunately, I

Re: [FFmpeg-devel] [PATCH] avutil/opt: display a better default value for int/int64 options

2015-10-18 Thread Clément Bœsch
On Sat, Oct 17, 2015 at 05:24:03PM +0200, Moritz Barsnick wrote: > On Sat, Oct 17, 2015 at 14:54:31 +0200, Clément Bœsch wrote: > > % ./ffmpeg -h encoder=aac > > -aac_coder E...A... Coding algorithm (from -1 to 3) > > (default twoloop) > > faac E...A.

Re: [FFmpeg-devel] [PATCH] avcodec/bitstream: replace qsort with AV_QSORT

2015-10-18 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 10:25 PM, Michael Niedermayer wrote: > On Sat, Oct 17, 2015 at 09:30:01PM -0400, Ganesh Ajjanagadde wrote: >> Commit 3a0a2f33a6c955823fa4fb12c0b49cd29a496659 claims large performance >> advantages for AV_QSORT over libc's qsort. The reason is that I suspect >> that libc's q

Re: [FFmpeg-devel] [PATCH]Remove --disable-avutil and sanitize --disable-all

2015-10-18 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > >          --disable-all) > >              map 'eval unset \${$(toupper ${v%s})_LIST}' $COMPONENT_LIST > >              disable $LIBRARY_LIST $PROGRAM_LIST doc > > +            enable avutil > > I believe this simplifies using --disable-all, so I > will commi

Re: [FFmpeg-devel] [PATCH] avformat: add msf demuxer

2015-10-18 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > +static int msf_probe(AVProbeData *p) > +{ > +if (memcmp(p->buf, "MSF", 3)) > +return 0; > + > +return AVPROBE_SCORE_MAX; Should be less than EXTENSION / 2 but you can check codec and channels. Carl Eugen __

[FFmpeg-devel] [PATCH]lavf/ingenient: Add a probe function

2015-10-18 Thread Carl Eugen Hoyos
Hi! In the past, the ingenient demuxer was the reason for many bug reports. This has changed since mjpeg autodetection exists, but since ingenient auto-detection is easily possible, it should be added. Unfortunately, I cannot test attached patch, please review wrt http://multimedia.cx/ingenient.

Re: [FFmpeg-devel] [PATCH] Stream specifier enhancement

2015-10-18 Thread Bodecs Bela
Dear Marton Balint, see may comments below. 2015.10.18. 1:10 keltezéssel, Marton Balint írta: On Thu, 15 Oct 2015, Bodecs Bela wrote: [...] My enhancement does not alter the current behaviour in any way. Are you sure? What if somebody matched a semicolon, or a string which contained a se

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mpegvideo_enc.c: Fix encoding videos with less frames than the delay of the encoder.

2015-10-18 Thread Michael Niedermayer
On Fri, Oct 16, 2015 at 10:42:33AM +0200, Alexis Ballier wrote: > When the encoder is fed with less frames than its delay, the picture list > looks like { NULL, NULL, ..., frame, frame, frame }. When flushing the > encoder (input frame == NULL), we need to ensure the picture list is shifted > en

[FFmpeg-devel] [PATCHv2 1/5] ffmpeg: log failed av_write_trailer

2015-10-18 Thread Marton Balint
Signed-off-by: Marton Balint --- ffmpeg.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ffmpeg.c b/ffmpeg.c index 36a68fb..252bc0d 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -4104,7 +4104,9 @@ static int transcode(void) /* write the trailer if needed and close file */

Re: [FFmpeg-devel] [PATCH 2/3] lavc/mjpegdec: honor skip_frame option

2015-10-18 Thread Matthieu Bouron
On Sat, Oct 17, 2015 at 10:34:23PM +0200, Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > libavcodec/mjpegdec.c | 20 > 1 file changed, 20 insertions(+) > > diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c > index 1a86b7b..8a90e94 100644 > --- a/libavcode