Re: [FFmpeg-devel] [PATCH] define AVPixelFormat aliases as enumerators instead of macros

2016-01-16 Thread wm4
On Fri, 15 Jan 2016 10:39:59 -0800 Richard Smith wrote: > On Fri Jan 15 08:51:07 CET 2016 wm4 wrote; > > On Thu, 14 Jan 2016 13:58:14 -0800 Richard Smith > > wrote: > > > libavutil/pixfmt.h defines a collection of endian-specific pixel formats > > > as > > > macros. These macro names can ca

Re: [FFmpeg-devel] [PATCH] remove the deprecated avpicture_get_size() function

2016-01-16 Thread wm4
On Fri, 15 Jan 2016 23:34:01 -0800 Eddie Hao wrote: > --- > libavcodec/avcodec.h | 6 -- > libavcodec/avpicture.c | 5 - > libavcodec/libutvideodec.cpp | 3 ++- > libavcodec/libutvideoenc.cpp | 3 ++- > libavdevice/decklink_dec.cpp | 2 +- > libavdevice/lavfi.c | 3

Re: [FFmpeg-devel] [PATCH] lavc/rawdec: Use AV_PIX_FMT_PAL8 for 1-bit raw QuickTime video

2016-01-16 Thread Michael Niedermayer
On Wed, Jan 13, 2016 at 01:42:24PM +0100, Mats Peterson wrote: > Match the use of AV_PIX_FMT_PAL8 for 1-bit QuickTime Animation in > lavc/qtrle. To reiterate, 1-bit video is not necessary black & white in > QuickTime, merely bi-level. The two colors can be any color. The palette, > either included

Re: [FFmpeg-devel] [PATCH] lavc/rawdec: Use AV_PIX_FMT_PAL8 for 1-bit raw QuickTime video

2016-01-16 Thread Mats Peterson
On 01/16/2016 12:10 PM, Michael Niedermayer wrote: breaks https://trac.ffmpeg.org/raw-attachment/ticket/823/1bpp.mov and https://trac.ffmpeg.org/raw-attachment/ticket/823/1bpp_129.mov do you have a file that this fixes ? Interesting. Well, I don't have a file that gets "fixed" in any way, bu

Re: [FFmpeg-devel] [PATCH] lavc/rawdec: Use AV_PIX_FMT_PAL8 for 1-bit raw QuickTime video

2016-01-16 Thread Mats Peterson
On 01/16/2016 12:10 PM, Michael Niedermayer wrote: breaks https://trac.ffmpeg.org/raw-attachment/ticket/823/1bpp.mov and https://trac.ffmpeg.org/raw-attachment/ticket/823/1bpp_129.mov do you have a file that this fixes ? Could it have something to do with the "linesize_align" variable that is

Re: [FFmpeg-devel] [PATCH] lavc/rawdec: Use AV_PIX_FMT_PAL8 for 1-bit raw QuickTime video

2016-01-16 Thread Mats Peterson
On 01/16/2016 12:24 PM, Mats Peterson wrote: Could it have something to do with the "linesize_align" variable that is 4 by default? I have no idea. I suppose since the width of both these files is an odd value, there has to be some alignment to even boundaries. I'll try various values of lines

Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: Get sample size from private data

2016-01-16 Thread Michael Niedermayer
On Fri, Jan 15, 2016 at 11:31:49PM +0100, Mats Peterson wrote: > Since track->audio.bitdepth is zero for A_QUICKTIME, the sample size > has to be retrieved from the private data. Also, 'twos' and 'sowt' > audio can be signed 8-bit. > > Mats > matroskadec.c | 10 -- > 1 file changed, 8

Re: [FFmpeg-devel] [PATCH] lavf/mov: Don't limit fourcc 0 -> raw/twos to version 0 sample descriptions

2016-01-16 Thread Michael Niedermayer
On Fri, Jan 15, 2016 at 11:35:08PM +0100, Mats Peterson wrote: > I had the notion that the fourcc 0x to raw/twos mapping was > only valid for version 0 sound sample descriptions. However, the > documentation is quite foggy regarding this subject, so it's better > to do this mapping regardle

Re: [FFmpeg-devel] [PATCH] mov: Add an option to toggle dref opening

2016-01-16 Thread Michael Niedermayer
On Fri, Jan 15, 2016 at 05:03:49PM +, Derek Buitenhuis wrote: > This feature is mostly only used by NLE software, and is > both of dubious value being enabled by default, and a > possible security risk. > > Signed-off-by: Derek Buitenhuis > --- > libavformat/isom.h| 1 + > libavformat/m

Re: [FFmpeg-devel] [PATCH] News entry for FFmpeg 2.8.5, 2.7.5, 2.6.7, 2.5.10

2016-01-16 Thread Michael Niedermayer
On Fri, Jan 15, 2016 at 09:22:27PM -0500, Ganesh Ajjanagadde wrote: > On Fri, Jan 15, 2016 at 9:15 PM, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > --- > > src/index | 14 ++ > > 1 file changed, 14 insertions(+) > > > > diff --git a/src/index b/src/index > > ind

Re: [FFmpeg-devel] [PATCH 2/2] fate: add fixed-dsp test

2016-01-16 Thread Ronald S. Bultje
Hi, On Fri, Jan 15, 2016 at 8:10 PM, James Almer wrote: > Adapted from float-dsp So, it seems this just compares C vs. SIMD, right? Shouldn't we use checkasm for that? Ronald ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/ma

Re: [FFmpeg-devel] [PATCH] lavf/mov: Don't limit fourcc 0 -> raw/twos to version 0 sample descriptions

2016-01-16 Thread Mats Peterson
On 01/16/2016 02:17 PM, Michael Niedermayer wrote: applied thanks Thanks, Michael. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: Get sample size from private data

2016-01-16 Thread Mats Peterson
On 01/16/2016 02:16 PM, Michael Niedermayer wrote: applied thx Thanks, Michael. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] mov: Add an option to toggle dref opening

2016-01-16 Thread wm4
On Sat, 16 Jan 2016 14:22:21 +0100 Michael Niedermayer wrote: > On Fri, Jan 15, 2016 at 05:03:49PM +, Derek Buitenhuis wrote: > > This feature is mostly only used by NLE software, and is > > both of dubious value being enabled by default, and a > > possible security risk. > > > > Signed-off-

Re: [FFmpeg-devel] [PATCH 2/2] fate: add fixed-dsp test

2016-01-16 Thread James Almer
On 1/16/2016 10:43 AM, Ronald S. Bultje wrote: > Hi, > > On Fri, Jan 15, 2016 at 8:10 PM, James Almer wrote: > >> Adapted from float-dsp > > > So, it seems this just compares C vs. SIMD, right? Shouldn't we use > checkasm for that? > > Ronald I guess that'd be best, yeah. I'll port this one

Re: [FFmpeg-devel] [PATCH] lavc/rawdec: Use AV_PIX_FMT_PAL8 for 1-bit raw QuickTime video

2016-01-16 Thread Mats Peterson
On 01/16/2016 12:10 PM, Michael Niedermayer wrote: do you have a file that this fixes ? I do have a file, *now*. I've added a palette with blue color to a raw 1-bit QuickTime file just to have something to work on. https://drive.google.com/open?id=0B3_pEBoLs0faSDhIMWdmeFJtcDg

[FFmpeg-devel] [PATCH] avformat/hls: Require the file extension to be m3u / m3u8 for probing to succeed

2016-01-16 Thread Michael Niedermayer
From: Michael Niedermayer If the filename isnt set by the user application then the code behaves like before Signed-off-by: Michael Niedermayer --- libavformat/hls.c |4 1 file changed, 4 insertions(+) diff --git a/libavformat/hls.c b/libavformat/hls.c index 7a8610c..846d884 100644

[FFmpeg-devel] [PATCH] videodsp: fix 1-byte overread in top/bottom READ_NUM_BYTES iterations.

2016-01-16 Thread Ronald S. Bultje
This can overread (either before start or beyond end) of the buffer in Nx1 (i.e. height=1) images. Fixes mozilla bug 1240080. --- libavcodec/x86/videodsp.asm | 21 ++--- 1 file changed, 6 insertions(+), 15 deletions(-) diff --git a/libavcodec/x86/videodsp.asm b/libavcodec/x86/vid

Re: [FFmpeg-devel] [PATCH] mov: Add an option to toggle dref opening

2016-01-16 Thread Marton Balint
On Sat, 16 Jan 2016, wm4 wrote: On Sat, 16 Jan 2016 14:22:21 +0100 Michael Niedermayer wrote: On Fri, Jan 15, 2016 at 05:03:49PM +, Derek Buitenhuis wrote: > This feature is mostly only used by NLE software, and is > both of dubious value being enabled by default, and a > possible security

[FFmpeg-devel] [PATCH] avfilter: add afftfilter

2016-01-16 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- configure | 3 + libavfilter/Makefile | 1 + libavfilter/af_afftfilt.c | 363 ++ libavfilter/allfilters.c | 1 + 4 files changed, 368 insertions(+) create mode 100644 libavfilter/af_afftfilt.c

Re: [FFmpeg-devel] [PATCH 1/2] ffplay: Seek only when pressing the right mouse button on the video window.

2016-01-16 Thread Marton Balint
On Fri, 15 Jan 2016, Vittorio Gambaletta (VittGam) wrote: Seeking by clicking on the video window can be annoying, because the user might click on it accidentally while eg. trying to get focus on it, and ffplay seeks instead. This commit changes that behaviour to seek only when the right mouse

Re: [FFmpeg-devel] [PATCH 2/2] ffplay: Toggle full screen when double-clicking the video window with the left mouse button.

2016-01-16 Thread Marton Balint
On Fri, 15 Jan 2016, Vittorio Gambaletta (VittGam) wrote: Now that the seek only happens with the right mouse button, it makes sense to toggle full screen when double-clicking with the left mouse button, like other video players do. I am not against this. Signed-off-by: Vittorio Gambaletta

[FFmpeg-devel] [PATCH] lavf/img2dec: change the default pattern to none

2016-01-16 Thread Marton Balint
The current default pattern type, glob_sequence is deprecated for almost 3.5 years, before removing it, we need a new, sensible pattern default. I suggest "none" which turns off pattern matching, because using the filename as a pattern can be a security risk exposing other files in the source direc

Re: [FFmpeg-devel] [PATCH 1/2] ffplay: Seek only when pressing the right mouse button on the video window.

2016-01-16 Thread Vittorio Gambaletta (VittGam)
Hi, On 16/01/2016 21:13:14 CET, Marton Balint wrote: On Fri, 15 Jan 2016, Vittorio Gambaletta (VittGam) wrote: Seeking by clicking on the video window can be annoying, because the user might click on it accidentally while eg. trying to get focus on it, and ffplay seeks instead. This commit ch

Re: [FFmpeg-devel] [PATCH 2/2] ffplay: Toggle full screen when double-clicking the video window with the left mouse button.

2016-01-16 Thread Vittorio Gambaletta (VittGam)
Hi, On 16/01/2016 21:25:00 CET, Marton Balint wrote: On Fri, 15 Jan 2016, Vittorio Gambaletta (VittGam) wrote: Now that the seek only happens with the right mouse button, it makes sense to toggle full screen when double-clicking with the left mouse button, like other video players do. I am n

Re: [FFmpeg-devel] [PATCH] lavf/img2dec: change the default pattern to none

2016-01-16 Thread wm4
On Sat, 16 Jan 2016 21:42:48 +0100 Marton Balint wrote: > The current default pattern type, glob_sequence is deprecated for almost 3.5 > years, before removing it, we need a new, sensible pattern default. I suggest > "none" which turns off pattern matching, because using the filename as a > patte

[FFmpeg-devel] [PATCH v2 1/2] ffplay: Seek only when pressing the right mouse button on the video window.

2016-01-16 Thread Vittorio Gambaletta (VittGam)
Seeking by clicking on the video window can be annoying, because the user might click on it accidentally while eg. trying to get focus on it, and ffplay seeks instead. This commit changes that behaviour to seek only when the right mouse button is used to click and drag on the window. Signed-off-b

[FFmpeg-devel] [PATCH v2 2/2] ffplay: Toggle full screen when double-clicking the video window with the left mouse button.

2016-01-16 Thread Vittorio Gambaletta (VittGam)
Now that the seek only happens with the right mouse button, it makes sense to toggle full screen when double-clicking with the left mouse button, like other video players do. Signed-off-by: Vittorio Gambaletta --- ffplay.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/ffplay

Re: [FFmpeg-devel] [PATCH v2 1/2] ffplay: Seek only when pressing the right mouse button on the video window.

2016-01-16 Thread Vittorio Gambaletta (VittGam)
Hi, It seems to me that whitespace was right on my end in the other patches... This time I've literally piped the mails to my mailserver using netcat, like I always do with OpenWrt patches anyway, so please check if I'm still getting something wrong with whitespace now. Thank you, Vittorio On

Re: [FFmpeg-devel] [PATCH 1/2] x86/fixed_dsp: add ff_butterflies_fixed_sse2

2016-01-16 Thread Paul B Mahol
On 1/16/16, James Almer wrote: > Signed-off-by: James Almer > --- > libavutil/fixed_dsp.c | 3 +++ > libavutil/fixed_dsp.h | 2 ++ > libavutil/x86/Makefile | 2 ++ > libavutil/x86/fixed_dsp.asm| 48 > ++ > libavutil/x86/fix

Re: [FFmpeg-devel] [PATCH 1/2] x86/fixed_dsp: add ff_butterflies_fixed_sse2

2016-01-16 Thread James Almer
On 1/16/2016 8:55 PM, Paul B Mahol wrote: > On 1/16/16, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavutil/fixed_dsp.c | 3 +++ >> libavutil/fixed_dsp.h | 2 ++ >> libavutil/x86/Makefile | 2 ++ >> libavutil/x86/fixed_dsp.asm| 48 >>

[FFmpeg-devel] [PATCH] libavutil: add mastering display metadata sidedata

2016-01-16 Thread Neil Birkbeck
Adding mastering display metadata struct to avutil. The mastering display metadata contains information about the mastering display color volume (SMPTE 2086:2014). This info comes from HEVC in the SEI_TYPE_MASTERING_DISPLAY_INFO and is soon to be included in MKV: https://mailarchive.ietf.org/a

Re: [FFmpeg-devel] [PATCH 1/4] fate: add 10-bit v210 encoder tests

2016-01-16 Thread James Darnley
On 2016-01-15 20:07, James Darnley wrote: > ... If nobody has further comments about the patches I will probably push these after I wake up. signature.asc Description: OpenPGP digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org htt

[FFmpeg-devel] [PATCH] remove the deprecated avpicture_get_size() function

2016-01-16 Thread Eddie Hao
--- libavcodec/libutvideodec.cpp | 3 ++- libavcodec/libutvideoenc.cpp | 3 ++- libavdevice/decklink_dec.cpp | 2 +- libavdevice/lavfi.c | 3 ++- libavformat/frmdec.c | 3 ++- 5 files changed, 9 insertions(+), 5 deletions(-) diff --git a/libavcodec/libutvideodec.cpp b/libavcodec/