Merge "fdct4x4_test: fix unsigned overflow" into nextgenv2
This commit is contained in:
		@@ -533,10 +533,10 @@ class Trans16x16TestBase {
 | 
			
		||||
 | 
			
		||||
      for (int j = 0; j < kNumCoeffs; ++j) {
 | 
			
		||||
#if CONFIG_AOM_HIGHBITDEPTH
 | 
			
		||||
        const uint32_t diff =
 | 
			
		||||
        const int diff =
 | 
			
		||||
            bit_depth_ == AOM_BITS_8 ? dst[j] - src[j] : dst16[j] - src16[j];
 | 
			
		||||
#else
 | 
			
		||||
        const uint32_t diff = dst[j] - src[j];
 | 
			
		||||
        const int diff = dst[j] - src[j];
 | 
			
		||||
#endif  // CONFIG_AOM_HIGHBITDEPTH
 | 
			
		||||
        const uint32_t error = diff * diff;
 | 
			
		||||
        EXPECT_GE(1u, error) << "Error: 16x16 IDCT has error " << error
 | 
			
		||||
@@ -589,10 +589,10 @@ class Trans16x16TestBase {
 | 
			
		||||
 | 
			
		||||
      for (int j = 0; j < kNumCoeffs; ++j) {
 | 
			
		||||
#if CONFIG_AOM_HIGHBITDEPTH
 | 
			
		||||
        const uint32_t diff =
 | 
			
		||||
        const int diff =
 | 
			
		||||
            bit_depth_ == AOM_BITS_8 ? dst[j] - ref[j] : dst16[j] - ref16[j];
 | 
			
		||||
#else
 | 
			
		||||
        const uint32_t diff = dst[j] - ref[j];
 | 
			
		||||
        const int diff = dst[j] - ref[j];
 | 
			
		||||
#endif  // CONFIG_AOM_HIGHBITDEPTH
 | 
			
		||||
        const uint32_t error = diff * diff;
 | 
			
		||||
        EXPECT_EQ(0u, error) << "Error: 16x16 IDCT Comparison has error "
 | 
			
		||||
 
 | 
			
		||||
@@ -90,11 +90,11 @@ class TransformTestBase {
 | 
			
		||||
 | 
			
		||||
      for (int j = 0; j < num_coeffs_; ++j) {
 | 
			
		||||
#if CONFIG_AOM_HIGHBITDEPTH
 | 
			
		||||
        const uint32_t diff =
 | 
			
		||||
        const int diff =
 | 
			
		||||
            bit_depth_ == AOM_BITS_8 ? dst[j] - src[j] : dst16[j] - src16[j];
 | 
			
		||||
#else
 | 
			
		||||
        ASSERT_EQ(AOM_BITS_8, bit_depth_);
 | 
			
		||||
        const uint32_t diff = dst[j] - src[j];
 | 
			
		||||
        const int diff = dst[j] - src[j];
 | 
			
		||||
#endif
 | 
			
		||||
        const uint32_t error = diff * diff;
 | 
			
		||||
        if (max_error < error) max_error = error;
 | 
			
		||||
@@ -309,10 +309,10 @@ class TransformTestBase {
 | 
			
		||||
 | 
			
		||||
      for (int j = 0; j < num_coeffs_; ++j) {
 | 
			
		||||
#if CONFIG_AOM_HIGHBITDEPTH
 | 
			
		||||
        const uint32_t diff =
 | 
			
		||||
        const int diff =
 | 
			
		||||
            bit_depth_ == AOM_BITS_8 ? dst[j] - src[j] : dst16[j] - src16[j];
 | 
			
		||||
#else
 | 
			
		||||
        const uint32_t diff = dst[j] - src[j];
 | 
			
		||||
        const int diff = dst[j] - src[j];
 | 
			
		||||
#endif
 | 
			
		||||
        const uint32_t error = diff * diff;
 | 
			
		||||
        EXPECT_GE(static_cast<uint32_t>(limit), error)
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user