Re: [FFmpeg-devel] [PATCH]configure: Add -Wno-main

2017-09-17 Thread Clément Bœsch
On Mon, Sep 18, 2017 at 03:55:12AM +0200, Carl Eugen Hoyos wrote: > 2017-09-18 3:47 GMT+02:00 James Almer : > > On 9/17/2017 10:37 PM, Carl Eugen Hoyos wrote: > >> Hi! > >> > >> Attached patch fixes several warnings when compiling libavfilter with > >> current gcc. > >> > >> Please comment, Carl Eu

Re: [FFmpeg-devel] [PATCH 3/3] avfilter/interlace: add support for 10 and 12 bit

2017-09-17 Thread James Almer
On 9/14/2017 5:58 PM, Thomas Mundt wrote: > Patch attached > Applied the addition of the two lowpass filtering tests only for now. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] Configure Script change request to support libcdio on macOS

2017-09-17 Thread 桃源老師
Hello, > 2017/09/17 午後11:58、桃源老師 のメール: > > I have tried to use libcdio function with ffmpeg on macOS. Then I get an > information from libcdio team. That is ffmpeg configure script should read > libcdio's pkgconfig file, libcdio.pc. I attach my request as patch file. Please review it. ff

[FFmpeg-devel] [PATCH 1/2] avformat/hlsenc: fix CID 1418106

2017-09-17 Thread Steven Liu
fix the "Uninitialized scalar variable (UNINIT)" problem. Signed-off-by: Steven Liu --- libavformat/hlsenc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 3a9a235514..d70a2f7ff6 100644 --- a/libavformat/hlsenc.c +++ b/

[FFmpeg-devel] [PATCH 2/2] MAINTAINERS: modify the hlsenc description

2017-09-17 Thread Steven Liu
change the hlsenc from hls encryption to hlsenc Suggested-by: Aman Gupta Signed-off-by: Steven Liu --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 0b0f7fa1e4..81e1e4916a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -409,7 +409

Re: [FFmpeg-devel] [PATCH]configure: Add -Wno-main

2017-09-17 Thread Carl Eugen Hoyos
2017-09-18 3:47 GMT+02:00 James Almer : > On 9/17/2017 10:37 PM, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes several warnings when compiling libavfilter with >> current gcc. >> >> Please comment, Carl Eugen > > IMO, it would be better if we instead rename all the cases of "main" > use

[FFmpeg-devel] [PATCH] x86/exrdsp: optimize ff_reorder_pixels_avx2()

2017-09-17 Thread James Almer
From: Henrik Gramner Tested with "checkasm --test=exrdsp -bench" Before: reorder_pixels_c: 5187.8 reorder_pixels_sse2: 377.0 reorder_pixels_avx2: 331.3 After: reorder_pixels_c: 5181.5 reorder_pixels_sse2: 377.0 reorder_pixels_avx2: 313.8 Signed-off-by: James Almer --- libavcodec/x86/exrdsp.a

Re: [FFmpeg-devel] [PATCH]configure: Add -Wno-main

2017-09-17 Thread James Almer
On 9/17/2017 10:37 PM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes several warnings when compiling libavfilter with > current gcc. > > Please comment, Carl Eugen IMO, it would be better if we instead rename all the cases of "main" used across the codebase. I'm surprised for that matte

[FFmpeg-devel] [PATCH]configure: Add -Wno-main

2017-09-17 Thread Carl Eugen Hoyos
Hi! Attached patch fixes several warnings when compiling libavfilter with current gcc. Please comment, Carl Eugen From 2d2893e8f7406043f970ae347509c39832c97b50 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon, 18 Sep 2017 03:35:37 +0200 Subject: [PATCH] configure: Add -Wno-main. Silenc

Re: [FFmpeg-devel] [PATCH] avfilter: add vmafmotion filter

2017-09-17 Thread James Almer
On 9/15/2017 5:47 PM, Ashish Pratap Singh wrote: > From: Ashish Singh > > Hi, this patch addresses the previous issues and changes it to a single > input filter. > > Signed-off-by: Ashish Singh > --- > Changelog | 1 + > doc/filters.texi| 14 ++ > libavfilter/M

[FFmpeg-devel] [PATCH] avcodec/dirac_dwt: Fix integer overflow in COMPOSE_FIDELITYi*()

2017-09-17 Thread Michael Niedermayer
Fixes: runtime error: signed integer overflow: 161 * 13872281 cannot be represented in type 'int' Fixes: 3295/clusterfuzz-testcase-minimized-4738998142500864 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer -

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wmv2dec: Check end of bitstream in parse_mb_skip() and ff_wmv2_decode_mb()

2017-09-17 Thread Michael Niedermayer
Hi On Sun, Sep 17, 2017 at 04:16:13PM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Sep 16, 2017 at 8:42 PM, Michael Niedermayer > wrote: > > > +if (get_bits_left(&s->gb) < 0) { > > +av_log(s->avctx, AV_LOG_ERROR, > > +"Insufficient bits left at %d %d\n", s->mb_x,

[FFmpeg-devel] [PATCH]lavf/utils: Do not force the chapter end time before chapter start

2017-09-17 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #6671. Please comment, Carl Eugen From 87d7914d0f60c26ac89fde3cf2a21c6c95c64b0f Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Sun, 17 Sep 2017 23:34:58 +0200 Subject: [PATCH] lavf/utils: Do not force chapter end time before chapter start. Fixes ticket #6

Re: [FFmpeg-devel] [PATCH] lavd: add audio packets adopting example of NewTek NDI

2017-09-17 Thread Moritz Barsnick
On Mon, Sep 04, 2017 at 17:12:45 +0300, Maksym Veremeyenko wrote: > Subject: [PATCH 2/3] lavd: add audio packets adopting example of NewTek NDI ^ doc/outdevs: > +Some NDI receiver want audio packets with 1920 samples per audio packet and > interleaved closer to video pa

Re: [FFmpeg-devel] libavcodec/exr : add X86 64 SIMD for reorder pixels (SSE and AVX2) (v4)

2017-09-17 Thread James Almer
On 9/17/2017 5:05 PM, Martin Vignali wrote: >> >> fate-exr passes on mingw-w64 as well. >> > > Thanks for testing > > Following your comments, new patch in attach > > Martin Pushed, thanks. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wmv2dec: Check end of bitstream in parse_mb_skip() and ff_wmv2_decode_mb()

2017-09-17 Thread Ronald S. Bultje
Hi, On Sat, Sep 16, 2017 at 8:42 PM, Michael Niedermayer wrote: > +if (get_bits_left(&s->gb) < 0) { > +av_log(s->avctx, AV_LOG_ERROR, > +"Insufficient bits left at %d %d\n", s->mb_x, s->mb_y); > +return AVERROR_INVALIDDATA; > +} We've talked about this b

Re: [FFmpeg-devel] libavcodec/exr : add X86 64 SIMD for reorder pixels (SSE and AVX2) (v4)

2017-09-17 Thread Martin Vignali
> > fate-exr passes on mingw-w64 as well. > Thanks for testing Following your comments, new patch in attach Martin 0001-libavcodec-exr-add-X86-64-SIMD-for-reorder_pixels.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg

Re: [FFmpeg-devel] libavcodec/exr : add X86 64 SIMD for reorder pixels (SSE and AVX2) (v4)

2017-09-17 Thread James Almer
On 9/17/2017 3:22 PM, Martin Vignali wrote: > From 338f96a7f3c0f97cfafc0deda2322695a4006b5a Mon Sep 17 00:00:00 2001 > From: Martin Vignali > Date: Sun, 17 Sep 2017 20:05:16 +0200 > Subject: [PATCH] libavcodec/exr : add X86 64 SIMD for reorder_pixels > > --- > libavcodec/Makefile | 2 +

[FFmpeg-devel] libavcodec/exr : add X86 64 SIMD for reorder pixels (SSE and AVX2) (v4)

2017-09-17 Thread Martin Vignali
Hello, Following Henrik Grammer comments new patch in attach replace int size by ptrdiff_t size I simplify the code, keeping only 1 loop (more easy to read, and doesn't have a real impact on speed) i use the SBUTTERFLY Macro for sse for avx2 i keep my previous approach Pass fate-exr tests for m

Re: [FFmpeg-devel] [PATCH] avformat/libopenmpt: Query duration and metadata after selecting subsong

2017-09-17 Thread Reino Wijnsma
On 17-9-2017 15:35, Jörn Heusipp wrote: > Duration depends on the selected subsong and thus must be queried after > selecting the subsong. There is no compelling reason to query other > metadata earlier either. > > Signed-off-by: Jörn Heusipp > --- > libavformat/libopenmpt.c | 17 +--

[FFmpeg-devel] Configure Script change request to support libcdio on macOS

2017-09-17 Thread 桃源老師
Hello, In fact, I'm not a kind of ffmpeg developer at all. While trying to link libcdio library with ffmpeg, I just happened to find out the way... So I'm not sure I can answer any question about this configure script change... I have tried to use libcdio function with ffmpeg on macOS. Then I

[FFmpeg-devel] [PATCH] avformat/libopenmpt: Query duration and metadata after selecting subsong

2017-09-17 Thread Jörn Heusipp
Duration depends on the selected subsong and thus must be queried after selecting the subsong. There is no compelling reason to query other metadata earlier either. Signed-off-by: Jörn Heusipp --- libavformat/libopenmpt.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) d