diff --git a/pdal/ArtifactManager.hpp b/pdal/ArtifactManager.hpp
index 5a534613d2..69e286b5b4 100644
--- a/pdal/ArtifactManager.hpp
+++ b/pdal/ArtifactManager.hpp
@@ -86,7 +86,8 @@ class ArtifactManager
     std::vector<std::string> keys() const
     {
         std::vector<std::string> ks;
-        for(auto e : m_storage) ks.push_back(e.first);
+        for (auto e : m_storage)
+            ks.push_back(e.first);
         return ks;
     }
 
diff --git a/test/unit/ArtifactTest.cpp b/test/unit/ArtifactTest.cpp
index 7611acd176..bbb8e2412b 100644
--- a/test/unit/ArtifactTest.cpp
+++ b/test/unit/ArtifactTest.cpp
@@ -130,12 +130,12 @@ TEST(ArtifactTest, key_access)
     EXPECT_TRUE(t.artifactManager().keys().empty());
 
     t.artifactManager().put("MyTest", ta);
-    EXPECT_EQ(t.artifactManager().keys().size(), 1);
+    EXPECT_EQ(t.artifactManager().keys().size(), 1U);
     EXPECT_EQ(t.artifactManager().keys().at(0), "MyTest");
 
     t.artifactManager().put("MyTest2", ta);
     auto keys = t.artifactManager().keys();
-    EXPECT_EQ(keys.size(), 2);
+    EXPECT_EQ(keys.size(), 2U);
     EXPECT_EQ(std::find(keys.begin(), keys.end(), "Foo"), keys.end());
     EXPECT_NE(std::find(keys.begin(), keys.end(), "MyTest"), keys.end());
     EXPECT_NE(std::find(keys.begin(), keys.end(), "MyTest2"), keys.end());