clang-tidy: Enable cppcoreguidelines-explicit-virtual-functions
Adds override (introduced in C++11) to overridden virtual functions
and removes virtual from those functions as it is not required.
virtual on non base class implementations was used to help indicate
to the user that a function was virtual. C++ compilers did not use
the presence of this to signify an overridden function.
In C++11 override and final keywords were introduced to allow
overridden functions to be marked appropriately. Their presence
allows compilers to verify that an overridden function correctly
overrides a base class implementation.
This can be useful as compilers can generate a compile time error
when:
- The base class implementation function signature changes.
- The user has not created the override with the correct signature.
Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: I428f69c11b071a1a50e7f93f7bcc29c8300dcddb
diff --git a/test/test_activation.cpp b/test/test_activation.cpp
index 2e93672..97b8ff3 100644
--- a/test/test_activation.cpp
+++ b/test/test_activation.cpp
@@ -33,7 +33,7 @@
.WillByDefault(Return(any(PropertyType(std::string("TestModel")))));
ON_CALL(mockedUtils, isAssociated(_, _)).WillByDefault(Return(false));
}
- ~TestActivation()
+ ~TestActivation() override
{
utils::freeUtils();
}