Re: [FFmpeg-devel] [PATCH] aacenc_utils: unroll loops to allow compiler to use SIMD.

On 08.03.2016, at 04:50, Ganesh Ajjanagadde wrote: > On Mon, Mar 7, 2016 at 2:54 AM, Reimar Döffinger > wrote: >> >>> Can you be more specific, and are you sure about this? >> >> Just run your favourite performance analysis tool and you'll see. >>

[FFmpeg-devel] [PATCH] Add tests for functions in hash.c

--- libavutil/Makefile | 1 + libavutil/hash.c | 42 ++ tests/fate/libavutil.mak | 4 tests/ref/fate/hash | 45 + 4 files changed, 92 insertions(+) create mode 100644 tests/ref/fate/hash

Re: [FFmpeg-devel] [PATCH] lavf/avienc: New AVOption write_keyframe_palette

On 03/08/2016 02:59 AM, Mats Peterson wrote: On 03/07/2016 07:48 PM, Mats Peterson wrote: On 03/07/2016 07:43 PM, Mats Peterson wrote: Here's a somewhat experimental patch with a new AVOption write_keyframe_palette that allows you to write the palette to every keyframe. It is disabled by

Re: [FFmpeg-devel] [PATCH 1/3] tests/gapless: add gapless aac tests

On Tue, Mar 08, 2016 at 01:49:43AM +0100, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > tests/fate-run.sh | 30 + > tests/fate/gapless.mak| 20 - > tests/ref/fate/gapless2-ipod-aac1 | 88 >

Re: [FFmpeg-devel] [PATCH] aacenc_utils: unroll loops to allow compiler to use SIMD.

On Mon, Mar 7, 2016 at 2:54 AM, Reimar Döffinger wrote: > On 07.03.2016, at 04:04, Ganesh Ajjanagadde wrote: >> On Sun, Mar 6, 2016 at 1:43 PM, Reimar Döffinger >> wrote: >>> On Sun, Mar 06, 2016 at 07:35:58PM +0100, Reimar

[FFmpeg-devel] [PATCH] lavc/aacenc_utils: replace powf(x, y) by expf(logf(x), y)

This is ~2x faster for y not an integer on Haswell+GCC, and should generally be faster due to the fact that anyway powf essentially does this under the hood. Note that there are some accuracy differences, that should generally be negligible. In particular, FATE still passes on this platform.

Re: [FFmpeg-devel] [PATCH] lavf/avienc: New AVOption write_keyframe_palette

On 03/07/2016 07:48 PM, Mats Peterson wrote: On 03/07/2016 07:43 PM, Mats Peterson wrote: Here's a somewhat experimental patch with a new AVOption write_keyframe_palette that allows you to write the palette to every keyframe. It is disabled by default, but it is needed for any file that

Re: [FFmpeg-devel] [PATCH] lavc/psymodel: check for av_malloc failure

On Thu, Mar 3, 2016 at 10:06 PM, Ganesh Ajjanagadde wrote: > No idea why in commit 01ecb7172b684f1c4b3e748f95c5a9a494ca36ec the > checks were removed; this can lead to NULL pointer dereferences. This > effectively reverts that portion of the commit. > > Signed-off-by: Ganesh

[FFmpeg-devel] [PATCH 3/3] avformat/mov: read start_pad from edit list start time if codec is aac

Related to ticket #2324, #2325. Stream duration still need to be fixed... Signed-off-by: Marton Balint --- libavformat/mov.c | 3 +++ tests/ref/fate/gapless2-ipod-aac1 | 36 ++-- tests/ref/fate/gapless2-ipod-aac2 | 36

[FFmpeg-devel] [PATCH 2/3] avformat/utils: increase detected start_time with skip_samples

Signed-off-by: Marton Balint --- libavformat/utils.c | 10 -- tests/ref/fate/gapless2-ipod-aac1 | 74 +++ tests/ref/fate/gapless2-ipod-aac2 | 74 +++ 3 files changed, 82 insertions(+), 76

[FFmpeg-devel] [PATCH 1/3] tests/gapless: add gapless aac tests

Signed-off-by: Marton Balint --- tests/fate-run.sh | 30 + tests/fate/gapless.mak| 20 - tests/ref/fate/gapless2-ipod-aac1 | 88 +++ tests/ref/fate/gapless2-ipod-aac2 | 88

[FFmpeg-devel] [PATCH] Add encoder stats to the NVENC codec.

--- libavcodec/nvenc.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index a3b02fa..5489610 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -1167,6 +1167,8 @@ static int

[FFmpeg-devel] [PATCH] lavf/matroskaenc.c: add early support for colour elements

Adding early support for a subset of the proposed colour elements according to the latest version of spec: https://mailarchive.ietf.org/arch/search/?email_list=cellar=1=hIKLhMdgTMTEwUTeA4ct38h0tmE Like matroskadec, I've left out elements for pix_fmt related things as there still seems to be some

Re: [FFmpeg-devel] [PATCH] avcodec: try to document timebase and framerate a bit more

On Sat, Mar 05, 2016 at 07:55:47PM +0100, Reimar Döffinger wrote: > On Sat, Mar 05, 2016 at 06:56:30PM +0100, Michael Niedermayer wrote: > > On Sat, Mar 05, 2016 at 06:01:11PM +0100, Reimar Döffinger wrote: > > > On Sat, Mar 05, 2016 at 01:41:32PM +0100, Michael Niedermayer wrote: > > > > + *

Re: [FFmpeg-devel] [PATCH] lavf/utils: has_decode_delay_been_guessed - return true if nb_decoded_frames >= 2, when there are no B-frames.

On Mon, Mar 07, 2016 at 11:10:16AM -0800, Sasi Inguva wrote: > Fixes DTS detection for any H264 file of less than 7 frames length. Without > this DTS is N/A. > > Signed-off-by: Sasi Inguva > --- > libavformat/utils.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-)

Re: [FFmpeg-devel] [PATCH] Opus in MP4 support

On 3/7/2016 1:29 AM, Matthew Gregan wrote: > Hi, > > The attached patch adds basic mux/demux support for the Opus audio codec in > MP4. > > Mozilla have expressed interest in shipping support for this in: > https://groups.google.com/d/msg/mozilla.dev.platform/mdDZ-nBr_jM/MaLi2BDOAgAJ > >

Re: [FFmpeg-devel] [PATCH] lavf/utils: has_decode_delay_been_guessed - always return true if there are no B-frames at all.

Please ignore this patch as the commit message is incorrect. Sent a new patch with corrected commit message. On Mon, Mar 7, 2016 at 11:02 AM, Sasi Inguva wrote: > Fixes DTS detection for any H264 file of less than 7 frames length. > Without this DTS is N/A. > > Signed-off-by:

[FFmpeg-devel] [PATCH] lavf/utils: has_decode_delay_been_guessed - return true if nb_decoded_frames >= 2, when there are no B-frames.

Fixes DTS detection for any H264 file of less than 7 frames length. Without this DTS is N/A. Signed-off-by: Sasi Inguva --- libavformat/utils.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index

[FFmpeg-devel] [PATCH] lavf/utils: has_decode_delay_been_guessed - always return true if there are no B-frames at all.

Fixes DTS detection for any H264 file of less than 7 frames length. Without this DTS is N/A. Signed-off-by: Sasi Inguva --- libavformat/utils.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index

Re: [FFmpeg-devel] [PATCH] lavf/avienc: New AVOption write_keyframe_palette

On 03/07/2016 07:43 PM, Mats Peterson wrote: Here's a somewhat experimental patch with a new AVOption write_keyframe_palette that allows you to write the palette to every keyframe. It is disabled by default, but it is needed for any file that contains palette changes. It is needed in order to

[FFmpeg-devel] [PATCH] lavf/avienc: New AVOption write_keyframe_palette

Here's a somewhat experimental patch with a new AVOption write_keyframe_palette that allows you to write the palette to every keyframe. It is disabled by default, but it is needed for any file that contains palette changes. This patch has to be applied after the previous 2-part patch set.

Re: [FFmpeg-devel] [PATCH] Add encoder stats to the NVENC codec.

> --- > libavcodec/nvenc.c | 18 +++--- > 1 file changed, 11 insertions(+), 7 deletions(-) > > diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c > index a3b02fa..67232c7 100644 > --- a/libavcodec/nvenc.c > +++ b/libavcodec/nvenc.c > @@ -1208,31 +1208,35 @@ static int

Re: [FFmpeg-devel] [PATCH] Add encoder stats to the NVENC codec.

On 2016-03-02 15:21, Lucas Cooper wrote: --- libavcodec/nvenc.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index a3b02fa..67232c7 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -1208,31 +1208,35 @@

Re: [FFmpeg-devel] [PATCH] Add encoder stats to the NVENC codec.

Ping. On 2 March 2016 at 15:21, Lucas Cooper wrote: > --- > libavcodec/nvenc.c | 18 +++--- > 1 file changed, 11 insertions(+), 7 deletions(-) > > diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c > index a3b02fa..67232c7 100644 > --- a/libavcodec/nvenc.c >

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette handling code only concerns AV_PIX_FMT_PAL8

Mats Peterson skrev: (7 mars 2016 18:04:34 CET) >Mats Peterson skrev: (7 mars 2016 >17:49:41 CET) >>Mats Peterson skrev: (7 mars 2016 >>15:00:29 CET) >>>Forget the separate patch posted

[FFmpeg-devel] [PATCH] lavc/libopenh264enc: document cabac option, and change its type to boolean

--- doc/encoders.texi | 3 +++ libavcodec/libopenh264enc.c | 2 +- 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/doc/encoders.texi b/doc/encoders.texi index dfab007..76dd53c 100644 --- a/doc/encoders.texi +++ b/doc/encoders.texi @@ -1425,6 +1425,9 @@ Set number of MBs

[FFmpeg-devel] [PATCH] lavc/libopenh264enc: update to openh264 1.6

In particular, the slice mode API was changed in commit: commit 33c378f7b791310e4cb64b53e2bb8f3f3bded105 Author: sijchen Date: Tue Nov 10 09:50:06 2015 -0800 change API for slicing part for easier usage (the UseLoadBalancing flag is still under working) This fixes

[FFmpeg-devel] [PATCH] lavc/libopenh264enc: drop OPENH264_VER_AT_LEAST usage

Now FFmpeg requires openh264 at least at version 1.6. --- libavcodec/libopenh264enc.c | 8 1 file changed, 8 deletions(-) diff --git a/libavcodec/libopenh264enc.c b/libavcodec/libopenh264enc.c index 0c1b1b8..ec78b12 100644 --- a/libavcodec/libopenh264enc.c +++

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette handling code only concerns AV_PIX_FMT_PAL8

Mats Peterson skrev: (7 mars 2016 17:49:41 CET) >Mats Peterson skrev: (7 mars 2016 >15:00:29 CET) >>Forget the separate patch posted before, and my previous experimental >>variants of your patch, Michael. And if we're going to

Re: [FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette handling code only concerns AV_PIX_FMT_PAL8

Mats Peterson skrev: (7 mars 2016 15:00:29 CET) >Forget the separate patch posted before, and my previous experimental >variants of your patch, Michael. And if we're going to use keyframe >xxpc >chunks, we'll have to invent something really good. Here is some

Re: [FFmpeg-devel] [PATCH] Opus in MP4 support

On 3/7/2016 4:29 AM, Matthew Gregan wrote: > Hi, > > The attached patch adds basic mux/demux support for the Opus audio codec in > MP4. > > Mozilla have expressed interest in shipping support for this in: > https://groups.google.com/d/msg/mozilla.dev.platform/mdDZ-nBr_jM/MaLi2BDOAgAJ Hi, Why

[FFmpeg-devel] [PATCH] configure: build fix for P5600 with mips code restructuring

From: Shivraj Patil Signed-off-by: Shivraj Patil --- configure | 312 +++-- 1 file changed, 117 insertions(+), 195 deletions(-) diff --git a/configure b/configure index

[FFmpeg-devel] [PATCH 2/2] lavf/avienc: New static function avi_write_xxpc()

-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From 0b38769f175ceafff8150a97be06b1a2fbf0bc3d Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Mon, 7 Mar 2016 14:54:26 +0100 Subject: [PATCH 2/2] lavf/avienc: New static function avi_write_xxpc() ---

[FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette handling code only concerns AV_PIX_FMT_PAL8

Forget the separate patch posted before, and my previous experimental variants of your patch, Michael. And if we're going to use keyframe xxpc chunks, we'll have to invent something really good. Here is some useful stuff added in the meantime. Mats -- Mats Peterson

Re: [FFmpeg-devel] [PATCH] lavf/img2dec: disable parsing if frame_size is specified

Matthieu Bouron gmail.com> writes: > else { > s->is_pipe = 1; > -st->need_parsing = AVSTREAM_PARSE_FULL; > +st->need_parsing = s->frame_size > 0 ? AVSTREAM_PARSE_NONE > : AVSTREAM_PARSE_FULL; Makes sense imo. Carl Eugen

[FFmpeg-devel] [PATCH] lavf/img2dec: disable parsing if frame_size is specified

From: Matthieu Bouron --- Hello, The following patch disable parsing if the frame_size option is specified. The main purpose here is to disable the use of parsers (which have a huge performance cost on embedded platforms) for single images when their size is

Re: [FFmpeg-devel] [PATCH] lavc/mjpegdec: avoid unneeded allocation if the frame is to be skipped

On Tue, Mar 01, 2016 at 08:53:33PM +0100, Paul B Mahol wrote: > On 3/1/16, Matthieu Bouron wrote: > > From: Matthieu Bouron > > > > --- > > libavcodec/mjpegdec.c | 7 +++ > > 1 file changed, 7 insertions(+) > > > > probbably ok

Re: [FFmpeg-devel] [PATCH]lavfi/extractplanes: Fix in_pixfmts

Paul B Mahol gmail.com> writes: > OK, then push. Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2 5/9] lavc/ccaption_dec: implement tab offset commands

On Tue, Jan 12, 2016 at 05:42:56PM -0800, Aman Gupta wrote: > From: Aman Gupta > > --- > libavcodec/ccaption_dec.c | 5 + > 1 file changed, 5 insertions(+) do you have a testcase / sample that shows the problem this fixes ? i think this patch wasnt applied yet [...] --

Re: [FFmpeg-devel] [PATCH 2/2] lavc: add h264 mediacodec decoder

On Thu, Mar 03, 2016 at 02:03:01PM +0100, Matthieu Bouron wrote: [...] > > Patch updated with the following differences: > * ff_set_dimensions return code is now used > * add missing exception when trying to call the MediaCodec object > constructor > * remove leftover avctx_internal

Re: [FFmpeg-devel] [PATCH]lavfi/extractplanes: Fix in_pixfmts

On 3/7/16, Carl Eugen Hoyos wrote: > Paul B Mahol gmail.com> writes: > >> > LGTM >> >> Wait, have you checked this doesn't break previously working cases? > > No. > > What cases could this be? > After the patch, for le original formats, 16bit be input > formats are not

Re: [FFmpeg-devel] [PATCH]lavfi/extractplanes: Fix in_pixfmts

Paul B Mahol gmail.com> writes: > > LGTM > > Wait, have you checked this doesn't break previously working cases? No. What cases could this be? After the patch, for le original formats, 16bit be input formats are not supported anymore, for be original formats, 16bit le input formats are not

Re: [FFmpeg-devel] [PATCH 1/2] lavc: add JNI support

On Thu, Mar 03, 2016 at 01:56:16PM +0100, Matthieu Bouron wrote: [...] > > New patch attached with the following differences: > * added myself as a maintainer of jni* and ffjni* > Pushed. [...] ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH]lavfi/extractplanes: Fix in_pixfmts

On 3/7/16, Paul B Mahol wrote: > On 3/5/16, Carl Eugen Hoyos wrote: >> Hi! >> >> I see the following issue here with extractplanes: >> If the input file has a 16bit little endian pix_fmt not supported by >> extractplanes (gbra12le), the filter system has to

Re: [FFmpeg-devel] [PATCH]lavfi/extractplanes: Fix in_pixfmts

On 3/5/16, Carl Eugen Hoyos wrote: > Hi! > > I see the following issue here with extractplanes: > If the input file has a 16bit little endian pix_fmt not supported by > extractplanes (gbra12le), the filter system has to choose a compatible > pix_fmt supported by the filter and

Re: [FFmpeg-devel] [PATCH] lavf/avienc: Palette handling code only concerns AV_PIX_FMT_PAL8

On 03/07/2016 09:17 AM, Mats Peterson wrote: On 03/07/2016 08:59 AM, Mats Peterson wrote: ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel Interestingly enough, on stream copy of 1 bpp raw

Re: [FFmpeg-devel] [PATCH 0/2] New versioning (was: version.sh: Always use latest tag for generated version number)

On Sun, Mar 6, 2016 at 2:50 AM, Timothy Gu wrote: > This patch set implements the versioning algorithm I recently proposed. > The first patch addresses an already existing problem with version.sh. > The second is the one actually implementing the change. > > To Michael and

Re: [FFmpeg-devel] [PATCH 0/2] New versioning (was: version.sh: Always use latest tag for generated version number)

Timothy Gu gmail.com> writes: > Timothy Gu (2): > version.sh: Fix Git directory > version.sh: Add next release to the version Without these patches applied, I get the following: $ ./version.sh N-78949-g6f5048f With your patches applied, I see: $ ./version.sh cat: /RELEASE: No such file or

Re: [FFmpeg-devel] [PATCH]lavfi/extractplanes: Fix in_pixfmts

Carl Eugen Hoyos ag.or.at> writes: > If the input file has a 16bit little endian pix_fmt not supported > by extractplanes (gbra12le), the filter system has to choose a > compatible pix_fmt supported by the filter and chooses rgba64be > here. The extractplanes filter returns gray16le as output

Re: [FFmpeg-devel] [PATCH] lavf/avienc: Palette handling code only concerns AV_PIX_FMT_PAL8

On 03/07/2016 08:59 AM, Mats Peterson wrote: ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel Interestingly enough, on stream copy of 1 bpp raw video, the pixel format will be AV_PIX_FMT_PAL8,

[FFmpeg-devel] [PATCH] lavf/avienc: Palette handling code only concerns AV_PIX_FMT_PAL8

-- Mats Peterson http://matsp888.no-ip.org/~mats/ >From ceaee5adb51c68bd35794f6c6165cd8c3f3b6619 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Mon, 7 Mar 2016 08:58:30 +0100 Subject: [PATCH] lavf/avienc: Palette handling code only concerns AV_PIX_FMT_PAL8 ---