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.