diff options
author | Martin Storsjö <martin@martin.st> | 2016-06-27 22:24:15 +0300 |
---|---|---|
committer | Martin Storsjö <martin@martin.st> | 2016-06-28 14:24:04 +0300 |
commit | e48746deec48e9ff195841bc3266b4e153a878cd (patch) | |
tree | f8470f11f24df9a7782a059819745e343c0b9189 | |
parent | e57de6faa1e796099091c9af947d4755edacccaf (diff) | |
download | ffmpeg-streaming-e48746deec48e9ff195841bc3266b4e153a878cd.zip ffmpeg-streaming-e48746deec48e9ff195841bc3266b4e153a878cd.tar.gz |
checkasm: h264dsp: Move the x and y variables into the randomize_buffer macro
This avoids the risk of accidentally clobbering such variables outside
of the macro if the same variables are used there.
Signed-off-by: Martin Storsjö <martin@martin.st>
-rw-r--r-- | tests/checkasm/h264dsp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/checkasm/h264dsp.c b/tests/checkasm/h264dsp.c index c3065d2..c9ddd52 100644 --- a/tests/checkasm/h264dsp.c +++ b/tests/checkasm/h264dsp.c @@ -34,6 +34,7 @@ static const uint32_t pixel_mask[3] = { 0xffffffff, 0x01ff01ff, 0x03ff03ff }; #define randomize_buffers() \ do { \ + int x, y; \ uint32_t mask = pixel_mask[bit_depth - 8]; \ for (y = 0; y < sz; y++) { \ for (x = 0; x < PIXEL_STRIDE; x += 4) { \ @@ -178,8 +179,7 @@ static void check_idct(void) LOCAL_ALIGNED_16(int16_t, subcoef0, [8 * 8 * 2]); LOCAL_ALIGNED_16(int16_t, subcoef1, [8 * 8 * 2]); H264DSPContext h; - int bit_depth, sz, align; - int x, y, dc; + int bit_depth, sz, align, dc; declare_func_emms(AV_CPU_FLAG_MMX, void, uint8_t *dst, int16_t *block, int stride); for (bit_depth = 8; bit_depth <= 10; bit_depth++) { |