common/scope_exit: Replace std::move with std::forward in ScopeExit()

The template type here is actually a forwarding reference, not an rvalue
reference in this case, so it's more appropriate to use std::forward to
preserve the value category of the type being moved.
This commit is contained in:
Lioncash 2019-04-11 20:01:31 -04:00 committed by fearlessTobi
parent 11754778bb
commit aab503fe71

View file

@ -20,7 +20,7 @@ struct ScopeExitHelper {
template <typename Func>
ScopeExitHelper<Func> ScopeExit(Func&& func) {
return ScopeExitHelper<Func>(std::move(func));
return ScopeExitHelper<Func>(std::forward<Func>(func));
}
} // namespace detail