clang-tidy: add Chrome-inspired checks

[lttng-tools.git] / src / common / scope-exit.hpp

diff --git a/src/common/scope-exit.hpp b/src/common/scope-exit.hpp
index 46479bc572439762ecded83cad8d4b682927cac0..e73b5792b776910c5191f6583b27cce2a2d97860 100644 (file)
--- a/src/common/scope-exit.hpp
+++ b/src/common/scope-exit.hpp
@@ -36,15 +36,15 @@ public:
         * must be `noexcept` lest we anger the undefined behaviour gods.
         */
        static_assert(details::is_invocation_noexcept<ScopeExitInvocableType>::value,
-                       "scope_exit requires a noexcept invocable type");
+                     "scope_exit requires a noexcept invocable type");
 
        explicit scope_exit(ScopeExitInvocableType&& scope_exit_callable) :
-               _on_scope_exit{std::forward<ScopeExitInvocableType>(scope_exit_callable)}
+               _on_scope_exit{ std::forward<ScopeExitInvocableType>(scope_exit_callable) }
        {
        }
 
-       scope_exit(scope_exit&& rhs) :
-               _on_scope_exit{std::move(rhs._on_scope_exit)}, _armed{rhs._armed}
+       scope_exit(scope_exit&& rhs) noexcept :
+               _on_scope_exit{ std::move(rhs._on_scope_exit) }, _armed{ rhs._armed }
        {
                /* Don't invoke ScopeExitInvocableType for the moved-from copy. */
                rhs.disarm();
@@ -81,7 +81,7 @@ template <typename ScopeExitInvocableType>
 scope_exit<ScopeExitInvocableType> make_scope_exit(ScopeExitInvocableType&& scope_exit_callable)
 {
        return scope_exit<ScopeExitInvocableType>(
-                       std::forward<ScopeExitInvocableType>(scope_exit_callable));
+               std::forward<ScopeExitInvocableType>(scope_exit_callable));
 }
 
 } /* namespace lttng */