[Spice-devel] [PATCH spice-common] quic: fix sign-compare warning
Frediano Ziglio
fziglio at redhat.com
Thu Jan 17 08:19:37 UTC 2019
>
> From: Marc-André Lureau <marcandre.lureau at redhat.com>
>
> ../subprojects/spice-common/common/quic.c: In function
> 'fill_model_structures':
> ../subprojects/spice-common/common/quic.c:695:55: error: comparison of
> integer expressions of different signedness: 'int' and 'unsigned int'
> [-Werror=sign-compare]
> spice_assert(free_counter - family_stat->counters == nbuckets *
> ncounters);
OT: is it a 32 bit architecture? Why a pointer difference is "int" ?
> ^~
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau at redhat.com>
Acked-by: Frediano Ziglio <figlio at redhat.com>
> ---
> common/quic.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/common/quic.c b/common/quic.c
> index c28974e..1760274 100644
> --- a/common/quic.c
> +++ b/common/quic.c
> @@ -692,7 +692,7 @@ static void fill_model_structures(SPICE_GNUC_UNUSED
> Encoder *encoder, FamilyStat
> bnumber++;
> } while (bend < levels - 1);
>
> - spice_assert(free_counter - family_stat->counters == nbuckets *
> ncounters);
> + spice_assert(free_counter - family_stat->counters ==
> (ptrdiff_t)(nbuckets * ncounters));
> }
>
> static void find_model_params(Encoder *encoder,
Frediano
More information about the Spice-devel
mailing list