Re: [FFmpeg-devel] [PATCH] lavc/x86: clear r2 higher bits in ff_sbr_sum_square

2017-06-23 Thread James Almer
On 6/23/2017 11:56 AM, Matthieu Bouron wrote: > Suggested-by: James Almer > --- > libavcodec/x86/sbrdsp.asm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/x86/sbrdsp.asm b/libavcodec/x86/sbrdsp.asm > index 07a412b2ae..02b52ee300 100644 >

Re: [FFmpeg-devel] configure/libopenjpegdec.c/libopenjpegenc.c: Add support for LibOpenJPEG v2.2/git

2017-06-23 Thread Michael Bradshaw
On Fri, Jun 23, 2017 at 2:57 PM, Reino Wijnsma wrote: > > Like this? Yup, just like that. Thanks for the patch! I've applied it. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] bug in stream_loop

2017-06-23 Thread Mike Versteeg
Hi! For the last couple of days I have been trying to get FFmpeg to loop a video with audio, but both the stream_loop option and loop filter have a bug. Since stream_loop is the most efficient of the two (and almost works), that one is my favourite. Can I persuade someone to look at ticket 5719

Re: [FFmpeg-devel] [PATCH] vf_spp: only assign function pointers if permutation matches expectations.

2017-06-23 Thread Michael Niedermayer
On Fri, Jun 23, 2017 at 11:02:07AM -0400, Ronald S. Bultje wrote: > --- > libavfilter/x86/vf_spp.c | 8 +++- > 1 file changed, 7 insertions(+), 1 deletion(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I have never wished to cater to the

Re: [FFmpeg-devel] [PATCH] avcodec/g722enc: force mono channel layout

2017-06-23 Thread Kyle Swanson
On Fri, Jun 23, 2017 at 5:46 PM, Paul B Mahol wrote: > On 6/23/17, Kyle Swanson wrote: >> Signed-off-by: Kyle Swanson >> --- >> libavcodec/g722enc.c | 27 +++ >> 1 file changed, 11 insertions(+), 16 deletions(-) >> > > Should

Re: [FFmpeg-devel] [PATCH v5] mdct15: add assembly optimizations for the 15-point FFT

2017-06-23 Thread Rostislav Pehlivanov
On 23 June 2017 at 23:45, Michael Niedermayer wrote: > On Fri, Jun 23, 2017 at 10:14:25PM +0100, Rostislav Pehlivanov wrote: > > c:1802 decicycles in fft15,16774635 runs, 2581 skips > > avx: 865 decicycles in fft15,16776378 runs,838 skips > > > >

Re: [FFmpeg-devel] [PATCH] avcodec/g722enc: force mono channel layout

2017-06-23 Thread Paul B Mahol
On 6/23/17, Kyle Swanson wrote: > Signed-off-by: Kyle Swanson > --- > libavcodec/g722enc.c | 27 +++ > 1 file changed, 11 insertions(+), 16 deletions(-) > Should be fine iff other than mono is really dissalowed.

Re: [FFmpeg-devel] [PATCH v5] mdct15: add assembly optimizations for the 15-point FFT

2017-06-23 Thread Michael Niedermayer
On Fri, Jun 23, 2017 at 10:14:25PM +0100, Rostislav Pehlivanov wrote: > c:1802 decicycles in fft15,16774635 runs, 2581 skips > avx: 865 decicycles in fft15,16776378 runs,838 skips > > Signed-off-by: Rostislav Pehlivanov > --- > libavcodec/mdct15.c | 182

Re: [FFmpeg-devel] [PATCH v4] mdct15: add assembly optimizations for the 15-point FFT

2017-06-23 Thread James Almer
On 6/23/2017 7:11 PM, Michael Niedermayer wrote: > On Fri, Jun 23, 2017 at 10:35:43PM +0200, Paul B Mahol wrote: >> On 6/23/17, Michael Niedermayer wrote: >>> On Fri, Jun 23, 2017 at 02:51:37AM +0100, Rostislav Pehlivanov wrote: c:1802 decicycles in fft15,16774635

Re: [FFmpeg-devel] [PATCH v4] mdct15: add assembly optimizations for the 15-point FFT

2017-06-23 Thread Michael Niedermayer
On Fri, Jun 23, 2017 at 10:35:43PM +0200, Paul B Mahol wrote: > On 6/23/17, Michael Niedermayer wrote: > > On Fri, Jun 23, 2017 at 02:51:37AM +0100, Rostislav Pehlivanov wrote: > >> c:1802 decicycles in fft15,16774635 runs, 2581 skips > >> avx: 865 decicycles in

Re: [FFmpeg-devel] configure/libopenjpegdec.c/libopenjpegenc.c: Add support for LibOpenJPEG v2.2/git

2017-06-23 Thread Reino Wijnsma
On 22-6-2017 2:00, Michael Bradshaw wrote: > Almost done! The OPJ_STATIC change that was introduced in OpenJPEG 2.1+ > means FFmepg's configure script has to do some extra work. You'll see that > there are two check_lib calls for openjpeg-2.1. You'll need to

[FFmpeg-devel] [PATCH] avcodec/g722enc: force mono channel layout

2017-06-23 Thread Kyle Swanson
Signed-off-by: Kyle Swanson --- libavcodec/g722enc.c | 27 +++ 1 file changed, 11 insertions(+), 16 deletions(-) diff --git a/libavcodec/g722enc.c b/libavcodec/g722enc.c index 01a3db26fd..473646f45c 100644 --- a/libavcodec/g722enc.c +++

[FFmpeg-devel] [PATCH v5] mdct15: add assembly optimizations for the 15-point FFT

2017-06-23 Thread Rostislav Pehlivanov
c:1802 decicycles in fft15,16774635 runs, 2581 skips avx: 865 decicycles in fft15,16776378 runs,838 skips Signed-off-by: Rostislav Pehlivanov --- libavcodec/mdct15.c | 182 +-- libavcodec/mdct15.h | 26

Re: [FFmpeg-devel] [PATCH] configure: fix assignment of assembler specific flags

2017-06-23 Thread James Almer
On 6/23/2017 5:31 PM, Michael Niedermayer wrote: > On Fri, Jun 23, 2017 at 12:25:00PM -0300, James Almer wrote: >> If the first assembler to be probed is an old nasm build, X86ASM_DEPFLAGS >> will be set and remain so after yasm is ultimately used as fallback. >> This results in yasm being called

Re: [FFmpeg-devel] [PATCH v4] mdct15: add assembly optimizations for the 15-point FFT

2017-06-23 Thread Paul B Mahol
On 6/23/17, Michael Niedermayer wrote: > On Fri, Jun 23, 2017 at 02:51:37AM +0100, Rostislav Pehlivanov wrote: >> c:1802 decicycles in fft15,16774635 runs, 2581 skips >> avx: 865 decicycles in fft15,16776378 runs,838 skips >> >> Signed-off-by: Rostislav

Re: [FFmpeg-devel] [PATCH v4] mdct15: add assembly optimizations for the 15-point FFT

2017-06-23 Thread Henrik Gramner
On Fri, Jun 23, 2017 at 10:18 PM, Michael Niedermayer wrote: > seems to fail to build here: > > libavcodec/x86/mdct15.asm:116: error: invalid combination of opcode and > operands > libavcodec/x86/mdct15.asm:117: error: invalid combination of opcode and > operands >

Re: [FFmpeg-devel] [PATCH] configure: fix assignment of assembler specific flags

2017-06-23 Thread Michael Niedermayer
On Fri, Jun 23, 2017 at 12:25:00PM -0300, James Almer wrote: > If the first assembler to be probed is an old nasm build, X86ASM_DEPFLAGS > will be set and remain so after yasm is ultimately used as fallback. > This results in yasm being called with said nasm specific flags and failing > during

Re: [FFmpeg-devel] [PATCH v4] mdct15: add assembly optimizations for the 15-point FFT

2017-06-23 Thread Michael Niedermayer
On Fri, Jun 23, 2017 at 02:51:37AM +0100, Rostislav Pehlivanov wrote: > c:1802 decicycles in fft15,16774635 runs, 2581 skips > avx: 865 decicycles in fft15,16776378 runs,838 skips > > Signed-off-by: Rostislav Pehlivanov > --- > libavcodec/mdct15.c | 182

[FFmpeg-devel] [PATCH v2] lavu: add ff_pthread_setname() and use it in various places

2017-06-23 Thread Rostislav Pehlivanov
This is a re-posting of an old patch by Clément Bœsch . Signed-off-by: Rostislav Pehlivanov --- Changes from V1: renamve function to avpriv_thread_setname use prctl on linux (universal across all libc) configure | 2 ++

Re: [FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2017-06-23 Thread wm4
On Fri, 23 Jun 2017 18:48:04 +0100 Rostislav Pehlivanov wrote: > This is a re-posting of an old patch by Clément Bœsch , > with the addition it'll define _GNU_SOURCE in the configure system, > making the functionality usable on modern glibc systems. > >

[FFmpeg-devel] [PATCH] lavu: add ff_pthread_setname() and use it in various places

2017-06-23 Thread Rostislav Pehlivanov
This is a re-posting of an old patch by Clément Bœsch , with the addition it'll define _GNU_SOURCE in the configure system, making the functionality usable on modern glibc systems. Signed-off-by: Rostislav Pehlivanov --- configure | 2

Re: [FFmpeg-devel] [PATCH] configure: use pkg-config for libgme, if available

2017-06-23 Thread wm4
On Fri, 23 Jun 2017 01:53:37 +0100 Ricardo Constantino wrote: > The pkg-config file is relatively new (2013), so some distros might > not have it yet. And the -lstdc++ being required for the static lib > is only present since the last release in December 2016. > --- >

[FFmpeg-devel] [PATCH] configure: fix assignment of assembler specific flags

2017-06-23 Thread James Almer
If the first assembler to be probed is an old nasm build, X86ASM_DEPFLAGS will be set and remain so after yasm is ultimately used as fallback. This results in yasm being called with said nasm specific flags and failing during actual object assembly but not with configure sanity checks. Regression

Re: [FFmpeg-devel] [PATCH] vf_spp: only assign function pointers if permutation matches expectations.

2017-06-23 Thread Ronald S. Bultje
Hi, On Thu, Jun 22, 2017 at 4:28 PM, Michael Niedermayer wrote: > On Wed, Jun 21, 2017 at 09:03:11PM -0400, Ronald S. Bultje wrote: > > On Wed, Jun 21, 2017 at 8:42 PM, Michael Niedermayer > > also the memcmp() is a very ugly way to check the

[FFmpeg-devel] [PATCH] vf_spp: only assign function pointers if permutation matches expectations.

2017-06-23 Thread Ronald S. Bultje
--- libavfilter/x86/vf_spp.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/libavfilter/x86/vf_spp.c b/libavfilter/x86/vf_spp.c index 45a9eb0..7d5da61 100644 --- a/libavfilter/x86/vf_spp.c +++ b/libavfilter/x86/vf_spp.c @@ -21,6 +21,7 @@ #include

Re: [FFmpeg-devel] [PATCH 1/2] checkasm: add sbrdsp tests

2017-06-23 Thread Matthieu Bouron
On Thu, Jun 22, 2017 at 12:53:52PM -0300, James Almer wrote: > On 6/22/2017 9:56 AM, Matthieu Bouron wrote: > > --- > > > > The following patchset applies on top of Clément's aacpsdsp patchset. > > > > --- > > tests/checkasm/Makefile | 3 +- > > tests/checkasm/checkasm.c | 1 + > >

[FFmpeg-devel] [PATCH] lavc/x86: clear r2 higher bits in ff_sbr_sum_square

2017-06-23 Thread Matthieu Bouron
Suggested-by: James Almer --- libavcodec/x86/sbrdsp.asm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/x86/sbrdsp.asm b/libavcodec/x86/sbrdsp.asm index 07a412b2ae..02b52ee300 100644 --- a/libavcodec/x86/sbrdsp.asm +++ b/libavcodec/x86/sbrdsp.asm

[FFmpeg-devel] Multiprogram mode for mpeg TS

2017-06-23 Thread ffmpeg
Hi! (corrected version of the patch.) 1. adding missed but required options for libavformat/mpegts encoder in ffmpeg_opt.c. also, it makes it possible to add pcr_pid for separate program inside TS. options are: service_provider: any provider name (only default FFMPEG is present

Re: [FFmpeg-devel] videotoolboxenc.c crash on iOS8

2017-06-23 Thread Richard Kern
On June 17, 2017 at 6:46:58 PM, Steven Liu (lingjiujia...@gmail.com) wrote: 2017-06-17 11:17 GMT+08:00 姜 文杰 : > when use videotoolboxencoder in ffmpeg, some codes will lead to crash. > > ___ > ffmpeg-devel mailing list >

Re: [FFmpeg-devel] [PATCH v2 1/5] Interplay MVE: Implement MVE SEND_BUFFER operation

2017-06-23 Thread Hein-Pieter van Braam
On Wed, 2017-06-21 at 22:28 +0200, Paul B Mahol wrote: > > > Checks only for 2 bytes. > > > > > +send_buffer = AV_RL8(avpkt->data); > > + > >  /* decoding map contains 4 bits of information per 8x8 block > > */ > > -s->decoding_map_size = AV_RL16(avpkt->data); > > +

Re: [FFmpeg-devel] [PATCH V5 1/4] lavc/golomb: add value range comment for set_ue_golomb().

2017-06-23 Thread Michael Niedermayer
On Wed, Jun 14, 2017 at 11:00:50AM +0800, Jun Zhao wrote: > V5: Split the patch. > V4: Fix rang check error in assert base on Mark's review > V3: Clean the code logic base on Michael's review. > V2: Add set_ue_golomb_long() to support 32bits UE golomb and update the unit > test. > golomb.h |

Re: [FFmpeg-devel] [PATCH] Fix static linking openssl library

2017-06-23 Thread Alexandr Topilski
Hi, thank you for reply but output of pkg-config --static --libs openssl is next: $ -L/usr/local/lib -lssl -ldl -lcrypto -ldl Also i built openssl without any flags, simple execute: ./config && make && make install On Thu, Jun 22, 2017 at 11:07 AM, Moritz Barsnick wrote: >

Re: [FFmpeg-devel] [PATCH] Fix static linking openssl library

2017-06-23 Thread Alexandr Topilski
Hi, thank you for replay but output of pkg-config --static --libs openssl is next: -L/usr/local/lib -lssl -ldl -lcrypto -ldl BR, Alex On Thu, Jun 22, 2017 at 11:07 AM, Moritz Barsnick wrote: > (This really belongs to ffmpeg-user.) > > On Thu, Jun 22, 2017 at 08:14:56 +0300,

Re: [FFmpeg-devel] [PATCH] Fix static linking openssl library

2017-06-23 Thread Alexandr Topilski
Also i built openssl without any flags, simple execute: ./config && make && make install On Fri, Jun 23, 2017 at 10:19 AM, Alexandr Topilski wrote: > Hi, thank you for replay but output of pkg-config --static --libs openssl > is next: > > -L/usr/local/lib -lssl -ldl