> On 15 Feb 2018, at 11:00, Frediano Ziglio <fzig...@redhat.com> wrote:
>
> From: Christophe de Dinechin <dinec...@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 <dinec...@redhat.com>
> Acked-by: Frediano Ziglio <fzig...@redhat.com>
> ---
> Change since v1:
> - remove spurious hunk.
See, this is exactly what I want to avoid with the guideline about whitespaces.
I would not have minded much if you had split the whitespace fix in a separate
commit ;-)
Sent separate patch for it, but why, oh why!
> ---
> 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
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel
