Using std::decay_t

Author: polytypic

provides/include/testing_v1/private.hpp

@@ -39,5 +39,5 @@ template <class Action> class testing_v1::Private::test_t : test_base_t {
 
   void run() override;
 
-  std::conditional_t<std::is_function_v<Action>, Action *, Action> action;
+  Action action;
 };

provides/include/testing_v1/synopsis.hpp

@@ -21,8 +21,7 @@ template <class Action> struct test_t : Private::test_t<Action> {
 
 /// Creates and registers a test action and should only be used at namespace
 /// scope.
-template <class Action>
-test_t<std::remove_cvref_t<Action>> test(Action &&action);
+template <class Action> test_t<std::decay_t<Action>> test(Action &&action);
 
 /// Prints `FAIL` and `exit(1)`s the program if the result is false.
 void verify(bool result);

provides/include/testing_v1/test.hpp

@@ -19,7 +19,6 @@ testing_v1::test_t<Action>::test_t(ForwardableAction &&action)
     : Private::test_t<Action>(std::forward<ForwardableAction>(action)) {}
 
 template <class Action>
-testing_v1::test_t<std::remove_cvref_t<Action>>
-testing_v1::test(Action &&action) {
-  return test_t<std::remove_cvref_t<Action>>(std::forward<Action>(action));
+testing_v1::test_t<std::decay_t<Action>> testing_v1::test(Action &&action) {
+  return test_t<std::decay_t<Action>>(std::forward<Action>(action));
 }