From 2a9a8eda358cb45c621531b6bb61257d5cef5456 Mon Sep 17 00:00:00 2001 From: momo5502 Date: Tue, 18 Mar 2025 19:39:13 +0100 Subject: [PATCH] Fix clang-tidy warnings --- src/common/utils/finally.hpp | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/common/utils/finally.hpp b/src/common/utils/finally.hpp index 65c74392..f4a68e44 100644 --- a/src/common/utils/finally.hpp +++ b/src/common/utils/finally.hpp @@ -13,7 +13,7 @@ namespace utils class final_action { public: - static_assert(!std::is_reference::value && !std::is_const::value && !std::is_volatile::value, + static_assert(!std::is_reference_v && !std::is_const_v && !std::is_volatile_v, "Final_action should store its callable by value"); explicit final_action(F f) noexcept @@ -33,14 +33,16 @@ namespace utils ~final_action() noexcept { - if (invoke_) - f_(); + if (this->invoke_) + { + this->f_(); + } } // Added by momo5502 void cancel() { - invoke_ = false; + this->invoke_ = false; } private: @@ -49,8 +51,8 @@ namespace utils }; template - final_action::type>::type> finally(F&& f) noexcept + final_action>> finally(F&& f) noexcept { - return final_action::type>::type>(std::forward(f)); + return final_action>>(std::forward(f)); } }