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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> On 15 Feb 2018, at 11:00, Frediano Ziglio <fziglio@xxxxxxxxxx> wrote:
> 
> From: Christophe de Dinechin <dinechin@xxxxxxxxxx>
> 
> 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@xxxxxxxxxx>
> Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
> 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@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]