[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH] gtk: Fix compiler warnings with
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH] gtk: Fix compiler warnings with -Werror=sign-compare |
Date: |
Mon, 04 Nov 2013 21:38:42 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130912 Thunderbird/17.0.9 |
On 11/04/13 21:07, Peter Maydell wrote:
> On 4 November 2013 19:51, Stefan Weil <address@hidden> wrote:
>> With -Werror=sign-compare (not enabled by default), gcc shows these errors:
>>
>> ui/gtk.c: In function ‘gtk_release_modifiers’:
>> ui/gtk.c:288:19: error:
>> comparison between signed and unsigned integer expressions
>> [-Werror=sign-compare]
>> ui/gtk.c: In function ‘gd_key_event’:
>> ui/gtk.c:746:19: error:
>> comparison between signed and unsigned integer expressions
>> [-Werror=sign-compare]
>
> If this warning is going to complain about entirely
> safe and idiomatic code like
>
> int i;
> static const int some_array[] = {
> 0x2a, 0x36, 0x1d, 0x9d, 0x38, 0xb8, 0xdb, 0xdd,
> };
>
> for (i = 0; i < ARRAY_SIZE(some_array); i++) {
> ...
> }
(Entirely safe, and completely non-idiomatic: "i" should be size_t, as
that is the type of the sizeof operator's result.)
Laszlo
/me apologizes for the raging pedantry