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 <[email protected]>
Reviewed-by: Alex Vakulenko <[email protected]>
Commit-Queue: Alex Deymo <[email protected]>
diff --git a/hardware_interface.h b/hardware_interface.h
index 2c9c100..d5a1f1f 100644
--- a/hardware_interface.h
+++ b/hardware_interface.h
@@ -20,6 +20,8 @@
// unit testing.
class HardwareInterface {
public:
+ virtual ~HardwareInterface() {}
+
// Returns the currently booted kernel partition. "/dev/sda2", for example.
virtual std::string BootKernelDevice() const = 0;
@@ -70,8 +72,6 @@
// or is invalid, returns -1. Brand new machines out of the factory or after
// recovery don't have this value set.
virtual int GetPowerwashCount() const = 0;
-
- virtual ~HardwareInterface() {}
};
} // namespace chromeos_update_engine
