Re: [libav-devel] [PATCH 07/14] vc1: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Diego Biurrun
On Thu, Sep 29, 2016 at 02:24:32PM +0300, Martin Storsjö wrote: > On Tue, 20 Sep 2016, Diego Biurrun wrote: > > > This avoids SIMD-optimized functions having to sign-extend their > > stride argument manually to be able to do pointer arithmetic. > > > > Also rename all such parameters to "stride"

Re: [libav-devel] [PATCH 01/13] configure: check for stdatomic.h

2016-09-29 Thread Diego Biurrun
On Wed, Sep 28, 2016 at 02:12:53PM +0200, Anton Khirnov wrote: > Quoting Diego Biurrun (2016-09-08 12:34:14) > > On Wed, Sep 07, 2016 at 02:59:38PM +0200, Anton Khirnov wrote: > > > --- a/configure > > > +++ b/configure > > > @@ -4533,6 +4555,12 @@ check_header VideoDecodeAcceleration/VDADecoder.h

Re: [libav-devel] [PATCH 06/14] rv34: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Diego Biurrun
On Thu, Sep 29, 2016 at 02:21:17PM +0300, Martin Storsjö wrote: > On Tue, 20 Sep 2016, Diego Biurrun wrote: > > > This avoids SIMD-optimized functions having to sign-extend their > > stride argument manually to be able to do pointer arithmetic. > > --- > > libavcodec/arm/rv34dsp_neon.S | 4 +-- >

Re: [libav-devel] [PATCH 01/13] configure: check for stdatomic.h

2016-09-29 Thread Anton Khirnov
Quoting Diego Biurrun (2016-09-29 18:19:13) > On Wed, Sep 28, 2016 at 02:12:53PM +0200, Anton Khirnov wrote: > > Quoting Diego Biurrun (2016-09-08 12:34:14) > > > On Wed, Sep 07, 2016 at 02:59:38PM +0200, Anton Khirnov wrote: > > > > --- a/configure > > > > +++ b/configure > > > > @@ -4533,6

Re: [libav-devel] [PATCH 07/14] vc1: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Thu, 29 Sep 2016, Diego Biurrun wrote: On Thu, Sep 29, 2016 at 02:24:32PM +0300, Martin Storsjö wrote: On Tue, 20 Sep 2016, Diego Biurrun wrote: > This avoids SIMD-optimized functions having to sign-extend their > stride argument manually to be able to do pointer arithmetic. > > Also

Re: [libav-devel] [PATCH 05/14] h264chroma: Change type of stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: This avoids SIMD-optimized functions having to sign-extend their stride argument manually to be able to do pointer arithmetic. --- libavcodec/aarch64/h264chroma_init_aarch64.c | 12 ++-- libavcodec/aarch64/h264cmc_neon.S| 7 ++-

Re: [libav-devel] [PATCH 01/14] x86: fpel: Remove unnecessary sign extend

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: --- libavcodec/x86/fpel.asm | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/x86/fpel.asm b/libavcodec/x86/fpel.asm index b581471..b1be289 100644 --- a/libavcodec/x86/fpel.asm +++ b/libavcodec/x86/fpel.asm @@ -35,7 +35,6 @@ INIT_MMX mmxext

Re: [libav-devel] [PATCH 02/14] intrax8: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: ptrdiff_t is the correct type for array strides and similar. Also rename all such parameters to "stride" for consistency. --- libavcodec/intrax8.c| 8 +++--- libavcodec/intrax8dsp.c | 67 +

Re: [libav-devel] [PATCH 03/14] blockdsp: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: ptrdiff_t is the correct type for array strides and similar. --- libavcodec/blockdsp.c | 6 -- libavcodec/blockdsp.h | 3 ++- 2 files changed, 6 insertions(+), 3 deletions(-) Ok // Martin ___ libav-devel

Re: [libav-devel] [PATCH 04/14] idct: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: ptrdiff_t is the correct type for array strides and similar. --- doc/optimization.txt| 2 +- libavcodec/arm/idct.h | 13 libavcodec/arm/idctdsp_arm.S| 2 +- libavcodec/arm/idctdsp_init_arm.c | 15

Re: [libav-devel] [PATCH] x86/h264_weight: use appropriate register size for weight parameters

2016-09-29 Thread Luca Barbato
On 29/09/16 13:28, Martin Storsjö wrote: > From: Hendrik Leppkes > > This fixes decoding corruption on 64 bit windows. > --- > libavcodec/x86/h264_weight.asm | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > Looks fine.

Re: [libav-devel] [PATCH] x86/h264_weight: use appropriate register size for weight parameters

2016-09-29 Thread Anton Khirnov
Quoting Martin Storsjö (2016-09-29 13:28:46) > From: Hendrik Leppkes > > This fixes decoding corruption on 64 bit windows. > --- > libavcodec/x86/h264_weight.asm | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git

Re: [libav-devel] [PATCH 07/14] vc1: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: This avoids SIMD-optimized functions having to sign-extend their stride argument manually to be able to do pointer arithmetic. Also rename all such parameters to "stride" for consistency. --- libavcodec/arm/vc1dsp_init_neon.c | 2 +- libavcodec/vc1.c

Re: [libav-devel] [PATCH 14/14] hpeldsp: arm: Update comments left behind in 25841dfe806a13de526ae09c11149ab1f83555a8

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: --- libavcodec/arm/hpeldsp_arm.S | 16 1 file changed, 8 insertions(+), 8 deletions(-) Ok // Martin ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 12/14] mpegaudiodsp: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: ptrdiff_t is the correct type for array strides and similar. --- libavcodec/aarch64/mpegaudiodsp_neon.S | 1 - libavcodec/mpegaudiodsp.h | 16 ++-- libavcodec/mpegaudiodsp_template.c | 4 ++--

Re: [libav-devel] [PATCH 10/14] ea: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: ptrdiff_t is the correct type for array strides and similar. --- libavcodec/eacmv.c | 4 ++-- libavcodec/eaidct.c | 3 ++- libavcodec/eaidct.h | 3 ++- libavcodec/eamad.c | 4 ++-- libavcodec/eatgq.c | 6 +++--- libavcodec/eatgv.c | 2 +-

Re: [libav-devel] [PATCH 11/14] mss*: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: ptrdiff_t is the correct type for array strides and similar. --- libavcodec/mss12.c| 15 --- libavcodec/mss12.h| 6 +++--- libavcodec/mss2.c | 6 +++--- libavcodec/mss2dsp.c | 28 ++-- libavcodec/mss2dsp.h

Re: [libav-devel] [PATCH 06/14] rv34: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: This avoids SIMD-optimized functions having to sign-extend their stride argument manually to be able to do pointer arithmetic. --- libavcodec/arm/rv34dsp_neon.S | 4 +-- libavcodec/arm/rv40dsp_neon.S | 4 +-- libavcodec/rv30.c | 4 +--

Re: [libav-devel] [PATCH 08/14] hevc: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: ptrdiff_t is the correct type for array strides and similar. --- libavcodec/hevc.c | 12 ++-- libavcodec/hevc_filter.c | 4 ++-- libavcodec/hevcdsp_template.c | 36 ++-- tests/checkasm/hevc_mc.c

Re: [libav-devel] [PATCH 09/14] svq3: Change type of array stride parameters to ptrdiff_t

2016-09-29 Thread Martin Storsjö
On Tue, 20 Sep 2016, Diego Biurrun wrote: ptrdiff_t is the correct type for array strides and similar. --- libavcodec/svq3.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) Ok // Martin ___ libav-devel mailing list

[libav-devel] [PATCH] x86/h264_weight: use appropriate register size for weight parameters

2016-09-29 Thread Martin Storsjö
From: Hendrik Leppkes This fixes decoding corruption on 64 bit windows. --- libavcodec/x86/h264_weight.asm | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavcodec/x86/h264_weight.asm b/libavcodec/x86/h264_weight.asm index 9ad26de..e421ec8