ShuriZma
/
dolphin
mirror of https://github.com/dolphin-emu/dolphin.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #5154 from ligfx/bitutilstestsignedness 

BitUtilsTest: compare ints of the same signedness (fixes warnings)
This commit is contained in:
Matthew Parlane 2017-03-25 14:51:47 +13:00 committed by GitHub
parent c5cc781205 9234ed82da
commit d8fa465482
1 changed files with 14 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
Source/UnitTests/Common/BitUtilsTest.cpp
View File

@ -9,15 +9,15 @@
TEST(BitUtils, BitSize) TEST(BitUtils, BitSize)
{ {
EXPECT_EQ(Common::BitSize<s8>(), 8); EXPECT_EQ(Common::BitSize<s8>(), 8u);
EXPECT_EQ(Common::BitSize<s16>(), 16); EXPECT_EQ(Common::BitSize<s16>(), 16u);
EXPECT_EQ(Common::BitSize<s32>(), 32); EXPECT_EQ(Common::BitSize<s32>(), 32u);
EXPECT_EQ(Common::BitSize<s64>(), 64); EXPECT_EQ(Common::BitSize<s64>(), 64u);
EXPECT_EQ(Common::BitSize<u8>(), 8); EXPECT_EQ(Common::BitSize<u8>(), 8u);
EXPECT_EQ(Common::BitSize<u16>(), 16); EXPECT_EQ(Common::BitSize<u16>(), 16u);
EXPECT_EQ(Common::BitSize<u32>(), 32); EXPECT_EQ(Common::BitSize<u32>(), 32u);
EXPECT_EQ(Common::BitSize<u64>(), 64); EXPECT_EQ(Common::BitSize<u64>(), 64u);
} }
TEST(BitUtils, ExtractBit) TEST(BitUtils, ExtractBit)
@ -41,14 +41,14 @@ TEST(BitUtils, ExtractBits)
// mangling the template function usages. // mangling the template function usages.
constexpr s32 two_hundred_four_signed = 0b0011001100; constexpr s32 two_hundred_four_signed = 0b0011001100;
EXPECT_EQ((Common::ExtractBits<2, 3>(two_hundred_four_signed)), 3); EXPECT_EQ((Common::ExtractBits<2, 3>(two_hundred_four_signed)), 3u);
EXPECT_EQ((Common::ExtractBits<2, 7>(two_hundred_four_signed)), 51); EXPECT_EQ((Common::ExtractBits<2, 7>(two_hundred_four_signed)), 51u);
EXPECT_EQ((Common::ExtractBits<3, 6>(two_hundred_four_signed)), 9); EXPECT_EQ((Common::ExtractBits<3, 6>(two_hundred_four_signed)), 9u);
constexpr u32 two_hundred_four_unsigned = 0b0011001100; constexpr u32 two_hundred_four_unsigned = 0b0011001100;
EXPECT_EQ((Common::ExtractBits<2, 3>(two_hundred_four_unsigned)), 3); EXPECT_EQ((Common::ExtractBits<2, 3>(two_hundred_four_unsigned)), 3u);
EXPECT_EQ((Common::ExtractBits<2, 7>(two_hundred_four_unsigned)), 51); EXPECT_EQ((Common::ExtractBits<2, 7>(two_hundred_four_unsigned)), 51u);
EXPECT_EQ((Common::ExtractBits<3, 6>(two_hundred_four_unsigned)), 9); EXPECT_EQ((Common::ExtractBits<3, 6>(two_hundred_four_unsigned)), 9u);
// Ensure bit extraction remains sign-independent even when signed types are used. // Ensure bit extraction remains sign-independent even when signed types are used.
constexpr s32 negative_one = -1; constexpr s32 negative_one = -1;