generic-library/vpx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

quantize test: promote RandRange() result to signed

Browse Source 
Avoid unsigned overflow warning:
unsigned integer overflow: 19974 - 32703 cannot be represented in type
'unsigned int'

Change-Id: Ifebee014342e4c6f3b53306c0cad6ae0b465ac12
This commit is contained in:
Johann 2017-07-19 14:33:00 -07:00
parent c782f27ead
commit af08fbb444
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
test/vp9_quantize_test.cc
View File

@ -115,7 +115,7 @@ TEST_P(VP9QuantizeTest, OperationCheck) {
// round may be up to 685 for Y values or 914 for U/V.
round_ptr[j] = rnd.RandRange(914);
// quant ranges from 1 to -32703
quant_ptr[j] = rnd.RandRange(32704) - 32703;
quant_ptr[j] = static_cast<int>(rnd.RandRange(32704)) - 32703;
// quant_shift goes up to 1 << 16.
quant_shift_ptr[j] = rnd.RandRange(16384);
// dequant maxes out at 1828 for all cases.
@ -181,7 +181,7 @@ TEST_P(VP9Quantize32Test, OperationCheck) {
for (int j = 0; j < 2; j++) {
zbin_ptr[j] = rnd.RandRange(1200);
round_ptr[j] = rnd.RandRange(914);
quant_ptr[j] = rnd.RandRange(32704) - 32703;
quant_ptr[j] = static_cast<int>(rnd.RandRange(32704)) - 32703;
quant_shift_ptr[j] = rnd.RandRange(16384);
dequant_ptr[j] = rnd.RandRange(1828);
}
@ -248,7 +248,7 @@ TEST_P(VP9QuantizeTest, EOBCheck) {
for (int j = 0; j < 2; j++) {
zbin_ptr[j] = rnd.RandRange(1200);
round_ptr[j] = rnd.RandRange(914);
quant_ptr[j] = rnd.RandRange(32704) - 32703;
quant_ptr[j] = static_cast<int>(rnd.RandRange(32704)) - 32703;
quant_shift_ptr[j] = rnd.RandRange(16384);
dequant_ptr[j] = rnd.RandRange(1828);
}
@ -315,7 +315,7 @@ TEST_P(VP9Quantize32Test, EOBCheck) {
for (int j = 0; j < 2; j++) {
zbin_ptr[j] = rnd.RandRange(1200);
round_ptr[j] = rnd.RandRange(914);
quant_ptr[j] = rnd.RandRange(32704) - 32703;
quant_ptr[j] = static_cast<int>(rnd.RandRange(32704)) - 32703;
quant_shift_ptr[j] = rnd.RandRange(16384);
dequant_ptr[j] = rnd.RandRange(1828);
}