Don't compare a bool to an unsigned int constant, but compare to proper bool constants instead.
Showing the changes to test/decoder/DecUT_DecExt.cpp (1 of 1 file). Show all
Unified · Split
| @@ -622,7 +622,7 @@ void DecoderInterfaceTest::TestGetDecSarInfo() { · 621 unmodified lines | |||
| 622 | m_pDec->GetOption (DECODER_OPTION_GET_SAR_INFO, &sVuiSarInfo); | 622 | m_pDec->GetOption (DECODER_OPTION_GET_SAR_INFO, &sVuiSarInfo); |
| 623 | EXPECT_EQ (0u, sVuiSarInfo.uiSarWidth); | 623 | EXPECT_EQ (0u, sVuiSarInfo.uiSarWidth); |
| 624 | EXPECT_EQ (0u, sVuiSarInfo.uiSarHeight); | 624 | EXPECT_EQ (0u, sVuiSarInfo.uiSarHeight); |
| 625 | EXPECT_EQ (0u, sVuiSarInfo.bOverscanAppropriateFlag); | 625 | EXPECT_EQ (sVuiSarInfo.bOverscanAppropriateFlag, false); // EXPECT_EQ does not like "false" as its first arg |
| 626 | // setoption not support, | 626 | // setoption not support, |
| 627 | eRet = (CM_RETURN)m_pDec->SetOption (DECODER_OPTION_GET_SAR_INFO, NULL); | 627 | eRet = (CM_RETURN)m_pDec->SetOption (DECODER_OPTION_GET_SAR_INFO, NULL); |
| 628 | EXPECT_EQ (eRet, cmInitParaError); | 628 | EXPECT_EQ (eRet, cmInitParaError); |
| @@ -632,7 +632,7 @@ void DecoderInterfaceTest::TestGetDecSarInfo() { · 3 unmodified lines | |||
| 632 | m_pDec->GetOption (DECODER_OPTION_GET_SAR_INFO, &sVuiSarInfo); | 632 | m_pDec->GetOption (DECODER_OPTION_GET_SAR_INFO, &sVuiSarInfo); |
| 633 | EXPECT_EQ (80u, sVuiSarInfo.uiSarWidth); //DO NOT MODIFY the data value | 633 | EXPECT_EQ (80u, sVuiSarInfo.uiSarWidth); //DO NOT MODIFY the data value |
| 634 | EXPECT_EQ (33u, sVuiSarInfo.uiSarHeight); //DO NOT MODIFY the data value | 634 | EXPECT_EQ (33u, sVuiSarInfo.uiSarHeight); //DO NOT MODIFY the data value |
| 635 | EXPECT_EQ (1u, sVuiSarInfo.bOverscanAppropriateFlag); //DO NOT MODIFY the data value | 635 | EXPECT_EQ (true, sVuiSarInfo.bOverscanAppropriateFlag); //DO NOT MODIFY the data value |
| 636 | Uninit(); | 636 | Uninit(); |
| 637 | } | 637 | } |
| 638 | 638 | ||