update_engine: Add override when possible.
Google Style Guide requires to include the "override" keyword
when overriding a method on a derived class, so the compiler will
catch errors if the method is not overriding a member of the base
class.
This patch introduces the "override" keyword when possible.
BUG=None
TEST=FEATURES=test emerge-link update_engine
Change-Id: Ie83d115c5730f3b35b3d95859a54bc1a48e0be7b
Reviewed-on: https://chromium-review.googlesource.com/228928
Tested-by: Alex Deymo <deymo@chromium.org>
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Alex Deymo <deymo@chromium.org>
diff --git a/clock_interface.h b/clock_interface.h
index bafb7fc..8ff122c 100644
--- a/clock_interface.h
+++ b/clock_interface.h
@@ -17,6 +17,8 @@
// is also unit testing.
class ClockInterface {
public:
+ virtual ~ClockInterface() = default;
+
// Gets the current time e.g. similar to base::Time::Now().
virtual base::Time GetWallclockTime() = 0;
@@ -33,8 +35,6 @@
//
// (This is a simple wrapper around clock_gettime(2) / CLOCK_BOOTTIME.)
virtual base::Time GetBootTime() = 0;
-
- virtual ~ClockInterface() {}
};
} // namespace chromeos_update_engine