commit 892f1f40b41ef541c106120abcd5a79975e21c66
author danzh <danzh@google.com> Fri Apr 26 08:57:56 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Fri Apr 26 08:59:23 2019 -0700
tree 36d17e95c8b4bc2869062617da900c176c561925
parent 7805361f3a79b463da1e88a9e307a2455d819a0a

gfe-relnote: n/a(code not in use) a few changes to improve SimpleLinkedHashMap's portability.

Platformize CHECK macro;
Signed integer literal to unsigned.

PiperOrigin-RevId: 245428602
Change-Id: I7aa33a0eaf20c29d55de3087990b9999a073421a

common/simple_linked_hash_map_test.cc

diff --git a/common/simple_linked_hash_map_test.cc b/common/simple_linked_hash_map_test.cc
index 00179b3..ae73aa0 100644
--- a/common/simple_linked_hash_map_test.cc
+++ b/common/simple_linked_hash_map_test.cc
@@ -182,21 +182,21 @@
 TEST(LinkedHashMapTest, Erase) {
   SimpleLinkedHashMap<int, int> m;
   ASSERT_EQ(0u, m.size());
-  EXPECT_EQ(0, m.erase(2));  // Nothing to erase yet
+  EXPECT_EQ(0u, m.erase(2));  // Nothing to erase yet
 
   m.insert(std::make_pair(2, 12));
   ASSERT_EQ(1u, m.size());
-  EXPECT_EQ(1, m.erase(2));
+  EXPECT_EQ(1u, m.erase(2));
   EXPECT_EQ(0u, m.size());
 
-  EXPECT_EQ(0, m.erase(2));  // Make sure nothing bad happens if we repeat.
+  EXPECT_EQ(0u, m.erase(2));  // Make sure nothing bad happens if we repeat.
   EXPECT_EQ(0u, m.size());
 }
 
 TEST(LinkedHashMapTest, Erase2) {
   SimpleLinkedHashMap<int, int> m;
   ASSERT_EQ(0u, m.size());
-  EXPECT_EQ(0, m.erase(2));  // Nothing to erase yet
+  EXPECT_EQ(0u, m.erase(2));  // Nothing to erase yet
 
   m.insert(std::make_pair(2, 12));
   m.insert(std::make_pair(1, 11));
@@ -205,8 +205,8 @@
   ASSERT_EQ(4u, m.size());
 
   // Erase middle two
-  EXPECT_EQ(1, m.erase(1));
-  EXPECT_EQ(1, m.erase(3));
+  EXPECT_EQ(1u, m.erase(1));
+  EXPECT_EQ(1u, m.erase(3));
 
   EXPECT_EQ(2u, m.size());
 
@@ -220,14 +220,14 @@
   ++it;
   ASSERT_TRUE(it == m.end());
 
-  EXPECT_EQ(0, m.erase(1));  // Make sure nothing bad happens if we repeat.
+  EXPECT_EQ(0u, m.erase(1));  // Make sure nothing bad happens if we repeat.
   ASSERT_EQ(2u, m.size());
 
-  EXPECT_EQ(1, m.erase(2));
-  EXPECT_EQ(1, m.erase(4));
+  EXPECT_EQ(1u, m.erase(2));
+  EXPECT_EQ(1u, m.erase(4));
   ASSERT_EQ(0u, m.size());
 
-  EXPECT_EQ(0, m.erase(1));  // Make sure nothing bad happens if we repeat.
+  EXPECT_EQ(0u, m.erase(1));  // Make sure nothing bad happens if we repeat.
   ASSERT_EQ(0u, m.size());
 }