Fix compiler warnings

renatoGarcia · Feb 27, 2025 · 82f9c22 · 82f9c22
1 parent a4285d5
commit 82f9c22
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 14 deletions.
diff --git a/icecream.hpp b/icecream.hpp
@@ -988,8 +988,6 @@ namespace icecream{ namespace detail
             , count_{static_cast<size_t>(last - first)}
         {}
 
-        BasicStringView& operator=(BasicStringView const& other) = default;
-
         auto empty() const -> bool
         {
             return this->count_ == 0;

diff --git a/tests/test_c++20.cpp b/tests/test_c++20.cpp
@@ -38,7 +38,7 @@ TEST_CASE("ranges view")
 
         auto arr = std::vector<std::pair<double, int>>{{0.1, 10}, {1.1, 11}};
         auto v0 = arr | rv::transform([](auto i){return i.second;}) | IC_V();
-        for (auto i : v0){}
+        for (auto i : v0){(void)i;}
       #if defined(ICECREAM_SOURCE_LOCATION) && defined(__clang__)
         REQUIRE(str == "ic| range_view_40:76[0]: 10\nic| range_view_40:76[1]: 11\n");
       #elif defined(ICECREAM_SOURCE_LOCATION)
@@ -55,7 +55,7 @@ TEST_CASE("ranges view")
 
         auto arr = std::vector<std::pair<double, int>>{{0.1, 10}, {1.1, 11}, {2.1, 12}};
         auto v0 = arr | rv::drop(2) | IC_V([](auto i){return i.second;});
-        for (auto i : v0){}
+        for (auto i : v0){(void)i;}
       #if defined(ICECREAM_SOURCE_LOCATION) && defined(__clang__)
         REQUIRE(str == "ic| range_view_57:72[0]: 12\n");
       #elif defined(ICECREAM_SOURCE_LOCATION)
@@ -72,7 +72,7 @@ TEST_CASE("ranges view")
 
         auto arr = std::vector<std::pair<double, int>>{{0.1, 10}, {1.1, 11}};
         auto v0 = arr | IC_FV(":#x", [](auto i){return i.second;}) | rv::drop(0); //68
-        for (auto i : v0){}
+        for (auto i : v0){(void)i;}
       #if defined(ICECREAM_SOURCE_LOCATION) && defined(__clang__)
         auto const result =
             "ic| range_view_74:66[0]: 0xa\n"
@@ -188,7 +188,7 @@ TEST_CASE("ranges view erros")
 
         auto arr = std::vector<int>{1, 2, 3};
         auto v0 = arr | rv::drop(2) | IC_FV("[]", "v");
-        for (auto i : v0){}
+        for (auto i : v0){(void)i;}
         REQUIRE(str == "ic| v[0]: \"<invalid slice formatting string>\"\n");
     }
 
@@ -199,7 +199,7 @@ TEST_CASE("ranges view erros")
 
         auto arr = std::vector<int>{1, 2, 3};
         auto v0 = arr | rv::drop(2) | IC_FV("[::-1]", "v");
-        for (auto i : v0){}
+        for (auto i : v0){(void)i;}
         REQUIRE(str == "ic| v[0]: \"<slice steps must be strictly positive>\"\n");
     }
 
@@ -210,7 +210,7 @@ TEST_CASE("ranges view erros")
 
         auto arr = std::forward_list<int>{1, 2, 3};
         auto v0 = arr | rv::drop(2) | IC_FV("[:-1:]", "v");
-        for (auto i : v0){}
+        for (auto i : v0){(void)i;}
         REQUIRE(str == "ic| v[0]: \"<this range view supports only non-negative slice indexes>\"\n");
     }
 
@@ -221,7 +221,7 @@ TEST_CASE("ranges view erros")
 
         auto arr = std::vector<int>{1, 2, 3};
         auto v0 = rv::drop(2) | IC_FV("[:-1:]", "v");
-        for (auto i : arr | v0){}
+        for (auto i : arr | v0){(void)i;}
         REQUIRE(str == "ic| v[0]: \"<partial views supports only non-negative slice indexes>\"\n");
     }
 }

diff --git a/tests/test_range_v3.cpp b/tests/test_range_v3.cpp
@@ -25,7 +25,7 @@ TEST_CASE("ranges view")
         using Pair = std::pair<double, int>;
         auto arr = std::vector<Pair>{{0.1, 10}, {1.1, 11}};
         auto v0 = arr | rv::transform([](Pair const& i){return i.second;}) | IC_V();
-        for (auto i : v0){}
+        for (auto i : v0){(void)i;}
       #if defined(ICECREAM_SOURCE_LOCATION) && defined(__clang__)
         auto const result =
             "ic| range_view_27:83[0]: 10\n"
@@ -191,7 +191,7 @@ TEST_CASE("ranges view erros")
 
         auto arr = std::vector<int>{1, 2, 3};
         auto v0 = arr | rv::drop(2) | IC_FV("[]", "v");
-        for (auto i : v0){}
+        for (auto i : v0){(void)i;}
         REQUIRE(str == "ic| v[0]: \"<invalid slice formatting string>\"\n");
     }
 
@@ -202,7 +202,7 @@ TEST_CASE("ranges view erros")
 
         auto arr = std::vector<int>{1, 2, 3};
         auto v0 = arr | rv::drop(2) | IC_FV("[::-1]", "v");
-        for (auto i : v0){}
+        for (auto i : v0){(void)i;}
         REQUIRE(str == "ic| v[0]: \"<slice steps must be strictly positive>\"\n");
     }
 
@@ -213,7 +213,7 @@ TEST_CASE("ranges view erros")
 
         auto arr = std::forward_list<int>{1, 2, 3};
         auto v0 = arr | rv::drop(2) | IC_FV("[:-1:]", "v");
-        for (auto i : v0){}
+        for (auto i : v0){(void)i;}
         REQUIRE(str == "ic| v[0]: \"<this range view supports only non-negative slice indexes>\"\n");
     }
 
@@ -224,7 +224,7 @@ TEST_CASE("ranges view erros")
 
         auto arr = std::vector<int>{1, 2, 3};
         auto v0 = rv::drop(2) | IC_FV("[:-1:]", "v");
-        for (auto i : arr | v0){}
+        for (auto i : arr | v0){(void)i;}
         REQUIRE(str == "ic| v[0]: \"<partial views supports only non-negative slice indexes>\"\n");
     }
 }