Re: [FFmpeg-devel] [PATCH] hlsenc: don't use %s for strftime on msvc

2017-02-20 Thread Hendrik Leppkes
On Tue, Feb 21, 2017 at 8:24 AM, Hendrik Leppkes wrote: > MSVC doesn't support the %s time format, and instead of returning an > error the invalid parameter handler is invoked which (by default) > terminates the process. > --- > libavformat/hlsenc.c | 3 ++- > 1 file changed, 2 insertions(+), 1 d

[FFmpeg-devel] [PATCH] hlsenc: don't use %s for strftime on msvc

2017-02-20 Thread Hendrik Leppkes
MSVC doesn't support the %s time format, and instead of returning an error the invalid parameter handler is invoked which (by default) terminates the process. --- libavformat/hlsenc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c

[FFmpeg-devel] [PATCH] lavfi/buffersrc: fix directly setting channel layout

2017-02-20 Thread Rostislav Pehlivanov
When setting the channel layout directly using AVBufferSrcParameters the channel layout was correctly set however the init function still expected the old string format to set the number of channels (when it hadn't already been specified). Signed-off-by: Rostislav Pehlivanov --- libavfilter/buff

Re: [FFmpeg-devel] [PATCH] doc: drawtext options update

2017-02-20 Thread Mulvya V
Hi, Revised patch to include basetime option. Thanks, mulvya On Sun, Feb 19, 2017 at 10:00 PM, Mulvya V wrote: > Hi, > > Added entry for hitherto undocumented timecode wrap option. Removed > bizarre entry for non-existent draw option. > > Haven't added entry for basetime option since it applie

Re: [FFmpeg-devel] [PATCH] doc: drawtext options update

2017-02-20 Thread Lou Logan
Hi, On Sun, Feb 19, 2017, at 07:30 AM, Mulvya V wrote: > Haven't added entry for basetime option since it applies only in strftime > expansion mode. Should it be documented? If it still works, sure, as long as it is clear that it only applies to the deprecated strftime mode. I think we should kee

Re: [FFmpeg-devel] [PATCH]lavf/mpeg: Initialize a stack variable

2017-02-20 Thread Carl Eugen Hoyos
2017-02-21 2:09 GMT+01:00 Michael Niedermayer : > On Sun, Feb 19, 2017 at 04:18:16PM +0100, Carl Eugen Hoyos wrote: >> Hi! >> >> I cannot reproduce ticket #6160 and I wonder how memcmp() must be implemented >> to reproduce it but attached patch is supposed to fix it. >> >> Please comment, Carl Euge

Re: [FFmpeg-devel] [PATCH]lavf/mpeg: Initialize a stack variable

2017-02-20 Thread Michael Niedermayer
On Sun, Feb 19, 2017 at 04:18:16PM +0100, Carl Eugen Hoyos wrote: > Hi! > > I cannot reproduce ticket #6160 and I wonder how memcmp() must be implemented > to reproduce it but attached patch is supposed to fix it. > > Please comment, Carl Eugen > mpeg.c |2 +- > 1 file changed, 1 insertion

[FFmpeg-devel] [PATCH]lavd/opengl: Support BGR48

2017-02-20 Thread Carl Eugen Hoyos
Hi! Will apply if I there are no objections. Thank you, Carl Eugen From 2ebd6b283c5c881bf57fa3b0fe7d352ef447f117 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Tue, 21 Feb 2017 02:06:06 +0100 Subject: [PATCH] lavd/opengl_enc: Support BGR48. --- libavdevice/opengl_enc.c |1 + 1 file

[FFmpeg-devel] [PATCH] add locale month names to av_small_strptime

2017-02-20 Thread Micah Galizia
Signed-off-by: Micah Galizia --- libavutil/parseutils.c | 28 libavutil/tests/parseutils.c | 7 +++ tests/ref/fate/parseutils| 7 +++ 3 files changed, 42 insertions(+) diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c index 86d3dac..7ca

Re: [FFmpeg-devel] [PATCH] add locale month names to av_small_strptime

2017-02-20 Thread Micah Galizia
Sorry about that -- this actually makes sense now -- I thought it was a bit odd that the test program had no asserts or anything to validate the output... I've addressed the fate issues in this patch -- thanks again for the review. ___ ffmpeg-devel mail

Re: [FFmpeg-devel] [PATCH] add locale month names to av_small_strptime

2017-02-20 Thread Michael Niedermayer
On Mon, Feb 20, 2017 at 03:30:43PM -0500, Micah Galizia wrote: > Signed-off-by: Micah Galizia > --- > libavutil/parseutils.c | 28 > libavutil/tests/parseutils.c | 7 +++ > 2 files changed, 35 insertions(+) breaks fate --- ./tests/ref/fate/parseutils 2017-

Re: [FFmpeg-devel] [PATCH] speedhq: fix decoding artifacts

2017-02-20 Thread Michael Niedermayer
On Sat, Feb 18, 2017 at 08:37:26PM +0100, Paul B Mahol wrote: > On 2/18/17, Steinar H. Gunderson wrote: > > The quantization table is stored in the natural order, but when we > > access it, we use an index that's in zigzag order, causing us to read > > the wrong value. This causes artifacts, espec

[FFmpeg-devel] [PATCH] avcodec/lzwenc: Remove padding byte when writing gif

2017-02-20 Thread contact
Will Storey will at summercat.com wrote on Mon Feb 20 22:46:34 EET 2017: No longer write a final 0x00 byte when flushing the LZW bitstream. There is no mention of doing this in GIF89a specification to do this, and decoders may not expect it. Adding this byte for padding was added in commit b4

[FFmpeg-devel] [PATCH] avcodec: add ScreenPressor decoder

2017-02-20 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 + libavcodec/scpr.c | 799 libavformat/riff.c | 1 + 6 files changed, 810

[FFmpeg-devel] [PATCH] avcodec/lzwenc: Remove padding byte when writing gif

2017-02-20 Thread Will Storey
No longer write a final 0x00 byte when flushing the LZW bitstream. There is no mention of doing this in GIF89a specification to do this, and decoders may not expect it. Adding this byte for padding was added in commit b4e2e03709996a0836f6a71535d48b50201338eb apparently to resolve an issue with GIM

[FFmpeg-devel] [PATCH] add locale month names to av_small_strptime

2017-02-20 Thread Micah Galizia
Signed-off-by: Micah Galizia --- libavutil/parseutils.c | 28 libavutil/tests/parseutils.c | 7 +++ 2 files changed, 35 insertions(+) diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c index 86d3dac..7ca07b3 100644 --- a/libavutil/parseutils.c ++

Re: [FFmpeg-devel] [PATCHv2] avcodec/utils: do not reallocate packet buffer for AV_CODEC_ID_WRAPPED_AVFRAME

2017-02-20 Thread Marton Balint
On Sun, 19 Feb 2017, Hendrik Leppkes wrote: On Sun, Feb 19, 2017 at 2:41 PM, wm4 wrote: On Sun, 19 Feb 2017 14:35:42 +0100 Marton Balint wrote: Reallocating a wrapped avframe invalidates internal pointers, such as extended data. FFmpeg has another way of passing AVFrames to muxers, but it

Re: [FFmpeg-devel] [PATCH] fate/source: Check for cases that could use av_clip_uintp2() and av_clip_intp2()

2017-02-20 Thread Michael Niedermayer
On Mon, Feb 20, 2017 at 12:34:36PM +0100, Clément Bœsch wrote: > On Fri, Feb 17, 2017 at 01:59:08AM +0100, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > tests/fate/source-check.sh | 14 ++ > > tests/ref/fate/source | 2 ++ > > 2 files changed, 16

Re: [FFmpeg-devel] [PATCH 1/4] vp8: Add hwaccel hooks

2017-02-20 Thread Michael Niedermayer
On Mon, Feb 20, 2017 at 12:28:55PM +0100, wm4 wrote: > On Mon, 20 Feb 2017 11:20:48 + > Mark Thompson wrote: > > > On 20/02/17 10:44, Michael Niedermayer wrote: > > > On Mon, Feb 20, 2017 at 10:23:27AM +, Mark Thompson wrote: > > >> On 20/02/17 02:35, Michael Niedermayer wrote: > > >>

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Correct keyframe search in edit list to return the very first keyframe/frame with matching timestamp. Fixes ticket#5904

2017-02-20 Thread Michael Niedermayer
On Wed, Feb 15, 2017 at 12:07:55PM -0800, Sasi Inguva wrote: > Signed-off-by: Sasi Inguva > --- > libavformat/mov.c | 12 > 1 file changed, 12 insertions(+) applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The worst form of inequality

Re: [FFmpeg-devel] [PATCH 0/6] avx functions for h264 chroma deblocking

2017-02-20 Thread James Almer
On 2/20/2017 12:33 PM, James Darnley wrote: > 6 more functions which eke out a little more speed. > > James Darnley (6): > avcodec/h264: add avx 8-bit chroma v deblock/loop filter > avcodec/h264: add avx 8-bit 4:2:0 chroma h deblock/loop filter > avcodec/h264: add avx 8-bit 4:2:2 chroma h de

[FFmpeg-devel] [PATCH 3/6] avcodec/h264: add avx 8-bit 4:2:2 chroma h deblock/loop filter

2017-02-20 Thread James Darnley
~1.21x faster (68 vs. 56 cycles) compared with mmxext function --- libavcodec/x86/h264_deblock.asm | 27 +++ libavcodec/x86/h264dsp_init.c | 2 ++ 2 files changed, 29 insertions(+) diff --git a/libavcodec/x86/h264_deblock.asm b/libavcodec/x86/h264_deblock.asm index 0465

[FFmpeg-devel] [PATCH 1/6] avcodec/h264: add avx 8-bit chroma v deblock/loop filter

2017-02-20 Thread James Darnley
~1.24x faster (101 vs. 81 cycles) compared with mmxext function --- libavcodec/x86/h264_deblock.asm | 38 ++ libavcodec/x86/h264dsp_init.c | 2 ++ 2 files changed, 40 insertions(+) diff --git a/libavcodec/x86/h264_deblock.asm b/libavcodec/x86/h264_deblock.as

[FFmpeg-devel] [PATCH 5/6] avcodec/h264: add avx 8-bit 4:2:0 chroma h intra deblock/loop filter

2017-02-20 Thread James Darnley
~1.10x faster (69 vs. 63 cycles) compared to mmxext function --- libavcodec/x86/h264_deblock.asm | 9 + libavcodec/x86/h264dsp_init.c | 1 + 2 files changed, 10 insertions(+) diff --git a/libavcodec/x86/h264_deblock.asm b/libavcodec/x86/h264_deblock.asm index 1e6d822..2197608 100644 ---

[FFmpeg-devel] [PATCH 2/6] avcodec/h264: add avx 8-bit 4:2:0 chroma h deblock/loop filter

2017-02-20 Thread James Darnley
~1.14x faster (93 vs. 81 cycles) compared with mmxext function --- libavcodec/x86/h264_deblock.asm | 70 + libavcodec/x86/h264dsp_init.c | 3 ++ 2 files changed, 73 insertions(+) diff --git a/libavcodec/x86/h264_deblock.asm b/libavcodec/x86/h264_deblock.

[FFmpeg-devel] [PATCH 6/6] avcodec/h264: add avx 8-bit 4:2:2 chroma h intra deblock/loop filter

2017-02-20 Thread James Darnley
~1.37x faster (147 vs. 108 cycles) compared to mmxext function --- libavcodec/x86/h264_deblock.asm | 18 ++ libavcodec/x86/h264dsp_init.c | 1 + 2 files changed, 19 insertions(+) diff --git a/libavcodec/x86/h264_deblock.asm b/libavcodec/x86/h264_deblock.asm index 2197608..32aa3

[FFmpeg-devel] [PATCH 4/6] avcodec/h264: add avx 8-bit chroma v intra deblock/loop filter

2017-02-20 Thread James Darnley
~1.14x faster (90 vs 78 cycles) compared with mmxext --- libavcodec/x86/h264_deblock.asm | 33 + libavcodec/x86/h264dsp_init.c | 1 + 2 files changed, 34 insertions(+) diff --git a/libavcodec/x86/h264_deblock.asm b/libavcodec/x86/h264_deblock.asm index e2eb002..

[FFmpeg-devel] [PATCH 0/6] avx functions for h264 chroma deblocking

2017-02-20 Thread James Darnley
6 more functions which eke out a little more speed. James Darnley (6): avcodec/h264: add avx 8-bit chroma v deblock/loop filter avcodec/h264: add avx 8-bit 4:2:0 chroma h deblock/loop filter avcodec/h264: add avx 8-bit 4:2:2 chroma h deblock/loop filter avcodec/h264: add avx 8-bit chroma v

Re: [FFmpeg-devel] [PATCH v2 1/3] ffmpeg_opt: pass output framerate as a hint to the encoder

2017-02-20 Thread Tobias Rapp
On 20.02.2017 15:09, Mark Thompson wrote: On 06/02/17 12:33, Tobias Rapp wrote: Sets framerate field in output codec context if explicitly specified on the command-line. Signed-off-by: Tobias Rapp --- ffmpeg_opt.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ffmpeg_opt.c b/ffmpeg_opt

Re: [FFmpeg-devel] [PATCH v2 1/3] ffmpeg_opt: pass output framerate as a hint to the encoder

2017-02-20 Thread Mark Thompson
On 06/02/17 12:33, Tobias Rapp wrote: > Sets framerate field in output codec context if explicitly specified on > the command-line. > > Signed-off-by: Tobias Rapp > --- > ffmpeg_opt.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c > index 6a47d32..3b532da

Re: [FFmpeg-devel] [PATCH v2 1/3] ffmpeg_opt: pass output framerate as a hint to the encoder

2017-02-20 Thread Tobias Rapp
On 13.02.2017 08:51, Tobias Rapp wrote: On 06.02.2017 13:33, Tobias Rapp wrote: Sets framerate field in output codec context if explicitly specified on the command-line. Signed-off-by: Tobias Rapp --- ffmpeg_opt.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ffmpeg_opt.c b/ffmpeg_opt

Re: [FFmpeg-devel] [PATCH 1/4] vp8: Add hwaccel hooks

2017-02-20 Thread wm4
On Mon, 20 Feb 2017 11:20:48 + Mark Thompson wrote: > On 20/02/17 10:44, Michael Niedermayer wrote: > > On Mon, Feb 20, 2017 at 10:23:27AM +, Mark Thompson wrote: > >> On 20/02/17 02:35, Michael Niedermayer wrote: > >>> On Sun, Feb 19, 2017 at 09:29:33PM +, Mark Thompson wrote:

Re: [FFmpeg-devel] [PATCH] fate/source: Check for cases that could use av_clip_uintp2() and av_clip_intp2()

2017-02-20 Thread Clément Bœsch
On Fri, Feb 17, 2017 at 01:59:08AM +0100, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > tests/fate/source-check.sh | 14 ++ > tests/ref/fate/source | 2 ++ > 2 files changed, 16 insertions(+) > > diff --git a/tests/fate/source-check.sh b/tests/fate/sou

Re: [FFmpeg-devel] [PATCH 1/4] vp8: Add hwaccel hooks

2017-02-20 Thread Mark Thompson
On 20/02/17 10:44, Michael Niedermayer wrote: > On Mon, Feb 20, 2017 at 10:23:27AM +, Mark Thompson wrote: >> On 20/02/17 02:35, Michael Niedermayer wrote: >>> On Sun, Feb 19, 2017 at 09:29:33PM +, Mark Thompson wrote: On 19/02/17 21:04, Ronald S. Bultje wrote: > Hi, > > On

Re: [FFmpeg-devel] [PATCH 1/4] vp8: Add hwaccel hooks

2017-02-20 Thread Hendrik Leppkes
On Mon, Feb 20, 2017 at 11:44 AM, Michael Niedermayer wrote: > On Mon, Feb 20, 2017 at 10:23:27AM +, Mark Thompson wrote: >> On 20/02/17 02:35, Michael Niedermayer wrote: >> > On Sun, Feb 19, 2017 at 09:29:33PM +, Mark Thompson wrote: >> >> On 19/02/17 21:04, Ronald S. Bultje wrote: >> >>>

Re: [FFmpeg-devel] [PATCH] add locale month names to av_small_strptime

2017-02-20 Thread Nicolas George
Le primidi 1er ventôse, an CCXXV, Micah Galizia a écrit : > +static const char *mo_abr[] = { "jan", "feb", "mar", "apr", "may", "jun", > + "jul", "aug", "sep", "oct", "nov", "dec" }; > + > +static const char *mo_full[] = { "uary", "ruary", "ch", "il", NULL, "e", "y", >

Re: [FFmpeg-devel] [PATCH 1/4] vp8: Add hwaccel hooks

2017-02-20 Thread Michael Niedermayer
On Mon, Feb 20, 2017 at 10:23:27AM +, Mark Thompson wrote: > On 20/02/17 02:35, Michael Niedermayer wrote: > > On Sun, Feb 19, 2017 at 09:29:33PM +, Mark Thompson wrote: > >> On 19/02/17 21:04, Ronald S. Bultje wrote: > >>> Hi, > >>> > >>> On Sun, Feb 19, 2017 at 12:23 PM, Mark Thompson wr

Re: [FFmpeg-devel] [PATCH 1/4] vp8: Add hwaccel hooks

2017-02-20 Thread Mark Thompson
On 20/02/17 02:35, Michael Niedermayer wrote: > On Sun, Feb 19, 2017 at 09:29:33PM +, Mark Thompson wrote: >> On 19/02/17 21:04, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Sun, Feb 19, 2017 at 12:23 PM, Mark Thompson wrote: >>> diff --git a/libavcodec/webp.c b/libavcodec/webp.c >>> [

[FFmpeg-devel] [PATCH] avformat/hlsenc: fix cid 1401346 Dereferencing pointer error

2017-02-20 Thread Steven Liu
check if proto is null before av_strcasecmp CID: 1401346 Signed-off-by: Steven Liu --- libavformat/hlsenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index e673f59..fad6cae 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/