fixed how product_type access is done (#180)

* fixed how product_type access is done

* clang-format

Author: dietmarkuehl

include/beman/execution/detail/product_type.hpp

@@ -93,7 +93,12 @@ struct product_type : ::beman::execution::detail::product_type_base<::std::index
     }
     template <typename Fun, ::std::size_t... I>
     constexpr auto apply_elements(::std::index_sequence<I...>, Fun&& fun) -> decltype(auto) {
+#if 0
         return ::std::forward<Fun>(fun)(this->template get<I>()...);
+#else
+        //-dk:TODO provide rvalue, lvalue, const lvalue overloads?
+        return ::std::forward<Fun>(fun)(std::move(this->template get<I>())...);
+#endif
     }
     template <typename Fun>
     constexpr auto apply(Fun&& fun) -> decltype(auto) {

tests/beman/execution/exec-scope-simple-counting.test.cpp

@@ -109,7 +109,7 @@ auto mem() -> void {
         ASSERT(true == scope.get_token().try_associate());
         bool called{false};
         ASSERT(called == false);
-        auto state(test_std::connect(scope.join(), join_receiver(called)));
+        auto state(test_std::connect(scope.join(), join_receiver{called}));
         ASSERT(called == false);
         test_std::start(state);
         ASSERT(called == false);
@@ -126,7 +126,7 @@ auto token() -> void {
 
     ASSERT(true == tok.try_associate());
     bool called{false};
-    auto state(test_std::connect(scope.join(), join_receiver(called)));
+    auto state(test_std::connect(scope.join(), join_receiver{called}));
     test_std::start(state);
     ASSERT(false == called);
     scope.close();