You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by jo...@apache.org on 2015/09/28 04:02:29 UTC

mesos git commit: Fixed compiler warning in values tests.

Repository: mesos
Updated Branches:
  refs/heads/master 294de9fbc -> bfeb070a2


Fixed compiler warning in values tests.


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/bfeb070a
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/bfeb070a
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/bfeb070a

Branch: refs/heads/master
Commit: bfeb070a2aef52f445eb057076d344fd184eb461
Parents: 294de9f
Author: Joris Van Remoortere <jo...@gmail.com>
Authored: Sun Sep 27 18:25:23 2015 -0700
Committer: Joris Van Remoortere <jo...@gmail.com>
Committed: Sun Sep 27 18:44:04 2015 -0700

----------------------------------------------------------------------
 src/tests/values_tests.cpp | 28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/bfeb070a/src/tests/values_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/values_tests.cpp b/src/tests/values_tests.cpp
index b8302f5..e9b1079 100644
--- a/src/tests/values_tests.cpp
+++ b/src/tests/values_tests.cpp
@@ -146,10 +146,10 @@ TEST(ValuesTest, RangesCoalesce)
 
   // Should be [1-5, 7-10].
   ASSERT_EQ(2, ranges.range_size());
-  EXPECT_EQ(1, ranges.range(0).begin());
-  EXPECT_EQ(5, ranges.range(0).end());
-  EXPECT_EQ(7, ranges.range(1).begin());
-  EXPECT_EQ(10, ranges.range(1).end());
+  EXPECT_EQ(1U, ranges.range(0).begin());
+  EXPECT_EQ(5U, ranges.range(0).end());
+  EXPECT_EQ(7U, ranges.range(1).begin());
+  EXPECT_EQ(10U, ranges.range(1).end());
 }
 
 
@@ -272,8 +272,8 @@ TEST(ValuesTest, AddRangeCoalesce)
 
   // Should be coalesced to [1-10].
   ASSERT_EQ(1, ranges.range_size());
-  EXPECT_EQ(1, ranges.range(0).begin());
-  EXPECT_EQ(10, ranges.range(0).end());
+  EXPECT_EQ(1U, ranges.range(0).begin());
+  EXPECT_EQ(10U, ranges.range(0).end());
 
   // Multiple neighboring with explicit coalesce.
   // Range [5-7].
@@ -284,8 +284,8 @@ TEST(ValuesTest, AddRangeCoalesce)
 
   // Should be coalesced to [1-10].
   ASSERT_EQ(1, ranges.range_size());
-  EXPECT_EQ(1, ranges.range(0).begin());
-  EXPECT_EQ(10, ranges.range(0).end());
+  EXPECT_EQ(1U, ranges.range(0).begin());
+  EXPECT_EQ(10U, ranges.range(0).end());
 
   // Completely subsumed.
   // Range [5-7] (as before).
@@ -296,8 +296,8 @@ TEST(ValuesTest, AddRangeCoalesce)
 
   // Should be coalesced to [1-10].
   ASSERT_EQ(1, ranges.range_size());
-  EXPECT_EQ(1, ranges.range(0).begin());
-  EXPECT_EQ(10, ranges.range(0).end());
+  EXPECT_EQ(1U, ranges.range(0).begin());
+  EXPECT_EQ(10U, ranges.range(0).end());
 
   // None overlapping.
   // Range [5-7] (as before).
@@ -308,10 +308,10 @@ TEST(ValuesTest, AddRangeCoalesce)
 
   // Should be coalesced to [1-10, 20-21].
   ASSERT_EQ(2, ranges.range_size());
-  EXPECT_EQ(1, ranges.range(0).begin());
-  EXPECT_EQ(10, ranges.range(0).end());
-  EXPECT_EQ(20, ranges.range(1).begin());
-  EXPECT_EQ(21, ranges.range(1).end());
+  EXPECT_EQ(1U, ranges.range(0).begin());
+  EXPECT_EQ(10U, ranges.range(0).end());
+  EXPECT_EQ(20U, ranges.range(1).begin());
+  EXPECT_EQ(21U, ranges.range(1).end());
 }
 
 // Test adding two ranges.