Re: [FFmpeg-devel] [PATCH] avcodec/utils: initialize delay in avcodec_parameters_to_context()

2016-06-04 Thread Ronald S. Bultje
Hi, On Sat, Jun 4, 2016 at 1:09 PM, Michael Niedermayer wrote: > On Sat, Jun 04, 2016 at 09:16:09AM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Sat, Jun 4, 2016 at 7:27 AM, Michael Niedermayer > > > wrote: > > > > > On Sat, Jun 04, 20

Re: [FFmpeg-devel] [PATCH] avcodec/utils: initialize delay in avcodec_parameters_to_context()

2016-06-04 Thread Ronald S. Bultje
Hi, On Sat, Jun 4, 2016 at 7:27 AM, Michael Niedermayer wrote: > On Sat, Jun 04, 2016 at 12:15:50PM +0200, Hendrik Leppkes wrote: > > On Sat, Jun 4, 2016 at 12:09 PM, Michael Niedermayer > > wrote: > > > On Sat, Jun 04, 2016 at 09:47:47AM +0200, Hendrik Leppkes wrote: > > >> On Sat, Jun 4, 2016

Re: [FFmpeg-devel] [PATCH 2/2] avformat/rawdec: Fix avg_framerate for h264

2016-06-03 Thread Ronald S. Bultje
Hi, On Fri, Jun 3, 2016 at 6:12 PM, Michael Niedermayer wrote: > The framerate is 25 which is a fixed default and is wrong undo the 1 line > change which caused this regression > Fixes Ticket 5444 > > Signed-off-by: Michael Niedermayer > --- > libavformat/rawdec.c |2 +- > 1 file changed,

Re: [FFmpeg-devel] [PATCH] rtpenc: packetizer for VP9 RTP payload format (draft v2)

2016-06-02 Thread Ronald S. Bultje
Hi, On Thu, Jun 2, 2016 at 12:33 PM, Thomas Volkert wrote: > Hi, > > On 30.05.2016 17:43, Ronald S. Bultje wrote: > >> Hi, >> >> On Mon, May 30, 2016 at 10:41 AM, Thomas Volkert wrote: >> >> From: Thomas Volkert >>> >>> --- >

Re: [FFmpeg-devel] [PATCH] Use int32_t instead of int for gaintable in hdcd filter

2016-06-01 Thread Ronald S. Bultje
Hi, On Wed, Jun 1, 2016 at 7:47 PM, Michael Niedermayer wrote: > On Thu, Jun 02, 2016 at 12:01:50AM +0200, Benjamin St wrote: > > > > > af_hdcd.c |2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > b56f3e60dbd8c688ad37cc5dceb5378f68855622 > 0001-Use-int32_t-instead-of-int-for-gaint

Re: [FFmpeg-devel] [PATCH] rtpenc: packetizer for VP9 RTP payload format (draft v2)

2016-05-30 Thread Ronald S. Bultje
Hi, On Mon, May 30, 2016 at 10:41 AM, Thomas Volkert wrote: > From: Thomas Volkert > > --- > libavformat/Makefile | 1 + > libavformat/rtpenc.c | 14 + > libavformat/rtpenc.h | 1 + > libavformat/rtpenc_vp9.c | 54 > >

Re: [FFmpeg-devel] AVClass & AVOption [VOTE]

2016-05-29 Thread Ronald S. Bultje
Hi Thilo, On Sun, May 29, 2016 at 8:59 PM, Thilo Borgmann wrote: > Hi, > > > [...] > >> > >> The vote will end 1 week from now, simple 50% majority (Yes vs no) > >> I dont really know who should be eligible for voting, so i suggest > >> everyone from the vote comittee but iam

Re: [FFmpeg-devel] AVClass & AVOption [VOTE]

2016-05-29 Thread Ronald S. Bultje
Hi, On May 29, 2016 4:59 PM, "James Almer" wrote: > > On 5/29/2016 5:45 PM, Ronald S. Bultje wrote: > > Hi, > > > > On Sun, May 29, 2016 at 3:13 PM, Marton Balint wrote: > > > >> > >> > >> On Sun, 29 May 2016, Michael Nied

Re: [FFmpeg-devel] AVClass & AVOption [VOTE]

2016-05-29 Thread Ronald S. Bultje
Hi, On May 29, 2016 5:01 PM, "Marton Balint" wrote: > > > > On Sun, 29 May 2016, James Almer wrote: > >> On 5/29/2016 5:45 PM, Ronald S. Bultje wrote: >>> >>> Hi, >>> >>> On Sun, May 29, 2016 at 3:13 PM, Marton Balint wrote

Re: [FFmpeg-devel] AVClass & AVOption [VOTE]

2016-05-29 Thread Ronald S. Bultje
Hi, On Sun, May 29, 2016 at 3:13 PM, Marton Balint wrote: > > > On Sun, 29 May 2016, Michael Niedermayer wrote: > > Hi >> >> It was suggested in the IRC meeting today that i start a vote to >> resolve if AVClass & AVOption should be added to AVCodecParameters >> This question needs to be awnsere

Re: [FFmpeg-devel] AVClass & AVOption [VOTE]

2016-05-29 Thread Ronald S. Bultje
Hi, On Sun, May 29, 2016 at 3:25 PM, Thilo Borgmann wrote: > Am 29.05.16 um 01:32 schrieb Michael Niedermayer: > > Hi > > > > It was suggested in the IRC meeting today that i start a vote to > > resolve if AVClass & AVOption should be added to AVCodecParameters > > This question needs to be awns

Re: [FFmpeg-devel] AVClass & AVOption [VOTE]

2016-05-29 Thread Ronald S. Bultje
Hi, On Sat, May 28, 2016 at 7:32 PM, Michael Niedermayer wrote: > It was suggested in the IRC meeting today that i start a vote to > resolve if AVClass & AVOption should be added to AVCodecParameters > This question needs to be awnsered before the next release because > the ABI would be broken i

Re: [FFmpeg-devel] IRC meeting

2016-05-27 Thread Ronald S. Bultje
Hi, On Thu, May 26, 2016 at 7:00 PM, James Almer wrote: > On 5/26/2016 7:56 PM, Hendrik Leppkes wrote: > > On Thu, May 26, 2016 at 10:48 PM, Paul B Mahol wrote: > >> On 5/26/16, James Almer wrote: > >>> On 5/18/2016 5:33 PM, Paul B Mahol wrote: > Hi, > > I want to propose to hav

Re: [FFmpeg-devel] [PATCH 1/2] lavf/mp3dec: pass Xing gapless metadata to AVCodecParameters

2016-05-25 Thread Ronald S. Bultje
Hi, On Wed, May 25, 2016 at 1:24 PM, James Almer wrote: > On 5/25/2016 1:56 PM, Jon Toohill wrote: > > --- > > libavformat/mp3dec.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/libavformat/mp3dec.c b/libavformat/mp3dec.c > > index 3725d67..192f5ef 100644 > > --- a/libavforma

Re: [FFmpeg-devel] [PATCH] fix few compiler warnings

2016-05-24 Thread Ronald S. Bultje
Hi, On Tue, May 24, 2016 at 2:00 PM, Michael Niedermayer wrote: > On Mon, May 23, 2016 at 03:06:35PM +0200, Hendrik Leppkes wrote: > > On Mon, May 23, 2016 at 1:59 PM, Michael Niedermayer > > wrote: > > > On Mon, May 23, 2016 at 07:24:23AM -0400, Ronald S.

Re: [FFmpeg-devel] [PATCH] fix few compiler warnings

2016-05-23 Thread Ronald S. Bultje
Hi, On Sun, May 22, 2016 at 11:39 PM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Sun, May 22, 2016 at 01:51:05AM +, Davinder Singh wrote: > > On Sun, May 22, 2016 at 2:09 AM Michael Niedermayer > > > wrote: > > > > > On Sat, May 21, 2016 at 02:21:17PM +, Davinder Singh wro

Re: [FFmpeg-devel] Prores encoding optmizations

2016-05-23 Thread Ronald S. Bultje
Hi Havard, On Mon, May 23, 2016 at 6:36 AM, Håvard Espeland wrote: > Hi guys, > > We have been working on Prores Anatoliy optimizations to get the speed up > on an embedded x86 platform. Fdct (10bit), scaling and encoding of code > words have been optimized with AVX2 instructions, and the perfor

Re: [FFmpeg-devel] [PATCH] doc/developer.texi: Add a code of conduct

2016-05-21 Thread Ronald S. Bultje
Hi, On Sat, May 21, 2016 at 11:32 AM, James Almer wrote: > On 5/18/2016 3:40 PM, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > doc/developer.texi | 29 + > > 1 file changed, 29 insertions(+) > > > > diff --git a/doc/developer.texi b

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-19 Thread Ronald S. Bultje
Hi, On Thu, May 19, 2016 at 8:14 AM, Michael Niedermayer wrote: > On Thu, May 19, 2016 at 12:08:25PM +0200, Nicolas George wrote: > > Le nonidi 29 floréal, an CCXXIV, Michael Niedermayer a écrit : > > > there where and are accessor functions: > > > git grep MAKE_ACCESSORS > > > > Yes, and they s

Re: [FFmpeg-devel] [PATCH]lavf/rawenc: Add a G.729 muxer

2016-05-18 Thread Ronald S. Bultje
Hi, On Wed, May 18, 2016 at 10:25 AM, compn wrote: > On Wed, 18 May 2016 16:12:40 +0200 > Paul B Mahol wrote: > > > Dana 18. 5. 2016. 16:05 osoba "compn" napisala je: > > > > > > On Wed, 18 May 2016 14:10:57 +0200 > > > Paul B Mahol wrote: > > > > > > > On 5/18/16, Carl Eugen Hoyos wrote: >

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-17 Thread Ronald S. Bultje
Hi, On Tue, May 17, 2016 at 2:29 PM, Hendrik Leppkes wrote: > On Tue, May 17, 2016 at 7:46 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Tue, May 17, 2016 at 1:24 PM, Hendrik Leppkes > > wrote: > > > >> On Tue, May 17, 2016 at 7:2

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-17 Thread Ronald S. Bultje
Hi, On Tue, May 17, 2016 at 1:24 PM, Hendrik Leppkes wrote: > On Tue, May 17, 2016 at 7:22 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Tue, May 17, 2016 at 12:34 PM, Hendrik Leppkes > > wrote: > > > >> On Tue, May 17, 2016 at 6:2

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-17 Thread Ronald S. Bultje
Hi, On Tue, May 17, 2016 at 12:34 PM, Hendrik Leppkes wrote: > On Tue, May 17, 2016 at 6:26 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Tue, May 17, 2016 at 11:18 AM, Michael Niedermayer < > > mich...@niedermayer.cc> wrote: > > > &

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-17 Thread Ronald S. Bultje
Hi, On Tue, May 17, 2016 at 11:18 AM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > AVCodecContext contains all the fields that AVCodecParameters has. > Examples are width, height, bitrate, profile, sample_rate, channels > > In AVCodecContext these fields are accessible through AVOptions

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-17 Thread Ronald S. Bultje
Hi, On Tue, May 17, 2016 at 10:04 AM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Tue, May 17, 2016 at 06:49:44AM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Tue, May 17, 2016 at 6:07 AM, Michael Niedermayer > > > wrote: > > > &g

Re: [FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

2016-05-17 Thread Ronald S. Bultje
Hi, On Tue, May 17, 2016 at 6:07 AM, Michael Niedermayer wrote: > Most of the major public structures contain an AVClass. What are you hoping to accomplish with this struct that can currently not be done because it doesn't have an AVClass on top? I haven't seen this question answered anywhere

Re: [FFmpeg-devel] [PATCH] ffmpeg: copy color properties in case of -c:v copy.

2016-05-10 Thread Ronald S. Bultje
Hi, On Tue, May 3, 2016 at 10:21 PM, Michael Niedermayer wrote: > On Tue, May 03, 2016 at 08:58:07PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Tue, May 3, 2016 at 8:29 PM, Michael Niedermayer > > > wrote: > > > > > On Tue, May 03, 201

Re: [FFmpeg-devel] [PATCH] fate: add matroska-remux test.

2016-05-10 Thread Ronald S. Bultje
Hi, On Fri, May 6, 2016 at 6:33 PM, Michael Niedermayer wrote: > On Fri, May 06, 2016 at 09:27:20AM -0400, Ronald S. Bultje wrote: > > This tests automatic insertion of the vp9_superframe BSF as well as > > ensuring that the colorspace properties in the video header can be &g

Re: [FFmpeg-devel] [PATCH] vf_colorspace: make whitepoint adaptation mode configurable.

2016-05-10 Thread Ronald S. Bultje
Hi, On Tue, May 10, 2016 at 6:47 AM, Paul B Mahol wrote: > On 5/6/16, Ronald S. Bultje wrote: > > Also add von kries whitepoint adaptation, and add 'identity' to turn > > whitepoint adaptation off. > > --- > > doc/filters.texi| 15 +++

Re: [FFmpeg-devel] [PATCH] vf_colorspace: make whitepoint adaptation mode configurable.

2016-05-10 Thread Ronald S. Bultje
Hi, On Mon, May 9, 2016 at 8:36 AM, Ronald S. Bultje wrote: > Hi, > > On Fri, May 6, 2016 at 9:40 AM, Ronald S. Bultje > wrote: > >> Also add von kries whitepoint adaptation, and add 'identity' to turn >> whitepoint adaptation off. >&g

Re: [FFmpeg-devel] [PATCH] vf_colorspace: make whitepoint adaptation mode configurable.

2016-05-09 Thread Ronald S. Bultje
Hi, On Fri, May 6, 2016 at 9:40 AM, Ronald S. Bultje wrote: > Also add von kries whitepoint adaptation, and add 'identity' to turn > whitepoint adaptation off. > --- > doc/filters.texi| 15 +++ > libavfi

Re: [FFmpeg-devel] [PATCH 2/2] configure: Enable GCC vectorization on ≥4.9

2016-05-08 Thread Ronald S. Bultje
Hi, On Sun, May 8, 2016 at 1:55 PM, James Almer wrote: > On 5/8/2016 10:44 AM, Ronald S. Bultje wrote: > > Hi, > > > > On Fri, May 6, 2016 at 8:02 PM, James Almer wrote: > > > >> On 5/6/2016 8:48 PM, Timothy Gu wrote: > >>> On Fri, May 06,

Re: [FFmpeg-devel] [PATCH 2/2] configure: Enable GCC vectorization on ≥4.9

2016-05-08 Thread Ronald S. Bultje
Hi, On Fri, May 6, 2016 at 8:02 PM, James Almer wrote: > On 5/6/2016 8:48 PM, Timothy Gu wrote: > > On Fri, May 06, 2016 at 12:08:14PM +0200, Hendrik Leppkes wrote: > >> > >> Just to document it, this has caused build breakage in various > >> scenarios, even in GCC 5.3 (6.1 not tested). > >> > >

Re: [FFmpeg-devel] [PATCH 1/3] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-05-06 Thread Ronald S. Bultje
Hi, On Fri, May 6, 2016 at 2:55 PM, Michael Niedermayer wrote: > On Fri, May 06, 2016 at 01:17:56PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Fri, May 6, 2016 at 10:14 AM, Michael Niedermayer > > > wrote: > > > > > On Fri, May 06, 201

Re: [FFmpeg-devel] [PATCH] vf_colorspace: don't enable passthrough if bitdepth doesn't match.

2016-05-06 Thread Ronald S. Bultje
Hi, On Fri, May 6, 2016 at 3:23 PM, Michael Niedermayer wrote: > On Fri, May 06, 2016 at 12:27:02PM -0400, Ronald S. Bultje wrote: > > --- > > libavfilter/vf_colorspace.c | 7 +-- > > 1 file changed, 5 insertions(+), 2 deletions(-) > > > > diff --gi

Re: [FFmpeg-devel] [PATCH 1/3] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-05-06 Thread Ronald S. Bultje
Hi, On Fri, May 6, 2016 at 10:14 AM, Michael Niedermayer wrote: > On Fri, May 06, 2016 at 09:06:01AM -0400, Ronald S. Bultje wrote: > > --- > > doc/filters.texi | 13 > > libavfilter/colorspacedsp.c | 13 > > libavfilter/col

Re: [FFmpeg-devel] [PATCH] vf_colorspace: remove debug statement.

2016-05-06 Thread Ronald S. Bultje
Hi, On Fri, May 6, 2016 at 12:02 PM, Ronald S. Bultje wrote: > This always disabled lrgb2lrgb passthrough, which slightly decreases > performance. > --- > libavfilter/vf_colorspace.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfil

[FFmpeg-devel] [PATCH] vf_colorspace: don't enable passthrough if bitdepth doesn't match.

2016-05-06 Thread Ronald S. Bultje
--- libavfilter/vf_colorspace.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c index 7776a61..f932ce9 100644 --- a/libavfilter/vf_colorspace.c +++ b/libavfilter/vf_colorspace.c @@ -754,7 +754,8 @@ static int crea

[FFmpeg-devel] [PATCH] vf_colorspace: remove debug statement.

2016-05-06 Thread Ronald S. Bultje
This always disabled lrgb2lrgb passthrough, which slightly decreases performance. --- libavfilter/vf_colorspace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c index d119e36..dd66a4e 100644 --- a/libavfilter/vf_colorsp

[FFmpeg-devel] [PATCH] vf_colorspace: fix range order.

2016-05-06 Thread Ronald S. Bultje
--- libavfilter/vf_colorspace.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c index dd66a4e..7776a61 100644 --- a/libavfilter/vf_colorspace.c +++ b/libavfilter/vf_colorspace.c @@ -805,8 +805,8 @@ static int create_

[FFmpeg-devel] [PATCH] vf_colorspace: make whitepoint adaptation mode configurable.

2016-05-06 Thread Ronald S. Bultje
Also add von kries whitepoint adaptation, and add 'identity' to turn whitepoint adaptation off. --- doc/filters.texi| 15 +++ libavfilter/vf_colorspace.c | 39 --- 2 files changed, 47 insertions(+), 7 deletions(-) diff --git a/doc/filter

Re: [FFmpeg-devel] [PATCH] ffmpeg: copy color properties in case of -c:v copy.

2016-05-06 Thread Ronald S. Bultje
Hi, On Tue, May 3, 2016 at 10:21 PM, Michael Niedermayer wrote: > On Tue, May 03, 2016 at 08:58:07PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Tue, May 3, 2016 at 8:29 PM, Michael Niedermayer > > > wrote: > > > > > On Tue, May 03, 201

[FFmpeg-devel] [PATCH] fate: add matroska-remux test.

2016-05-06 Thread Ronald S. Bultje
This tests automatic insertion of the vp9_superframe BSF as well as ensuring that the colorspace properties in the video header can be modified when remuxing (-c:v copy). --- tests/Makefile | 1 + tests/fate/matroska.mak | 9 + 2 files changed, 10 insertions(+) create mode 100644

[FFmpeg-devel] [PATCH 1/3] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-05-06 Thread Ronald S. Bultje
--- doc/filters.texi | 13 libavfilter/colorspacedsp.c | 13 libavfilter/colorspacedsp.h | 6 ++ libavfilter/colorspacedsp_template.c | 128 +++ libavfilter/vf_colorspace.c | 64 +- 5 file

[FFmpeg-devel] [PATCH 3/3] vf_colorspace: use enums for bpp/subsampling array indices.

2016-05-06 Thread Ronald S. Bultje
Also add some documentation for each function to colorspacedsp.h. --- libavfilter/colorspacedsp.c | 78 ++-- libavfilter/colorspacedsp.h | 31 -- libavfilter/x86/colorspacedsp_init.c | 64 ++--- 3 files changed,

Re: [FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-05-06 Thread Ronald S. Bultje
Hi, On Thu, May 5, 2016 at 8:16 AM, Michael Niedermayer wrote: > On Tue, May 03, 2016 at 01:53:42PM -0400, Ronald S. Bultje wrote: > > --- > > doc/filters.texi | 13 > > libavfilter/colorspacedsp.c | 12 > > libavfilter/col

[FFmpeg-devel] [PATCH 2/3] vf_colorspace: add const to yuv_stride[] argument in DSP functions.

2016-05-06 Thread Ronald S. Bultje
--- libavfilter/colorspacedsp.h | 10 +- libavfilter/colorspacedsp_template.c | 6 +++--- libavfilter/colorspacedsp_yuv2yuv_template.c | 4 ++-- libavfilter/x86/colorspacedsp_init.c | 8 4 files changed, 14 insertions(+), 14 deletions(-) diff -

Re: [FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-05-04 Thread Ronald S. Bultje
Hi, On Tue, May 3, 2016 at 1:53 PM, Ronald S. Bultje wrote: > --- > doc/filters.texi | 13 > libavfilter/colorspacedsp.c | 12 > libavfilter/colorspacedsp.h | 6 ++ > libavfilter/colorspacedsp_t

Re: [FFmpeg-devel] [PATCH] ffmpeg: copy color properties in case of -c:v copy.

2016-05-03 Thread Ronald S. Bultje
Hi, On Tue, May 3, 2016 at 8:29 PM, Michael Niedermayer wrote: > On Tue, May 03, 2016 at 03:21:00PM -0400, Ronald S. Bultje wrote: > > I have no idea why the first hunk uses ost->enc_ctx, because as far as > > I understand, that is never used in case of -c:v copy, but this

[FFmpeg-devel] [PATCH] ffmpeg: copy color properties in case of -c:v copy.

2016-05-03 Thread Ronald S. Bultje
I have no idea why the first hunk uses ost->enc_ctx, because as far as I understand, that is never used in case of -c:v copy, but this code block is only entered if encoding_needed=0, which means stream_copy=1. My point being: review from someone that knows this really well would be appreciated. --

Re: [FFmpeg-devel] [PATCH]lavf/webpenc: Do not set alpha flag for yuv420p

2016-05-03 Thread Ronald S. Bultje
Hi, On Tue, May 3, 2016 at 3:22 PM, wm4 wrote: > On Tue, 3 May 2016 18:29:44 +0200 > Carl Eugen Hoyos wrote: > > > On Tuesday 03 May 2016 06:11:59 pm Ronald S. Bultje wrote: > > > Hi, > > > > > > On Tue, May 3, 2016 at 11:54 AM, Carl Eugen Hoyos >

Re: [FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-05-03 Thread Ronald S. Bultje
Hi, On Tue, May 3, 2016 at 1:49 PM, Ronald S. Bultje wrote: > --- > doc/filters.texi | 13 > libavfilter/colorspacedsp.c | 12 > libavfilter/colorspacedsp.h | 6 ++ > libavfilter/colorspacedsp_t

[FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-05-03 Thread Ronald S. Bultje
--- doc/filters.texi | 13 libavfilter/colorspacedsp.c | 12 libavfilter/colorspacedsp.h | 6 ++ libavfilter/colorspacedsp_template.c | 128 +++ libavfilter/vf_colorspace.c | 58 +++- 5 files

[FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-05-03 Thread Ronald S. Bultje
--- doc/filters.texi | 13 libavfilter/colorspacedsp.c | 12 libavfilter/colorspacedsp.h | 6 ++ libavfilter/colorspacedsp_template.c | 128 +++ libavfilter/vf_colorspace.c | 58 +++- 5 files

Re: [FFmpeg-devel] [PATCH]lavf/webpenc: Do not set alpha flag for yuv420p

2016-05-03 Thread Ronald S. Bultje
Hi, On Tue, May 3, 2016 at 11:54 AM, Carl Eugen Hoyos wrote: > On Tuesday 03 May 2016 03:04:50 pm Ronald S. Bultje wrote: > > > > +if (av_pix_fmt_desc_get(st->codecpar->format)->flags > && > > > AV_PIX_FMT_FLAG_ALPHA) > > >

[FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-05-03 Thread Ronald S. Bultje
--- doc/filters.texi | 13 libavfilter/colorspacedsp.c | 12 libavfilter/colorspacedsp.h | 6 ++ libavfilter/colorspacedsp_template.c | 128 +++ libavfilter/vf_colorspace.c | 53 ++- 5 files c

Re: [FFmpeg-devel] [PATCH]lavf/webpenc: Do not set alpha flag for yuv420p

2016-05-03 Thread Ronald S. Bultje
Hi, On Tue, May 3, 2016 at 4:18 AM, Carl Eugen Hoyos wrote: > On Monday 02 May 2016 08:57:31 pm wm4 wrote: > > On Mon, 2 May 2016 08:36:16 -0400 > > > > "Ronald S. Bultje" wrote: > > > Hi, > > > > > > On Mon, May 2, 2016 at 5:49 AM, Car

Re: [FFmpeg-devel] [PATCH]lavf/webpenc: Do not set alpha flag for yuv420p

2016-05-02 Thread Ronald S. Bultje
Hi, On Mon, May 2, 2016 at 5:49 AM, Carl Eugen Hoyos wrote: > On Wednesday 13 April 2016 10:27:03 am you wrote: > > Maybe it should just check whether the pixel format is alpha, instead > > of whether it's not yuv420p. (Just a suggestion.) > > Isn't that exactly equivalent to what my patch is do

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-30 Thread Ronald S. Bultje
Hi, On Sat, Apr 30, 2016 at 9:02 AM, Carl Eugen Hoyos wrote: > > cane please revert this patch? > > I don't think there's wide support for it. > > Two developers seem to wait for an explanation why it should be reverted. Last time I checked, commits were not free-for-all, but should be done on

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-30 Thread Ronald S. Bultje
Hi, On Fri, Apr 29, 2016 at 9:08 AM, Michael Niedermayer wrote: > On Fri, Apr 29, 2016 at 08:09:34AM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Fri, Apr 29, 2016 at 7:52 AM, Carl Eugen Hoyos > wrote: > > > > > I was under the impression that the o

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-29 Thread Ronald S. Bultje
Hi, On Fri, Apr 29, 2016 at 7:52 AM, Carl Eugen Hoyos wrote: > I was under the impression that the only reason that demuxers and decoders > are written like that is that we all want FFmpeg to support invalid streams > as long as this doesn't affect valid streams. It's too broad. I want to know

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-29 Thread Ronald S. Bultje
Hi, On Thu, Apr 28, 2016 at 3:42 AM, Carl Eugen Hoyos wrote: > Kieran Kunhya obe.tv> writes: > > > > It is very important that FFmpeg also decodes invalid > > > files as long as no valid files are affected. > > > > Nonsense, > > > should FFmpeg be able to decode any file created by using netcat

Re: [FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-04-26 Thread Ronald S. Bultje
Hi, On Tue, Apr 26, 2016 at 2:17 PM, Derek Buitenhuis < derek.buitenh...@gmail.com> wrote: > On 4/26/2016 5:41 PM, Ronald S. Bultje wrote: > > --- > > doc/filters.texi | 13 > > libavfilter/colorspacedsp.c | 9 +++ >

[FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-04-26 Thread Ronald S. Bultje
--- doc/filters.texi | 13 libavfilter/colorspacedsp.c | 9 +++ libavfilter/colorspacedsp.h | 6 ++ libavfilter/colorspacedsp_template.c | 128 +++ libavfilter/vf_colorspace.c | 53 ++- 5 files ch

Re: [FFmpeg-devel] CONFIG_W64_DEMUXER and dead-code elimination

2016-04-25 Thread Ronald S. Bultje
Hi, On Mon, Apr 25, 2016 at 1:17 AM, Matt Oliver wrote: > Obviously changing things would require a consensus from all the devs. > Personally I would like to see the DCE stuff removed as currently as stated > debug or lto builds will fail with msvc and these are both options that are > support b

Re: [FFmpeg-devel] [PATCH 2/2] avformat/mpegts: Skip over broken 0x80 headers

2016-04-23 Thread Ronald S. Bultje
Hi, On Sun, Apr 10, 2016 at 11:12 AM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > @@ -2344,6 +2344,12 @@ static int mpegts_resync(AVFormatContext *s, int > seekback) > > avio_seek(pb, -FFMIN(seekback, pos), SEEK_CUR); > > +//Special case for files like 01c56b0dc1.ts > +if

Re: [FFmpeg-devel] [PATCH 5/6] x86: lossless audio: SSE4 madd 32bits

2016-04-19 Thread Ronald S. Bultje
Hi, On Tue, Apr 19, 2016 at 4:42 PM, James Almer wrote: > On 4/18/2016 6:25 PM, Christophe Gisquet wrote: > > 2016-04-18 21:18 GMT+02:00 Michael Niedermayer : > >> > this breaks (only noise) > >> > \[CCCP\]_Mega_Weird_Audio_Test.mkv track 23 > > Worthwhile sample. > > > > I rewrote the patch to

Re: [FFmpeg-devel] [PATCH] vp9: ignore frame_context_index field in key-/intraonly-frames.

2016-04-17 Thread Ronald S. Bultje
Hi, On Sun, Apr 17, 2016 at 6:17 AM, wm4 wrote: > On Sat, 16 Apr 2016 13:56:18 -0400 > "Ronald S. Bultje" wrote: > > > Reproduces a bug to remain consistent with libvpx' behaviour. > > --- > > libavcodec/vp9.c | 2 ++ > > 1 file changed, 2 inser

Re: [FFmpeg-devel] [PATCH] vf_scale: support bt2020 in in/out_color_matrix properties.

2016-04-17 Thread Ronald S. Bultje
Hi, On Fri, Apr 15, 2016 at 8:02 PM, Michael Niedermayer wrote: > On Fri, Apr 15, 2016 at 02:26:47PM -0400, Ronald S. Bultje wrote: > > --- > > libavfilter/vf_scale.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > LGTM (if noone else objects) Pus

[FFmpeg-devel] [PATCH] vp9: ignore frame_context_index field in key-/intraonly-frames.

2016-04-16 Thread Ronald S. Bultje
Reproduces a bug to remain consistent with libvpx' behaviour. --- libavcodec/vp9.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index 5c6f176..70d9c4b 100644 --- a/libavcodec/vp9.c +++ b/libavcodec/vp9.c @@ -642,6 +642,8 @@ static int decode_frame_heade

Re: [FFmpeg-devel] [PATCH] vf_scale: support bt2020 in in/out_color_matrix properties.

2016-04-15 Thread Ronald S. Bultje
Hi, On Fri, Apr 15, 2016 at 2:39 PM, wm4 wrote: > On Fri, 15 Apr 2016 14:26:47 -0400 > "Ronald S. Bultje" wrote: > > > --- > > libavfilter/vf_scale.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/libavfil

[FFmpeg-devel] [PATCH] vf_scale: support bt2020 in in/out_color_matrix properties.

2016-04-15 Thread Ronald S. Bultje
--- libavfilter/vf_scale.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_scale.c b/libavfilter/vf_scale.c index ac9d4c3..22bee96 100644 --- a/libavfilter/vf_scale.c +++ b/libavfilter/vf_scale.c @@ -237,9 +237,11 @@ static const int *parse_yuv_type(const char

Re: [FFmpeg-devel] [PATCH 2/2] vf_colorspace: x86-64 SIMD (SSE2) optimizations.

2016-04-12 Thread Ronald S. Bultje
Hi, On Thu, Apr 7, 2016 at 10:05 AM, Kieran Kunhya wrote: > > On Wed, 6 Apr 2016 at 19:10 Ronald S. Bultje wrote: > >> +pw_1: times 8 dw 1 >> +pw_2: times 8 dw 2 >> +pw_4: times 8 dw 4 >> +pw_8: times 8 dw 8 >> +pw_16: times 8 dw 16 >> +pw_64: times

Re: [FFmpeg-devel] [PATCH 2/2] avformat/framehash: Add more information to the output

2016-04-12 Thread Ronald S. Bultje
Hi James, On Sun, Apr 10, 2016 at 5:22 PM, James Almer wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > Signed-off-by: James Almer > --- > The differences with Michael's original patch are that this one is updated > to use codecpar, and adds the new output as frameh

Re: [FFmpeg-devel] [PATCH] sws/aarch64/yuv2rgb: honor iOS calling convention

2016-04-08 Thread Ronald S. Bultje
Hi, On Fri, Apr 8, 2016 at 7:41 AM, Clément Bœsch wrote: > From: Clément Bœsch > > y_offset and y_coeff being successive 32-bit integers, they are packed > into 8 bytes instead of 2x8 bytes. lgtm. Ronald ___ ffmpeg-devel mailing list ffmpeg-devel@f

[FFmpeg-devel] [PATCH 1/2] lavfi: new colorspace conversion filter.

2016-04-06 Thread Ronald S. Bultje
@@ -0,0 +1,131 @@ +/* + * Copyright (c) 2016 Ronald S. Bultje + * + * This file is part of FFmpeg. + * + * FFmpeg is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1

[FFmpeg-devel] [PATCH 2/2] vf_colorspace: x86-64 SIMD (SSE2) optimizations.

2016-04-06 Thread Ronald S. Bultje
x86/colorspacedsp.asm @@ -0,0 +1,1115 @@ +;* +;* x86-optimized functions for colorspace filter +;* +;* Copyright (C) 2016 Ronald S. Bultje +;* +;* This file is part of FFmpeg. +;* +;* FFmpeg is free software; you can redistribute it and

Re: [FFmpeg-devel] [PATCH] lavfi: new colorspace conversion filter.

2016-04-06 Thread Ronald S. Bultje
Hi, On Tue, Apr 5, 2016 at 7:01 PM, Clément Bœsch wrote: > On Thu, Mar 31, 2016 at 08:29:37PM -0400, Ronald S. Bultje wrote: > > The intent here is similar to colormatrix, but it's LGPLv2.1-or-later > > (instead of GPLv2.0) and supports gamma/chromaticity correcti

Re: [FFmpeg-devel] Support master branch of OpenJPEG and Grok J2K codecs

2016-04-05 Thread Ronald S. Bultje
Hi, On Tue, Apr 5, 2016 at 6:15 AM, wm4 wrote: > On Tue, 05 Apr 2016 10:12:09 + > Kieran Kunhya wrote: > > > > > > > Yes. Well, AGPL extends the definition of distribution to include use > over > > > the network. > > > > > > Would we need then to send a copy of the AGPL to every viewer of o

Re: [FFmpeg-devel] Support master branch of OpenJPEG and Grok J2K codecs

2016-04-04 Thread Ronald S. Bultje
Hi, On Mon, Apr 4, 2016 at 9:21 AM, Aaron Boxer wrote: > Hi Ronald, > > > On Mon, Apr 4, 2016 at 8:56 AM, Ronald S. Bultje > wrote: > > > Hi, > > > > On Mon, Apr 4, 2016 at 8:43 AM, Aaron Boxer wrote: > > > > > Hi Ronald, > > >

Re: [FFmpeg-devel] Support master branch of OpenJPEG and Grok J2K codecs

2016-04-04 Thread Ronald S. Bultje
Hi, On Mon, Apr 4, 2016 at 8:43 AM, Aaron Boxer wrote: > Hi Ronald, > > > On Mon, Apr 4, 2016 at 8:34 AM, Ronald S. Bultje > wrote: > > > Hi, > > > > On Mon, Apr 4, 2016 at 7:59 AM, Aaron Boxer wrote: > > > > > On Mon, Apr 4, 2016 at 7:13 AM,

Re: [FFmpeg-devel] Support master branch of OpenJPEG and Grok J2K codecs

2016-04-04 Thread Ronald S. Bultje
Hi, On Mon, Apr 4, 2016 at 7:59 AM, Aaron Boxer wrote: > On Mon, Apr 4, 2016 at 7:13 AM, wm4 wrote: > > > On Sun, 3 Apr 2016 17:31:25 -0400 > > Aaron Boxer wrote: > > > > > Hi Folks, > > > > > > Here is a small patch to get FFmpeg working with both OpenJPEG master > and > > > Grok master, for

Re: [FFmpeg-devel] How to handled gzip'd files?

2016-04-03 Thread Ronald S. Bultje
Hi, On Sat, Apr 2, 2016 at 12:29 PM, Reimar Döffinger wrote: > Hello, > I was looking at these: > http://samples.mplayerhq.hu/game-formats/la-snm/grimfandango/ > http://samples.mplayerhq.hu/game-formats/la-snm/force-commander-znm/ > and realized that they are perfectly supported, if you first >

Re: [FFmpeg-devel] [PATCH]Addition of MLP encoder

2016-04-01 Thread Ronald S. Bultje
Hi, On Fri, Apr 1, 2016 at 4:57 AM, Disha Singh wrote: > Somehow I am not getting these warnings when I compile. (not all) OK, so we should stop here and fix this. Why is the encoder not being compiled for you? This back-and-forth on compiler warnings/errors is ridiculous. Please figure out ho

[FFmpeg-devel] [PATCH] lavfi: new colorspace conversion filter.

2016-03-31 Thread Ronald S. Bultje
* Copyright (c) 2016 Ronald S. Bultje + * + * This file is part of FFmpeg. + * + * FFmpeg is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or

Re: [FFmpeg-devel] [PATCH] lavfi: new colorspace conversion filter.

2016-03-31 Thread Ronald S. Bultje
Hi, On Wed, Mar 30, 2016 at 4:35 PM, Ronald S. Bultje wrote: > The intent here is similar to colormatrix, but it's LGPLv2.1-or-later > (instead of GPLv2.0) and supports gamma/chromaticity correction. One of the things I'm not 100% happy with is that this is hard to set up, y

[FFmpeg-devel] [PATCH 1/2] swscale: fix bt709 yuv2rgb coefficients.

2016-03-31 Thread Ronald S. Bultje
They were derived from incorrect values cr=0.2125 (should be cr=0.2126) and cb=0.0721 (should be cb=0.0722). --- libswscale/yuv2rgb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libswscale/yuv2rgb.c b/libswscale/yuv2rgb.c index 62abb7d..a0242bf 100644 --- a/libswscale/y

[FFmpeg-devel] [PATCH 2/2] swscale: add bt2020 yuv2rgb coefficients.

2016-03-31 Thread Ronald S. Bultje
--- libswscale/swscale.h | 1 + libswscale/swscale_internal.h | 2 +- libswscale/version.h | 2 +- libswscale/yuv2rgb.c | 9 ++--- 4 files changed, 9 insertions(+), 5 deletions(-) diff --git a/libswscale/swscale.h b/libswscale/swscale.h index da9dd2e..6b8b926 100644

Re: [FFmpeg-devel] [PATCH] swscale: add bt2020 yuv2rgb coefficients.

2016-03-31 Thread Ronald S. Bultje
Hi, On Thu, Mar 31, 2016 at 2:51 PM, Clément Bœsch wrote: > On Thu, Mar 31, 2016 at 02:29:12PM -0400, Ronald S. Bultje wrote: > > Also fix the bt709 ones. They seem derived from incorrect values > > cr=0.2125 (should be cr=0.2126) and cb=0.0721 (should be cb=0.0722). > >

Re: [FFmpeg-devel] [PATCH] swscale: add bt2020 yuv2rgb coefficients.

2016-03-31 Thread Ronald S. Bultje
Hi, On Thu, Mar 31, 2016 at 2:46 PM, Michael Niedermayer wrote: > On Thu, Mar 31, 2016 at 02:29:12PM -0400, Ronald S. Bultje wrote: > > Also fix the bt709 ones. They seem derived from incorrect values > > cr=0.2125 (should be cr=0.2126) and cb=0.0721 (shou

[FFmpeg-devel] [PATCH] swscale: add bt2020 yuv2rgb coefficients.

2016-03-31 Thread Ronald S. Bultje
Also fix the bt709 ones. They seem derived from incorrect values cr=0.2125 (should be cr=0.2126) and cb=0.0721 (should be cb=0.0722). --- libswscale/swscale.h | 1 + libswscale/yuv2rgb.c | 13 - 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/libswscale/swscale.h b/libs

Re: [FFmpeg-devel] [PATCH] avformat/latmenc: automatically insert aac_adtstoasc bitstream filter when needed

2016-03-30 Thread Ronald S. Bultje
Hi, On Sat, Mar 26, 2016 at 12:45 AM, James Almer wrote: > Signed-off-by: James Almer > --- > libavformat/latmenc.c | 20 +++- > 1 file changed, 15 insertions(+), 5 deletions(-) lgtm. Ronald ___ ffmpeg-devel mailing list ffmpeg-dev

[FFmpeg-devel] [PATCH] lavfi: new colorspace conversion filter.

2016-03-30 Thread Ronald S. Bultje
c new file mode 100644 index 000..e71bf1f --- /dev/null +++ b/libavfilter/colorspacedsp.c @@ -0,0 +1,130 @@ +/* + * Copyright (c) 2016 Ronald S. Bultje + * + * This file is part of FFmpeg. + * + * FFmpeg is free software; you can redistribute it and/or + * modify it under the terms of the GNU L

Re: [FFmpeg-devel] [PATCH 4/4] avfilter/vf_colormatrix: add 10 & 12 bit depth support

2016-03-27 Thread Ronald S. Bultje
Hi, On Sun, Mar 27, 2016 at 7:09 AM, Thomas Mundt < loudmax-at-yahoo...@ffmpeg.org> wrote: > >>>Ronald S. Bultje schrieb am Sa, 26.3.2016: > > On Sat, Mar 26, 2016 at 10:04 AM, Thomas Mundt ffmpeg.org> wrote: > > > >>>>>Kieran Kunhya schrieb

Re: [FFmpeg-devel] [PATCH 4/4] avfilter/vf_colormatrix: add 10 & 12 bit depth support

2016-03-26 Thread Ronald S. Bultje
Hi, On Sat, Mar 26, 2016 at 10:04 AM, Thomas Mundt < loudmax-at-yahoo...@ffmpeg.org> wrote: > >>>Kieran Kunhya schrieb am Sa, 26.3.2016: > >> On Fri, 25 Mar 2016 at 22:32 Thomas Mundt ffmpeg.org> wrote: > > > >> Signed-off-by: Thomas Mundt > >> --- > >> libavfilter/vf_colormatrix.c | 182 > >>

Re: [FFmpeg-devel] [PATCH 0/4] avfilter/vf_colormatrix: add UHD support

2016-03-26 Thread Ronald S. Bultje
Hi, On Fri, Mar 25, 2016 at 6:30 PM, Thomas Mundt < loudmax-at-yahoo...@ffmpeg.org> wrote: > These patches add bt.2020 colorspace and 10 and 12 bit depth support. > The calculation of the Y component is simplified but with identical > results. > Some unused variables are removed. > The YUV croma

Re: [FFmpeg-devel] [PATCH 0/4] fftw exploration (WIP)

2016-03-25 Thread Ronald S. Bultje
Hi, On Fri, Mar 25, 2016 at 3:55 PM, Ganesh Ajjanagadde wrote: > On Fri, Mar 25, 2016 at 12:11 PM, Paul B Mahol wrote: > > On 3/25/16, Ganesh Ajjanagadde wrote: > >> On Fri, Mar 25, 2016 at 9:36 AM, Nicolas George > wrote: > >>> Le sextidi 6 germinal, an CCXXIV, Ganesh Ajjanagadde a ecrit : >

Re: [FFmpeg-devel] [PATCH] sws/aarch64: add ff_hscale_8_to_15_neon

2016-03-24 Thread Ronald S. Bultje
Hi, On Mar 24, 2016 8:28 AM, "Clément Bœsch" wrote: > > From: Clément Bœsch > > ./ffmpeg -nostats -f lavfi -i testsrc2=4k:d=2 -vf bench=start,scale=1024x1024,bench=stop -f null - > > before: t:0.489726 avg:0.489883 max:0.491852 min:0.489482 > after: t:0.256515 avg:0.256458 max:0.256999

Re: [FFmpeg-devel] [PATCH] added support for hardware assist H264 video encoding for the Raspberry Pi

2016-03-23 Thread Ronald S. Bultje
Hi, On Wed, Mar 23, 2016 at 11:41 AM, Amancio Hasty wrote: > Is there a pretty formatter for ffmpeg’s coding style and > coding style guideline? https://ffmpeg.org/developer.html#Coding-Rules-1 Please don't top-post. Ronald ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] size=0, but av_malloc(1)

2016-03-23 Thread Ronald S. Bultje
Hi, On Wed, Mar 23, 2016 at 8:32 AM, Ronald S. Bultje wrote: > Hi, > > On Wed, Mar 23, 2016 at 2:43 AM, Chris Cunningham < > chcunning...@chromium.org> wrote: > >> Hey Group, >> >> I'm seeing an interesting pattern [0][1] where we allocate 1

<    6   7   8   9   10   11   12   13   14   15   >