Separate out variant_policy

alugowski · Dec 14, 2023 · 1bb7462 · 1bb7462
1 parent cddf3a9
commit 1bb7462
Show file tree

Hide file tree

Showing 5 changed files with 49 additions and 51 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -29,7 +29,7 @@ set(HEADER_FILES
         include/poolstl/algorithm
         include/poolstl/execution
         include/poolstl/numeric
-        include/poolstl/seq_fwd.hpp
+        include/poolstl/variant_policy.hpp
         include/poolstl/iota_iter.hpp
         include/poolstl/internal/utils.hpp
         include/poolstl/internal/ttp_impl.hpp

diff --git a/include/poolstl/execution b/include/poolstl/execution
@@ -15,10 +15,6 @@
 #include "internal/task_thread_pool.hpp"
 #include "internal/utils.hpp"
 
-#if POOLSTL_HAVE_CXX17
-#include <variant>
-#endif
-
 namespace poolstl {
 
     namespace ttp = task_thread_pool;
@@ -124,27 +120,6 @@ namespace poolstl {
             bool par_ok = true;
         };
 
-
-#if POOLSTL_HAVE_CXX17
-        /**
-         * A policy that allows selecting a policy at runtime.
-         *
-         * @tparam Variant std::variant<> of policy options.
-         */
-        template <typename Variant>
-        struct variant_policy {
-            explicit variant_policy(const Variant& policy): var(policy) {}
-            Variant var;
-        };
-
-        namespace internal {
-            using poolstl_policy_variant = std::variant<
-                poolstl::execution::parallel_policy,
-                poolstl::execution::sequenced_policy>;
-        }
-
-#endif
-
         /**
          * Choose parallel or sequential at runtime.
          *
@@ -180,9 +155,6 @@ namespace poolstl {
     using execution::seq;
     using execution::par;
     using execution::par_if;
-#if POOLSTL_HAVE_CXX17
-    using execution::variant_policy;
-#endif
 
     namespace internal {
         /**
@@ -223,25 +195,6 @@ namespace poolstl {
         template <class ExecPolicy>
         using is_pure_threads_policy = std::is_same<poolstl::execution::pure_threads_policy,
             typename std::remove_cv<typename std::remove_reference<ExecPolicy>::type>::type>;
-
-#if POOLSTL_HAVE_CXX17
-        /**
-         * Helper for enable_if_poolstl_variant
-         */
-        template <typename T> struct is_poolstl_variant_policy : std::false_type {};
-        template <typename V> struct is_poolstl_variant_policy<
-            ::poolstl::execution::variant_policy<V>> :std::true_type {};
-
-        /**
-         * To enable/disable variant_policy (for par_if) overload resolution
-         */
-        template <class ExecPolicy, class Tp>
-        using enable_if_poolstl_variant =
-            typename std::enable_if<
-                is_poolstl_variant_policy<
-                    typename std::remove_cv<typename std::remove_reference<ExecPolicy>::type>::type>::value,
-                Tp>::type;
-#endif
     }
 }
 

diff --git a/include/poolstl/poolstl.hpp b/include/poolstl/poolstl.hpp
@@ -15,7 +15,6 @@
 #include "execution"
 #include "algorithm"
 #include "numeric"
-#include "seq_fwd.hpp"
 
 // Note that iota_iter.hpp is self-contained in its own right.
 #include "iota_iter.hpp"

diff --git a/include/poolstl/seq_fwd.hpp → include/poolstl/variant_policy.hpp b/include/poolstl/seq_fwd.hpp → include/poolstl/variant_policy.hpp
@@ -3,11 +3,55 @@
 // the BSD 2-clause license, the MIT license, or at your choosing the BSL-1.0 license found in the LICENSE.*.txt files.
 // SPDX-License-Identifier: BSD-2-Clause OR MIT OR BSL-1.0
 
-#ifndef POOLSTL_SEQ_FWD_HPP
-#define POOLSTL_SEQ_FWD_HPP
+#ifndef POOLSTL_VARIANT_POLICY_HPP
+#define POOLSTL_VARIANT_POLICY_HPP
 
 #include "execution"
 
+#include <variant>
+
+namespace poolstl {
+    namespace execution {
+        /**
+         * A policy that allows selecting a policy at runtime.
+         *
+         * @tparam Variant std::variant<> of policy options.
+         */
+        template <typename Variant>
+        struct variant_policy {
+            explicit variant_policy(const Variant& policy): var(policy) {}
+            Variant var;
+        };
+
+        namespace internal {
+            using poolstl_policy_variant = std::variant<
+                poolstl::execution::parallel_policy,
+                poolstl::execution::sequenced_policy>;
+        }
+    }
+
+    using execution::variant_policy;
+
+    namespace internal {
+        /**
+         * Helper for enable_if_poolstl_variant
+         */
+        template <typename T> struct is_poolstl_variant_policy : std::false_type {};
+        template <typename V> struct is_poolstl_variant_policy<
+            ::poolstl::execution::variant_policy<V>> :std::true_type {};
+
+        /**
+         * To enable/disable variant_policy (for par_if) overload resolution
+         */
+        template <class ExecPolicy, class Tp>
+        using enable_if_poolstl_variant =
+            typename std::enable_if<
+                is_poolstl_variant_policy<
+                    typename std::remove_cv<typename std::remove_reference<ExecPolicy>::type>::type>::value,
+                Tp>::type;
+    }
+}
+
 /*
  * Forward poolstl::seq to the native sequential (no policy) method.
  */

diff --git a/tests/supplement_test.cpp b/tests/supplement_test.cpp
@@ -21,6 +21,8 @@
 #define POOLSTL_STD_SUPPLEMENT
 #include <poolstl/poolstl.hpp>
 
+// Test poolstl::variant_policy
+#include <poolstl/variant_policy.hpp>
 using std_policy_variant = std::variant<std::execution::parallel_policy, std::execution::sequenced_policy>;
 
 /**