On Mon, Feb 19, 2018 at 03:06:40PM +0100, Christophe de Dinechin wrote:
>
>
> > On 19 Feb 2018, at 13:24, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> >
> > On Fri, Feb 16, 2018 at 04:23:06PM +0100, Christophe de Dinechin wrote:
> >> From: Christophe de Dinechin <dinechin@xxxxxxxxxx>
> >>
> >> Without this, GCC complains about signed / unsigned comparisons:
> >>
> >> mjpeg-fallback.cpp:121:24: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
> >> if (win_info.width != last_width || win_info.height != last_height) {
> >> ~~~~~~~~~~~~~~~^~~~~~~~~~~~~
> >
> > Are you getting this using the default CXXFLAGS?
>
> No, this was a side effect of tracking some other warning.
Would be nice to mention this in the commit log then, while we want the
build with the default flags to be warning-free, fixing additional
warnings will definitely be considered on a case by case basis.
>
>
> > Here I seem to be getting -Wno-sign-compare by default.
>
> Why is this silenced?
This probably was not intentionally silenced, but enabling it would be a
good complement to this patch if this warning is deemed useful...
Christophe
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel