clang-tidy: Enable modernize-use-override check
This check adds override (introduced in C++11) to overridden
virtual functions and removes virtual from those functions as it
is not required.
Change-Id: I156fc5796ed1cf451491e446c211172f62316ecc
Signed-off-by: Pavithra Barithaya <pavithrabarithaya07@gmail.com>
diff --git a/.clang-tidy b/.clang-tidy
index b816a4e..c382910 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -221,6 +221,7 @@
modernize-use-equals-default,
modernize-use-equals-delete,
modernize-use-noexcept,
+modernize-use-override,
modernize-use-transparent-functors,
modernize-use-uncaught-exceptions,
modernize-use-using'
diff --git a/host_condition_gpio/host_condition.hpp b/host_condition_gpio/host_condition.hpp
index 4a8e200..6e260b8 100644
--- a/host_condition_gpio/host_condition.hpp
+++ b/host_condition_gpio/host_condition.hpp
@@ -22,7 +22,7 @@
Host& operator=(const Host&) = delete;
Host(Host&&) = delete;
Host& operator=(Host&&) = delete;
- virtual ~Host() = default;
+ ~Host() override = default;
Host(sdbusplus::bus_t& bus, const std::string& path,
const std::string& hostId) :
diff --git a/hypervisor_state_manager.hpp b/hypervisor_state_manager.hpp
index 88ea57c..ae43a3f 100644
--- a/hypervisor_state_manager.hpp
+++ b/hypervisor_state_manager.hpp
@@ -36,7 +36,7 @@
Hypervisor& operator=(const Hypervisor&) = delete;
Hypervisor(Hypervisor&&) = delete;
Hypervisor& operator=(Hypervisor&&) = delete;
- virtual ~Hypervisor() = default;
+ ~Hypervisor() override = default;
/** @brief Constructs Hypervisor State Manager
*
diff --git a/scheduled_host_transition.hpp b/scheduled_host_transition.hpp
index 530031f..5582bf6 100644
--- a/scheduled_host_transition.hpp
+++ b/scheduled_host_transition.hpp
@@ -45,7 +45,7 @@
this->emit_object_added();
}
- ~ScheduledHostTransition();
+ ~ScheduledHostTransition() override;
/**
* @brief Handle with scheduled time