using override on overridden methods instead of virtual

Author: andreagilardoni

src/property/types/CloudColor.h

@@ -161,7 +161,7 @@ class CloudColor : public Property {
     CloudColor() : _value(0, 0, 0), _cloud_value(0, 0, 0) {}
     CloudColor(float hue, float saturation, float brightness) : _value(hue, saturation, brightness), _cloud_value(hue, saturation, brightness) {}
 
-    virtual bool isDifferentFromCloud() {
+    bool isDifferentFromCloud() override {
 
       return _value != _cloud_value;
     }
@@ -182,19 +182,19 @@ class CloudColor : public Property {
       return _value;
     }
 
-    virtual void fromCloudToLocal() {
+    void fromCloudToLocal() override {
       _value = _cloud_value;
     }
-    virtual void fromLocalToCloud() {
+    void fromLocalToCloud() override {
       _cloud_value = _value;
     }
-    virtual CborError appendAttributesToCloud(CborEncoder *encoder) {
+    CborError appendAttributesToCloud(CborEncoder *encoder) override {
       CHECK_CBOR_MULTI(appendAttribute(_value.hue, "hue", encoder));
       CHECK_CBOR_MULTI(appendAttribute(_value.sat, "sat", encoder));
       CHECK_CBOR_MULTI(appendAttribute(_value.bri, "bri", encoder));
       return CborNoError;
     }
-    virtual void setAttributesFromCloud() {
+    void setAttributesFromCloud() override {
       setAttribute(_cloud_value.hue, "hue");
       setAttribute(_cloud_value.sat, "sat");
       setAttribute(_cloud_value.bri, "bri");

src/property/types/CloudLocation.h

@@ -63,7 +63,7 @@ class CloudLocation : public Property {
   public:
     CloudLocation() : _value(0, 0), _cloud_value(0, 0) {}
     CloudLocation(float lat, float lon) : _value(lat, lon), _cloud_value(lat, lon) {}
-    virtual bool isDifferentFromCloud() {
+    bool isDifferentFromCloud() override {
       float const distance = Location::distance(_value, _cloud_value);
       return _value != _cloud_value && (abs(distance) >= Property::_min_delta_property);
     }
@@ -83,18 +83,18 @@ class CloudLocation : public Property {
       return _value;
     }
 
-    virtual void fromCloudToLocal() {
+    void fromCloudToLocal() override {
       _value = _cloud_value;
     }
-    virtual void fromLocalToCloud() {
+    void fromLocalToCloud() override {
       _cloud_value = _value;
     }
-    virtual CborError appendAttributesToCloud(CborEncoder *encoder) {
+    CborError appendAttributesToCloud(CborEncoder *encoder) override {
       CHECK_CBOR_MULTI(appendAttribute(_value.lat, "lat", encoder));
       CHECK_CBOR_MULTI(appendAttribute(_value.lon, "lon", encoder));
       return CborNoError;
     }
-    virtual void setAttributesFromCloud() {
+    void setAttributesFromCloud() override {
       setAttribute(_cloud_value.lat, "lat");
       setAttribute(_cloud_value.lon, "lon");
     }

src/property/types/CloudSchedule.h

@@ -384,7 +384,7 @@ class CloudSchedule : public Property {
     CloudSchedule() : _value(0, 0, 0, 0), _cloud_value(0, 0, 0, 0) {}
     CloudSchedule(unsigned int frm, unsigned int to, unsigned int len, unsigned int msk) : _value(frm, to, len, msk), _cloud_value(frm, to, len, msk) {}
 
-    virtual bool isDifferentFromCloud() {
+    bool isDifferentFromCloud() override {
 
       return _value != _cloud_value;
     }
@@ -410,20 +410,20 @@ class CloudSchedule : public Property {
       return _value.isActive();
     }
 
-    virtual void fromCloudToLocal() {
+    void fromCloudToLocal() override {
       _value = _cloud_value;
     }
-    virtual void fromLocalToCloud() {
+    void fromLocalToCloud() override {
       _cloud_value = _value;
     }
-    virtual CborError appendAttributesToCloud(CborEncoder *encoder) {
+    CborError appendAttributesToCloud(CborEncoder *encoder) override {
       CHECK_CBOR_MULTI(appendAttribute(_value.frm, "frm", encoder));
       CHECK_CBOR_MULTI(appendAttribute(_value.to, "to", encoder));
       CHECK_CBOR_MULTI(appendAttribute(_value.len, "len", encoder));
       CHECK_CBOR_MULTI(appendAttribute(_value.msk, "msk", encoder));
       return CborNoError;
     }
-    virtual void setAttributesFromCloud() {
+    void setAttributesFromCloud() override {
       setAttribute(_cloud_value.frm, "frm");
       setAttribute(_cloud_value.to, "to");
       setAttribute(_cloud_value.len, "len");

src/property/types/CloudString.h

@@ -49,19 +49,19 @@ class CloudString : public Property {
     void clear() {
       _value = PropertyActions::CLEAR;
     }
-    virtual bool isDifferentFromCloud() {
+    bool isDifferentFromCloud() override {
       return _value != _cloud_value;
     }
-    virtual void fromCloudToLocal() {
+    void fromCloudToLocal() override {
       _value = _cloud_value;
     }
-    virtual void fromLocalToCloud() {
+    void fromLocalToCloud() override {
       _cloud_value = _value;
     }
-    virtual CborError appendAttributesToCloud(CborEncoder *encoder) {
+    CborError appendAttributesToCloud(CborEncoder *encoder) override {
       return appendAttribute(_value, "", encoder);
     }
-    virtual void setAttributesFromCloud() {
+    void setAttributesFromCloud() override {
       setAttribute(_cloud_value, "");
     }
     //modifiers