Merge pull request #13 from ament/xenial

fix sign compare warning with newer compiler
ament · Apr 28, 2016 · affbc78 · affbc78
2 parents 6822302 + 39b552d
commit affbc78
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/ament_index_cpp/test/utest.cpp b/ament_index_cpp/test/utest.cpp
@@ -67,7 +67,7 @@ TEST(AmentIndexCpp, search_paths) {
   subfolders.push_back("prefix2");
   set_ament_prefix_path(subfolders);
   std::list<std::string> search_paths = ament_index_cpp::get_search_paths();
-  EXPECT_EQ(search_paths.size(), 2);
+  EXPECT_EQ(search_paths.size(), 2UL);
 }
 
 TEST(AmentIndexCpp, not_existing_search_paths) {
@@ -76,7 +76,7 @@ TEST(AmentIndexCpp, not_existing_search_paths) {
   subfolders.push_back("not_existing_prefix");
   set_ament_prefix_path(subfolders);
   std::list<std::string> search_paths = ament_index_cpp::get_search_paths();
-  EXPECT_EQ(search_paths.size(), 1);
+  EXPECT_EQ(search_paths.size(), 1UL);
 }
 
 TEST(AmentIndexCpp, get_unknown_resources) {
@@ -85,15 +85,15 @@ TEST(AmentIndexCpp, get_unknown_resources) {
   set_ament_prefix_path(subfolders);
   std::map<std::string, std::string> resources =
     ament_index_cpp::get_resources("unknown_resource_type");
-  EXPECT_EQ(resources.size(), 0);
+  EXPECT_EQ(resources.size(), 0UL);
 }
 
 TEST(AmentIndexCpp, get_resources) {
   std::list<std::string> subfolders;
   subfolders.push_back("prefix1");
   set_ament_prefix_path(subfolders);
   std::map<std::string, std::string> resources = ament_index_cpp::get_resources("resource_type1");
-  EXPECT_EQ(resources.size(), 2);
+  EXPECT_EQ(resources.size(), 2UL);
   for (auto it : resources) {
     EXPECT_TRUE(it.first == "foo" || it.first == "bar");
   }
@@ -105,7 +105,7 @@ TEST(AmentIndexCpp, get_resources_overlay) {
   subfolders.push_back("prefix2");
   set_ament_prefix_path(subfolders);
   std::map<std::string, std::string> resources = ament_index_cpp::get_resources("resource_type2");
-  EXPECT_EQ(resources.size(), 2);
+  EXPECT_EQ(resources.size(), 2UL);
   for (auto it : resources) {
     EXPECT_TRUE(it.first == "foo" || it.first == "bar");
   }
@@ -117,7 +117,7 @@ TEST(AmentIndexCpp, get_resources_underlay) {
   subfolders.push_back("prefix2");
   set_ament_prefix_path(subfolders);
   std::map<std::string, std::string> resources = ament_index_cpp::get_resources("resource_type3");
-  EXPECT_EQ(resources.size(), 1);
+  EXPECT_EQ(resources.size(), 1UL);
   for (auto it : resources) {
     EXPECT_EQ(it.first, "bar");
   }