Re: [FFmpeg-devel] [PATCH 3/3] avcodec/x86/lossless_videoencdsp: Fix warning: signed dword value exceeds bounds

2017-10-04 Thread Michael Niedermayer
On Sun, Oct 01, 2017 at 02:10:41AM +0200, Henrik Gramner wrote: > On Sat, Sep 30, 2017 at 12:58 AM, Michael Niedermayer > wrote: > > -andi, -2 * regsize > > +andi, -(2 * regsize) > > regsize is defined to mmsize / 2 in the relevant case so the > expression

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/x86/lossless_videoencdsp: Fix warning: signed dword value exceeds bounds

2017-09-30 Thread Henrik Gramner
On Sat, Sep 30, 2017 at 12:58 AM, Michael Niedermayer wrote: > -andi, -2 * regsize > +andi, -(2 * regsize) regsize is defined to mmsize / 2 in the relevant case so the expression resolves to -2 * 16 / 2 In nasm integers are 64-bit and / is unsigned divisio

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/x86/lossless_videoencdsp: Fix warning: signed dword value exceeds bounds

2017-09-30 Thread Michael Niedermayer
On Sat, Sep 30, 2017 at 08:50:57AM -0400, Ronald S. Bultje wrote: > Hi, > > On Fri, Sep 29, 2017 at 6:58 PM, Michael Niedermayer > wrote: > > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/x86/lossless_videoencdsp.asm | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > >

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/x86/lossless_videoencdsp: Fix warning: signed dword value exceeds bounds

2017-09-30 Thread Ronald S. Bultje
Hi, On Fri, Sep 29, 2017 at 6:58 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/x86/lossless_videoencdsp.asm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/x86/lossless_videoencdsp.asm > b/libavcodec/x86/lossless_vide

[FFmpeg-devel] [PATCH 3/3] avcodec/x86/lossless_videoencdsp: Fix warning: signed dword value exceeds bounds

2017-09-29 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/x86/lossless_videoencdsp.asm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/x86/lossless_videoencdsp.asm b/libavcodec/x86/lossless_videoencdsp.asm index a9c7a0a73c..89f20dc4e2 100644 --- a/libavcodec/x86/lossless_v