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

2016-05-06 Thread Stephen Hutchinson
On 5/6/2016 6:08 AM, Hendrik Leppkes wrote: On Tue, Feb 9, 2016 at 1:28 AM, Timothy Gu wrote: On Sun, Jan 31, 2016 at 3:38 PM Timothy Gu wrote: On Sat, Jan 30, 2016 at 07:27:22PM +, Derek Buitenhuis wrote: On 1/30/2016 7:15 PM, Timothy Gu

Re: [FFmpeg-devel] [PATCH] avformat/riff: add M102 FourCC

2016-05-06 Thread Michael Niedermayer
On Fri, May 06, 2016 at 10:48:32PM +0200, Piotr Bandurski wrote: > From b365a1b2544e5e70652f41e7e8bf4066fb778b4c Mon Sep 17 00:00:00 2001 > From: Piotr Bandurski > Date: Fri, 6 May 2016 22:43:00 +0200 > Subject: [PATCH] avformat/riff: add M102 FourCC > > --- >

Re: [FFmpeg-devel] [PATCH] hevc: fix size condition in ptl parsing

2016-05-06 Thread Michael Niedermayer
On Fri, May 06, 2016 at 07:15:06PM +0200, Hendrik Leppkes wrote: > When only one sublayer is present, no information is coded. Only when at > least two > are present, all 8 sublayers are written. > --- > libavcodec/hevc_ps.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM thx

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

2016-05-06 Thread James Almer
On 5/6/2016 9:56 PM, Hendrik Leppkes wrote: > On Sat, May 7, 2016 at 2:02 AM, 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

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

2016-05-06 Thread Hendrik Leppkes
On Sat, May 7, 2016 at 2:02 AM, 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

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

2016-05-06 Thread James Almer
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). >> >> The latest reported on IRC just today here: >> libavcodec/sbrdsp.c:

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

2016-05-06 Thread Timothy Gu
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). > > The latest reported on IRC just today here: > libavcodec/sbrdsp.c: In function 'sbr_neg_odd_64_c': >

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

2016-05-06 Thread Michael Niedermayer
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 > modified when remuxing (-c:v copy). > --- > tests/Makefile | 1 + >

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

2016-05-06 Thread Michael Niedermayer
On Fri, May 06, 2016 at 03:44:49PM -0400, Ronald S. Bultje wrote: > 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

Re: [FFmpeg-devel] [PATCH] avformat/riff: add M102 FourCC

2016-05-06 Thread Piotr Bandurski
samples are here: https://www.datafilehost.com/d/1a93d9a6 (some matrox uncompressed samples I posted to ticket #2616 are invalid due to bug in the older version of matrox codec which I used to encode them) ___ ffmpeg-devel mailing list

[FFmpeg-devel] [PATCH] avformat/riff: add M102 FourCC

2016-05-06 Thread Piotr Bandurski
0001-avformat-riff-add-M102-FourCC.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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, 2016

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 --git

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

2016-05-06 Thread Michael Niedermayer
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 --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c > index 7776a61..f932ce9 100644 > ---

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

2016-05-06 Thread Michael Niedermayer
On Fri, May 06, 2016 at 09:06:02AM -0400, Ronald S. Bultje wrote: > --- > libavfilter/colorspacedsp.h | 10 +- > libavfilter/colorspacedsp_template.c | 6 +++--- > libavfilter/colorspacedsp_yuv2yuv_template.c | 4 ++-- > libavfilter/x86/colorspacedsp_init.c

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

2016-05-06 Thread Michael Niedermayer
On Fri, May 06, 2016 at 09:06:03AM -0400, Ronald S. Bultje wrote: > Also add some documentation for each function to colorspacedsp.h. > --- > libavfilter/colorspacedsp.c | 78 > ++-- > libavfilter/colorspacedsp.h | 31 -- >

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

2016-05-06 Thread Michael Niedermayer
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, 2016 at 09:06:01AM -0400, Ronald S. Bultje wrote: > > > --- > > > doc/filters.texi | 13

Re: [FFmpeg-devel] [PATCH] hevc: fix size condition in ptl parsing

2016-05-06 Thread Hendrik Leppkes
On Fri, May 6, 2016 at 7:49 PM, Carl Eugen Hoyos wrote: > Hendrik Leppkes gmail.com> writes: > >> -get_bits_left(gb) < 8 + 8*2) { >> +get_bits_left(gb) < 8 + (8*2 * (max_num_sub_layers - 1 > 0))) { > > Isnt't "max_num_sub_layers > 1" more readable? > Perhaps, but this

Re: [FFmpeg-devel] [PATCH] h2645_parse: allow partial escaping

2016-05-06 Thread Carl Eugen Hoyos
Hendrik Leppkes gmail.com> writes: > This ports the fix from 033a533 to the new parser module in > prepartion of using it for the h264 decoder.h2645_parse: > allow partial escaping Could this patch port ticket #5453 from one codec to the next? Carl Eugen

Re: [FFmpeg-devel] [PATCH] hevc: fix size condition in ptl parsing

2016-05-06 Thread Carl Eugen Hoyos
Hendrik Leppkes gmail.com> writes: > -get_bits_left(gb) < 8 + 8*2) { > +get_bits_left(gb) < 8 + (8*2 * (max_num_sub_layers - 1 > 0))) { Isnt't "max_num_sub_layers > 1" more readable? Carl Eugen ___ ffmpeg-devel mailing list

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 > >

[FFmpeg-devel] [PATCH] hevc: fix size condition in ptl parsing

2016-05-06 Thread Hendrik Leppkes
When only one sublayer is present, no information is coded. Only when at least two are present, all 8 sublayers are written. --- libavcodec/hevc_ps.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/hevc_ps.c b/libavcodec/hevc_ps.c index ce3f3df..83f2ec2 100644 ---

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

Re: [FFmpeg-devel] [Patch] Add input swap functionality to movie filter (src_movie.c)

2016-05-06 Thread Reimar Döffinger
On 04.05.2016, at 17:01, "Felt, Patrick" wrote: > > On 5/4/16, 5:52 AM, "ffmpeg-devel on behalf of Clément Bœsch" > wrote: > >> >> [...] >>> +/* libavfilter documentation says that filter init will be

Re: [FFmpeg-devel] [PATCH] h2645_parse: allow partial escaping

2016-05-06 Thread Michael Niedermayer
On Fri, May 06, 2016 at 04:30:38PM +0200, Hendrik Leppkes wrote: > This ports the fix from 033a533 to the new parser module in prepartion > of using it for the h264 decoder.h2645_parse: allow partial escaping > --- > libavcodec/h2645_parse.c | 4 ++-- LGTM thx [...] -- Michael GnuPG

[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 ---

[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

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

2016-05-06 Thread James Almer
On 5/6/2016 7:08 AM, Hendrik Leppkes wrote: > On Tue, Feb 9, 2016 at 1:28 AM, Timothy Gu wrote: >> On Sun, Jan 31, 2016 at 3:38 PM Timothy Gu wrote: >>> >>> On Sat, Jan 30, 2016 at 07:27:22PM +, Derek Buitenhuis wrote: On 1/30/2016 7:15 PM,

Re: [FFmpeg-devel] [PATCH] h2645_parse: allow partial escaping

2016-05-06 Thread Hendrik Leppkes
On Fri, May 6, 2016 at 4:30 PM, Hendrik Leppkes wrote: > This ports the fix from 033a533 to the new parser module in prepartion > of using it for the h264 decoder.h2645_parse: allow partial escaping Nevermind the broken message, already fixed.

[FFmpeg-devel] [PATCH] h2645_parse: allow partial escaping

2016-05-06 Thread Hendrik Leppkes
This ports the fix from 033a533 to the new parser module in prepartion of using it for the h264 decoder.h2645_parse: allow partial escaping --- libavcodec/h2645_parse.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/h2645_parse.c b/libavcodec/h2645_parse.c

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

2016-05-06 Thread Michael Niedermayer
On Fri, May 06, 2016 at 12:08:14PM +0200, Hendrik Leppkes wrote: > On Tue, Feb 9, 2016 at 1:28 AM, Timothy Gu wrote: > > On Sun, Jan 31, 2016 at 3:38 PM Timothy Gu wrote: > >> > >> On Sat, Jan 30, 2016 at 07:27:22PM +, Derek Buitenhuis wrote: >

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

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

[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

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add M101 decoder

2016-05-06 Thread Michael Niedermayer
On Thu, May 05, 2016 at 07:45:47PM +0200, Michael Niedermayer wrote: > TODO: find out which LSB for 10bit go where > > Signed-off-by: Michael Niedermayer > --- > libavcodec/Makefile |1 + > libavcodec/allcodecs.c |1 + > libavcodec/avcodec.h|1 + >

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, 2016

[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

[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

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 > >

[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 57/57] support for matrox m703 mpeg-2

2016-05-06 Thread Michael Niedermayer
On Fri, May 06, 2016 at 11:26:04AM +0200, Piotr Bandurski wrote: > > file attached but i think there is something wrong with the encoder > > used, the files do not seem to contain enough non zero bits to > > be able to contain full yuv422 10bit > > so i am not sure this will resolve this > > ok,

Re: [FFmpeg-devel] [PATCH] Bonk decoder and demuxer

2016-05-06 Thread Piotr Bandurski
> > > On 5/5/16, Piotr Bandurski wrote: > > >> > > > > > > > > Improved version attached. > > >> > > > > > >> > > > I found another bug, this time in lossy mode - the file encoded as > > >> > > > "joint stereo" decodes after a few seconds into noise: > > >> > > > > > >> > > >

Re: [FFmpeg-devel] [PATCH 1/2] vc2enc: prevent random data

2016-05-06 Thread Christophe Gisquet
Hi, 2016-05-06 2:19 GMT+02:00 Rostislav Pehlivanov : > I plan to merge the fate tests as well tomorrow or on Saturday when I'll > have time to quickly fix bugs which appear on platforms I haven't tested > the encoder on. Hopefully none, but you never know. Sure, makes sense.

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

2016-05-06 Thread Hendrik Leppkes
On Tue, Feb 9, 2016 at 1:28 AM, Timothy Gu wrote: > On Sun, Jan 31, 2016 at 3:38 PM Timothy Gu wrote: >> >> On Sat, Jan 30, 2016 at 07:27:22PM +, Derek Buitenhuis wrote: >> > On 1/30/2016 7:15 PM, Timothy Gu wrote: >> > > FATE passes here on a

Re: [FFmpeg-devel] [PATCH] set exact ref frame in AVMotionVector

2016-05-06 Thread Derek Buitenhuis
On 5/6/2016 10:54 AM, Davinder Singh wrote: > change the confusing "source" var to "ref" of AVMotionVector. This would be a backwards incompatible API break, no? - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] [PATCH] set exact ref frame in AVMotionVector

2016-05-06 Thread Davinder Singh
hi, change the confusing "source" var to "ref" of AVMotionVector. define two constants AV_MV_REF_FRAME_PREVIOUS and AV_MV_REF_FRAME_NEXT all changes: https://github.com/dsmudhar/FFmpeg/commit/33b5ad805cccd7b48a6b0d643c39e8dd26f0e98b please confirm this: snowdec.c Line 96: BlockNode *bn... ...

Re: [FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-05-06 Thread Piotr Bandurski
> file attached but i think there is something wrong with the encoder > used, the files do not seem to contain enough non zero bits to > be able to contain full yuv422 10bit > so i am not sure this will resolve this ok, encoded it to m101 10bit: https://www.datafilehost.com/d/119d82c3

[FFmpeg-devel] [PATCH 2/2] dshow: Make dshow device buildable on Cygwin

2016-05-06 Thread Moriyoshi Koizumi
From: Moriyoshi koizumi wchar_t is not always the same thing as WCHAR. Signed-off-by: Moriyoshi koizumi --- libavdevice/dshow.c | 6 +++--- libavdevice/dshow_capture.h | 10 +- libavdevice/dshow_filter.c | 16

[FFmpeg-devel] [PATCH 2/2] dshow: Make dshow device buildable on Cygwin

2016-05-06 Thread Moriyoshi Koizumi
From: Moriyoshi koizumi wchar_t is not always the same thing as WCHAR. Signed-off-by: Moriyoshi koizumi --- libavdevice/dshow.c | 6 +++--- libavdevice/dshow_capture.h | 10 +- libavdevice/dshow_filter.c | 16

[FFmpeg-devel] [PATCH 1/2] dshow: Add missing library dependency (dxguid)

2016-05-06 Thread Moriyoshi Koizumi
From: Moriyoshi koizumi Signed-off-by: Moriyoshi koizumi --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 1f684d2..fb8d7d3 100755 --- a/configure +++ b/configure @@ -2832,7 +2832,7 @@