chore: silence a few warnings

-Wdynamic-class-memaccess and -Wunused-lambda-capture
zhihaoy · Dec 9, 2024 · ac1e077 · ac1e077
1 parent 7a08778
commit ac1e077
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 5 deletions.
diff --git a/include/std23/function.h b/include/std23/function.h
@@ -314,7 +314,7 @@ template<class S, class R, class... Args> class function<S, R(Args...)>
     explicit operator bool() const noexcept
     {
         constexpr empty_target_object null;
-        return __builtin_memcmp(storage_, &null, sizeof(void *)) != 0;
+        return __builtin_memcmp(storage_, (void *)&null, sizeof(void *)) != 0;
     }
 
     friend bool operator==(function const &f, nullptr_t) noexcept { return !f; }

diff --git a/tests/function/test_ctad.cpp b/tests/function/test_ctad.cpp
@@ -7,13 +7,14 @@ inline constexpr bool deduction_enabled = type_traits::is_valid<T>(
 void test_ctad()
 {
     {
-        function fn = [i = 0] { return 0; };
+        function fn = [i = 0] { return i; };
         static_assert(std::is_same_v<decltype(fn), function<int()>>);
         static_assert(std::is_same_v<decltype(fn)::result_type, int>);
     }
 
     {
-        function fn = [i = 0](int &&, int const) mutable { return "lit"; };
+        function fn = [i = 0](int &&, int const) mutable
+        { return (void)i, "lit"; };
         static_assert(
             std::is_same_v<decltype(fn), function<char const *(int &&, int)>>);
         static_assert(std::is_same_v<decltype(fn)::result_type, char const *>);
@@ -27,7 +28,7 @@ void test_ctad()
     }
 
     {
-        function fn = [i = 0]() noexcept { return 0; };
+        function fn = [i = 0]() noexcept { return i; };
         static_assert(std::is_same_v<decltype(fn), function<int()>>,
                       "[func.wrap.func.con]/16");
         static_assert(std::is_same_v<decltype(fn)::result_type, int>);

diff --git a/tests/function_ref/test_call_pattern.cpp b/tests/function_ref/test_call_pattern.cpp
@@ -67,7 +67,7 @@ suite call_pattern = []
             {
                 expect(fr() == some_str);
 
-                auto fn = [i = 0] { return some_str.data(); };
+                auto fn = [i = 0] { return (void)i, some_str.data(); };
                 fr = decltype(fr)(fn);
 
                 expect(fr() == some_str);

diff --git a/tests/move_only_function/CMakeLists.txt b/tests/move_only_function/CMakeLists.txt
@@ -15,6 +15,7 @@ target_sources(run-move_only_function PRIVATE
  "test_unique.cpp"
 )
 target_compile_options(run-move_only_function PRIVATE
+    $<$<COMPILE_LANG_AND_ID:CXX,Clang>:-Wno-self-move>
     $<$<COMPILE_LANG_AND_ID:CXX,Clang>:-fsized-deallocation>)
 target_link_libraries(run-move_only_function PRIVATE nontype_functional kris-ut)
 set_target_properties(run-move_only_function PROPERTIES OUTPUT_NAME run)