another unused variable detected by MSVC++

Author: dietmarkuehl

include/beman/execution26/detail/allocator_aware_move.hpp

@@ -19,7 +19,7 @@ template <typename T, typename Context>
  * \headerfile beman/execution26/execution.hpp <beman/execution26/execution.hpp>
  * \internal
  */
-auto allocator_aware_move(T&& obj, Context&& context) -> decltype(auto) {
+auto allocator_aware_move(T&& obj, Context&& context) noexcept -> decltype(auto) {
     if constexpr (requires { ::beman::execution26::get_allocator(::beman::execution26::get_env(context)); }) {
         if constexpr (decltype(::beman::execution26::detail::is_product_type(obj))()) {
             return obj.make_from(::beman::execution26::get_allocator(::beman::execution26::get_env(context)),

tests/beman/execution26/exec-sync-wait.test.cpp

@@ -158,8 +158,8 @@ auto test_sync_wait_receiver() -> void {
         } catch (const error& e) {
             ASSERT(e.value == 17);
         } catch (...) {
-            ASSERT(nullptr ==
-                   "unexpected exception type"); // NOLINT(cert-dcl03-c,hicpp-static-assert,misc-static-assert)
+            // NOLINTNEXTLINE(cert-dcl03-c,hicpp-static-assert,misc-static-assert)
+            ASSERT(nullptr == "unexpected exception type");
         }
     }
     {
@@ -175,8 +175,8 @@ auto test_sync_wait_receiver() -> void {
         } catch (const error& e) {
             ASSERT(e.value == 17);
         } catch (...) {
-            ASSERT(nullptr ==
-                   "unexpected exception type"); // NOLINT(cert-dcl03-c,hicpp-static-assert,misc-static-assert)
+            // NOLINTNEXTLINE(cert-dcl03-c,hicpp-static-assert,misc-static-assert)
+            ASSERT(nullptr == "unexpected exception type");
         }
     }
     {
@@ -209,18 +209,18 @@ auto test_sync_wait() -> void {
     } catch (const error& e) {
         ASSERT(e.value == 17);
     } catch (...) {
-        ASSERT(
-            nullptr ==
-            "no exception expected from sync_wait(just(...)"); // NOLINT(cert-dcl03-c,hicpp-static-assert,misc-static-assert)
+        // NOLINTBEGIN(cert-dcl03-c,hicpp-static-assert,misc-static-assert)
+        ASSERT(nullptr == "no exception expected from sync_wait(just(...)");
+        // NOLINTEND(cert-dcl03-c,hicpp-static-assert,misc-static-assert)
     }
 
     try {
         auto value{test_std::sync_wait(send_stopped())};
         ASSERT(not value);
     } catch (...) {
-        ASSERT(
-            nullptr ==
-            "no exception expected from sync_wait(just(...)"); // NOLINT(cert-dcl03-c,hicpp-static-assert,misc-static-assert)
+        // NOLINTBEGIN(cert-dcl03-c,hicpp-static-assert,misc-static-assert)
+        ASSERT(nullptr == "no exception expected from sync_wait(just(...)");
+        // NOLINTEND(cert-dcl03-c,hicpp-static-assert,misc-static-assert)
     }
 }
 } // namespace

tests/beman/execution26/stoptoken-mem.test.cpp

@@ -96,7 +96,7 @@ TEST(stoptoken_mem) {
         test_stop_token_swap();
         test_stop_token_stop_requested();
         test_stop_token_stop_possible();
-    } catch (const std::exception& e) {
+    } catch (...) {
         // NOLINTBEGIN(cert-dcl03-c,hicpp-static-assert,misc-static-assert)
         ASSERT(nullptr == "the stop token tests are not expected to throw");
         // NOLINTEND(cert-dcl03-c,hicpp-static-assert,misc-static-assert)