]> granicus.if.org Git - libvpx/commitdiff
fix unit test failure on win32 vs2008 build
authorYaowu Xu <yaowu@google.com>
Fri, 9 Aug 2013 01:25:03 +0000 (18:25 -0700)
committerYaowu Xu <yaowu@google.com>
Fri, 9 Aug 2013 01:51:51 +0000 (18:51 -0700)
The mix use of double type and simd code caused invalid values stored
in double variables, further caused unit tests to fail. The failures
were only observed on x86-win32-vs9 build with vs2008.

Change-Id: If0131754a3bf217a5ace303b7963e8f5162c34b5

test/dct16x16_test.cc
test/fdct4x4_test.cc
test/fdct8x8_test.cc

index 079505475e8e8054b9fb923955cb3989e1a7fcef..89453bcd808de5a864646f826b04518f92c40ab4 100644 (file)
@@ -320,7 +320,7 @@ class FwdTrans16x16Test : public ::testing::TestWithParam<int> {
 TEST_P(FwdTrans16x16Test, AccuracyCheck) {
   ACMRandom rnd(ACMRandom::DeterministicSeed());
   int max_error = 0;
-  double total_error = 0;
+  int total_error = 0;
   const int count_test_block = 10000;
   for (int i = 0; i < count_test_block; ++i) {
     DECLARE_ALIGNED_ARRAY(16, int16_t, test_input_block, 256);
index 9dcc0785d7397f5efbdb972b1917a1a9b52c5ec5..39f915ca49f3b5a88952594b994d6480922d541b 100644 (file)
@@ -136,7 +136,7 @@ TEST_P(FwdTrans4x4Test, RoundTripErrorCheck) {
   ACMRandom rnd(ACMRandom::DeterministicSeed());
 
   int max_error = 0;
-  double total_error = 0;
+  int total_error = 0;
   const int count_test_block = 1000000;
   for (int i = 0; i < count_test_block; ++i) {
     DECLARE_ALIGNED_ARRAY(16, int16_t, test_input_block, 16);
index 50e2e9d4d76925d1efabe83bc2f9eb4bf8b3eaf4..81d242b4da3b40c5df31e2068b2dbbaf507f1d9d 100644 (file)
@@ -148,7 +148,7 @@ TEST_P(FwdTrans8x8Test, SignBiasCheck) {
 TEST_P(FwdTrans8x8Test, RoundTripErrorCheck) {
   ACMRandom rnd(ACMRandom::DeterministicSeed());
   int max_error = 0;
-  double total_error = 0;
+  int total_error = 0;
   const int count_test_block = 100000;
   for (int i = 0; i < count_test_block; ++i) {
     DECLARE_ALIGNED_ARRAY(16, int16_t, test_input_block, 64);
@@ -199,7 +199,7 @@ TEST_P(FwdTrans8x8Test, RoundTripErrorCheck) {
 TEST_P(FwdTrans8x8Test, ExtremalCheck) {
   ACMRandom rnd(ACMRandom::DeterministicSeed());
   int max_error = 0;
-  double total_error = 0;
+  int total_error = 0;
   const int count_test_block = 100000;
   for (int i = 0; i < count_test_block; ++i) {
     DECLARE_ALIGNED_ARRAY(16, int16_t, test_input_block, 64);