diff options
author | Hien Ho <hienho@google.com> | 2019-08-09 12:21:17 -0700 |
---|---|---|
committer | Hien Ho <hienho@google.com> | 2019-08-26 13:46:08 -0700 |
commit | ebadd5287a4456541daf36a6a718ea2467bb8c7e (patch) | |
tree | 5a812eced5cb02d13d5e1c1c52887707ce2f2614 /test/acm_random.h | |
parent | 59b7f2f36f534ec53caba6ee049bfafca02511c6 (diff) | |
download | libvpx-ebadd5287a4456541daf36a6a718ea2467bb8c7e.tar libvpx-ebadd5287a4456541daf36a6a718ea2467bb8c7e.tar.gz libvpx-ebadd5287a4456541daf36a6a718ea2467bb8c7e.tar.bz2 libvpx-ebadd5287a4456541daf36a6a718ea2467bb8c7e.zip |
test/acm_random.h: int sanitizer warning
runtime error: implicit conversion from type 'int' of value
-61240 (32-bit, signed) to type 'int16_t' (aka 'short') changed the
value to 4296 (16-bit, signed)
BUG=webm:1615
Change-Id: I213fc153f0df9ea46737a7fb98d909e670125724
Diffstat (limited to 'test/acm_random.h')
-rw-r--r-- | test/acm_random.h | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/test/acm_random.h b/test/acm_random.h index ccfa20681..3458340a1 100644 --- a/test/acm_random.h +++ b/test/acm_random.h @@ -42,8 +42,7 @@ class ACMRandom { int16_t Rand16Signed(void) { // Use 16 bits: values between 32767 and -32768. - const uint32_t value = random_.Generate(65536); - return static_cast<int16_t>(value) - 32768; + return static_cast<int16_t>(random_.Generate(65536)); } int16_t Rand13Signed(void) { @@ -69,7 +68,7 @@ class ACMRandom { // Returns a random value near 0 or near 255, to better exercise // saturation behavior. const uint8_t r = Rand8(); - return r < 128 ? r << 4 : r >> 4; + return static_cast<uint8_t>((r < 128) ? r << 4 : r >> 4); } uint32_t RandRange(const uint32_t range) { |