Using std::decay_t

Author: polytypic

provides/include/dumpster_v1/finally.hpp

@@ -21,14 +21,14 @@ dumpster_v1::finalizer<Action>::finalizer(ForwardableAction &&action)
     : Private::finalizer<Action>(std::forward<ForwardableAction>(action)) {}
 
 template <class Action>
-dumpster_v1::finalizer<std::remove_cvref_t<Action>>
+dumpster_v1::finalizer<std::decay_t<Action>>
 dumpster_v1::finally(Action &&action) {
-  return finalizer<std::remove_cvref_t<Action>>(std::forward<Action>(action));
+  return finalizer<std::decay_t<Action>>(std::forward<Action>(action));
 }
 
 template <class Action>
-std::optional<dumpster_v1::finalizer<std::remove_cvref_t<Action>>>
+std::optional<dumpster_v1::finalizer<std::decay_t<Action>>>
 dumpster_v1::finally_with_reset(Action &&action) {
-  return std::optional<finalizer<std::remove_cvref_t<Action>>>(
+  return std::optional<finalizer<std::decay_t<Action>>>(
       std::forward<Action>(action));
 }

provides/include/dumpster_v1/private.hpp

@@ -21,5 +21,5 @@ template <class Action> class dumpster_v1::Private::finalizer {
 
   template <class ForwardableAction> finalizer(ForwardableAction &&action);
 
-  std::conditional_t<std::is_function_v<Action>, Action *, Action> m_action;
+  Action m_action;
 };

provides/include/dumpster_v1/synopsis.hpp

@@ -50,12 +50,12 @@ template <class Action> struct finalizer : Private::finalizer<Action> {
 
 /// Creates a finalizer that invokes the action when destroyed.
 template <class Action>
-finalizer<std::remove_cvref_t<Action>> finally(Action &&action);
+finalizer<std::decay_t<Action>> finally(Action &&action);
 
 /// Creates a finalizer wrapped inside an optional so that the finalizer can be
 /// invoked explicitly by calling the `reset()` method.
 template <class Action>
-std::optional<finalizer<std::remove_cvref_t<Action>>>
+std::optional<finalizer<std::decay_t<Action>>>
 finally_with_reset(Action &&action);
 
 // insertion_sort.hpp ==========================================================