diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2015-09-24 18:33:11 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2015-09-24 18:34:43 +0200 |
| commit | 5ba40c3c712fdd44b6eecd5499799fe96225a01b (patch) | |
| tree | acbf133a224f349e984483d89c2f243f2c916263 | |
| parent | 435d000eb59ab974520bcfb3e77ddfa2473331f8 (diff) | |
| download | ffmpeg-5ba40c3c712fdd44b6eecd5499799fe96225a01b.tar.gz | |
tests/checkasm/vp9dsp: Revert first hunk of bddcf758d3a68ac0bcc3bc4fc4aa7156e05245d4
The change was wrong, also add a comment explaining it
Found-by: BBB
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | tests/checkasm/vp9dsp.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tests/checkasm/vp9dsp.c b/tests/checkasm/vp9dsp.c index e94daf3518..eb9228a681 100644 --- a/tests/checkasm/vp9dsp.c +++ b/tests/checkasm/vp9dsp.c @@ -102,7 +102,9 @@ static void check_ipred(void) ((uint16_t *)buf0)[(a) + (b) * jstride] = c; \ } \ } while (0) -#define setdx(a,b,c,d) setpx(a,b,(c)-(d)+(rnd()%((d)*2+1))) + +// c can be an assignment and must not be put under () +#define setdx(a,b,c,d) setpx(a,b,c-(d)+(rnd()%((d)*2+1))) #define setsx(a,b,c,d) setdx(a,b,c,(d) << (bit_depth - 8)) #define randomize_buffers(bidx, lineoff, str) \ do { \ |