[Spice-devel] [PATCH spice-streaming-agent v2 2/4] Remove clang warning on missing 'override'

[Frediano Ziglio]

From: Christophe de Dinechin <dinechin at redhat.com>

In file included from mjpeg-fallback.cpp:8:
./mjpeg-fallback.hpp:28:25: warning: 'VideoCodecType' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]
    SpiceVideoCodecType VideoCodecType() const;
                        ^
../include/spice-streaming-agent/plugin.hpp:92:33: note: overridden virtual function is here
    virtual SpiceVideoCodecType VideoCodecType() const = 0;
                                ^

Signed-off-by: Christophe de Dinechin <dinechin at redhat.com>
Acked-by: Frediano Ziglio <fziglio at redhat.com>
---
Change since v1:
- remove spurious hunk.
---
 src/mjpeg-fallback.cpp | 2 +-
 src/mjpeg-fallback.hpp | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/mjpeg-fallback.cpp b/src/mjpeg-fallback.cpp
index 74682f3..3cb708a 100644
--- a/src/mjpeg-fallback.cpp
+++ b/src/mjpeg-fallback.cpp
@@ -47,7 +47,7 @@ public:
     ~MjpegFrameCapture();
     FrameInfo CaptureFrame() override;
     void Reset() override;
-    SpiceVideoCodecType VideoCodecType() const {
+    SpiceVideoCodecType VideoCodecType() const override {
         return SPICE_VIDEO_CODEC_TYPE_MJPEG;
     }
 private:
diff --git a/src/mjpeg-fallback.hpp b/src/mjpeg-fallback.hpp
index 04fa2eb..ca5c7d3 100644
--- a/src/mjpeg-fallback.hpp
+++ b/src/mjpeg-fallback.hpp
@@ -25,7 +25,7 @@ public:
     FrameCapture *CreateCapture() override;
     unsigned Rank() override;
     void ParseOptions(const ConfigureOption *options);
-    SpiceVideoCodecType VideoCodecType() const;
+    SpiceVideoCodecType VideoCodecType() const override;
 private:
     MjpegSettings settings = { 10, 80 };
 };
-- 
2.14.3