Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Unified Diff: base/numerics/safe_numerics_unittest.cc

Issue 2545303002: Rename CheckNum to MakeCheckedNum (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/numerics/safe_math.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/numerics/safe_numerics_unittest.cc
diff --git a/base/numerics/safe_numerics_unittest.cc b/base/numerics/safe_numerics_unittest.cc
index 7d42a8361ec8cf801366c23ff0e452a63b32e77e..56b07427ca9b99510b284e875820976088c24b01 100644
--- a/base/numerics/safe_numerics_unittest.cc
+++ b/base/numerics/safe_numerics_unittest.cc
@@ -25,7 +25,7 @@ using base::CheckedNumeric;
using base::IsValidForType;
using base::ValueOrDieForType;
using base::ValueOrDefaultForType;
-using base::CheckNum;
+using base::MakeCheckedNum;
using base::CheckMax;
using base::CheckMin;
using base::CheckAdd;
@@ -252,7 +252,7 @@ static void TestSpecializedArithmetic(
TEST_EXPECTED_VALUE(0, CheckedNumeric<Dst>(0) & 1);
TEST_EXPECTED_VALUE(0, CheckedNumeric<Dst>(1) & 0);
TEST_EXPECTED_VALUE(std::numeric_limits<Dst>::max(),
- CheckNum(DstLimits::max()) & -1);
+ MakeCheckedNum(DstLimits::max()) & -1);
TEST_EXPECTED_VALUE(1, CheckedNumeric<Dst>(1) | 1);
TEST_EXPECTED_VALUE(1, CheckedNumeric<Dst>(1) | 0);
TEST_EXPECTED_VALUE(1, CheckedNumeric<Dst>(0) | 1);
@@ -483,18 +483,22 @@ void TestStrictComparison() {
// Due to differences in float handling between compilers, these aren't
// compile-time constants everywhere. So, we use run-time tests.
- EXPECT_EQ(SrcLimits::max(),
- CheckNum(SrcLimits::max()).Max(DstLimits::lowest()).ValueOrDie());
- EXPECT_EQ(DstLimits::max(),
- CheckNum(SrcLimits::lowest()).Max(DstLimits::max()).ValueOrDie());
- EXPECT_EQ(DstLimits::lowest(),
- CheckNum(SrcLimits::max()).Min(DstLimits::lowest()).ValueOrDie());
- EXPECT_EQ(SrcLimits::lowest(),
- CheckNum(SrcLimits::lowest()).Min(DstLimits::max()).ValueOrDie());
- EXPECT_EQ(SrcLimits::lowest(), CheckMin(MakeStrictNum(1), CheckNum(0),
+ EXPECT_EQ(
+ SrcLimits::max(),
+ MakeCheckedNum(SrcLimits::max()).Max(DstLimits::lowest()).ValueOrDie());
+ EXPECT_EQ(
+ DstLimits::max(),
+ MakeCheckedNum(SrcLimits::lowest()).Max(DstLimits::max()).ValueOrDie());
+ EXPECT_EQ(
+ DstLimits::lowest(),
+ MakeCheckedNum(SrcLimits::max()).Min(DstLimits::lowest()).ValueOrDie());
+ EXPECT_EQ(
+ SrcLimits::lowest(),
+ MakeCheckedNum(SrcLimits::lowest()).Min(DstLimits::max()).ValueOrDie());
+ EXPECT_EQ(SrcLimits::lowest(), CheckMin(MakeStrictNum(1), MakeCheckedNum(0),
DstLimits::max(), SrcLimits::lowest())
.ValueOrDie());
- EXPECT_EQ(DstLimits::max(), CheckMax(MakeStrictNum(1), CheckNum(0),
+ EXPECT_EQ(DstLimits::max(), CheckMax(MakeStrictNum(1), MakeCheckedNum(0),
DstLimits::max(), SrcLimits::lowest())
.ValueOrDie());
}
@@ -868,9 +872,9 @@ TEST(SafeNumerics, CastTests) {
EXPECT_EQ(0, saturated_cast<int>(not_a_number));
// Test the CheckedNumeric value extractions functions.
- auto int8_min = CheckNum(numeric_limits<int8_t>::min());
- auto int8_max = CheckNum(numeric_limits<int8_t>::max());
- auto double_max = CheckNum(numeric_limits<double>::max());
+ auto int8_min = MakeCheckedNum(numeric_limits<int8_t>::min());
+ auto int8_max = MakeCheckedNum(numeric_limits<int8_t>::max());
+ auto double_max = MakeCheckedNum(numeric_limits<double>::max());
static_assert(
std::is_same<int16_t,
decltype(int8_min.ValueOrDie<int16_t>())::type>::value,
@@ -1015,19 +1019,19 @@ TEST(SafeNumerics, CompoundNumericOperations) {
}
TEST(SafeNumerics, VariadicNumericOperations) {
- auto a = CheckAdd(1, 2UL, CheckNum(3LL), 4).ValueOrDie();
+ auto a = CheckAdd(1, 2UL, MakeCheckedNum(3LL), 4).ValueOrDie();
EXPECT_EQ(static_cast<decltype(a)::type>(10), a);
- auto b = CheckSub(CheckNum(20.0), 2UL, 4).ValueOrDie();
+ auto b = CheckSub(MakeCheckedNum(20.0), 2UL, 4).ValueOrDie();
EXPECT_EQ(static_cast<decltype(b)::type>(14.0), b);
- auto c = CheckMul(20.0, CheckNum(1), 5, 3UL).ValueOrDie();
+ auto c = CheckMul(20.0, MakeCheckedNum(1), 5, 3UL).ValueOrDie();
EXPECT_EQ(static_cast<decltype(c)::type>(300.0), c);
- auto d = CheckDiv(20.0, 2.0, CheckNum(5LL), -4).ValueOrDie();
+ auto d = CheckDiv(20.0, 2.0, MakeCheckedNum(5LL), -4).ValueOrDie();
EXPECT_EQ(static_cast<decltype(d)::type>(-.5), d);
- auto e = CheckMod(CheckNum(20), 3).ValueOrDie();
+ auto e = CheckMod(MakeCheckedNum(20), 3).ValueOrDie();
EXPECT_EQ(static_cast<decltype(e)::type>(2), e);
- auto f = CheckLsh(1, CheckNum(2)).ValueOrDie();
+ auto f = CheckLsh(1, MakeCheckedNum(2)).ValueOrDie();
EXPECT_EQ(static_cast<decltype(f)::type>(4), f);
- auto g = CheckRsh(4, CheckNum(2)).ValueOrDie();
+ auto g = CheckRsh(4, MakeCheckedNum(2)).ValueOrDie();
EXPECT_EQ(static_cast<decltype(g)::type>(1), g);
auto h = CheckRsh(CheckAdd(1, 1, 1, 1), CheckSub(4, 2)).ValueOrDie();
EXPECT_EQ(static_cast<decltype(h)::type>(1), h);
« no previous file with comments | « base/numerics/safe_math.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698