[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 1/1] Avoid compiler error when building block/bl
From: |
Jes Sorensen |
Subject: |
[Qemu-devel] Re: [PATCH 1/1] Avoid compiler error when building block/blkdebug.c with -Wtype-limits |
Date: |
Fri, 10 Sep 2010 19:15:14 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.9) Gecko/20100907 Fedora/3.1.3-1.fc13 Lightning/1.0b3pre Thunderbird/3.1.3 |
On 09/10/10 19:05, Blue Swirl wrote:
> On Fri, Sep 10, 2010 at 2:04 PM, <address@hidden> wrote:
>> From: Jes Sorensen <address@hidden>
>>
>> Signed-off-by: Jes Sorensen <address@hidden>
>> ---
>> block/blkdebug.c | 7 ++++++-
>> 1 files changed, 6 insertions(+), 1 deletions(-)
>>
>> diff --git a/block/blkdebug.c b/block/blkdebug.c
>> index 2a63df9..17d796d 100644
>> --- a/block/blkdebug.c
>> +++ b/block/blkdebug.c
>> @@ -439,7 +439,12 @@ static void blkdebug_debug_event(BlockDriverState *bs,
>> BlkDebugEvent event)
>> struct BlkdebugRule *rule;
>> BlkdebugVars old_vars = s->vars;
>>
>> - if (event < 0 || event >= BLKDBG_EVENT_MAX) {
>> + /*
>> + * enum is not guaranteed to be signed on all archs, so cast to
>> + * int before the comparison against zero to avoid compiler
>> + * warning when building with -Wtype-limits
>> + */
>> + if ((int)event < 0 || event >= BLKDBG_EVENT_MAX) {
>
> I changed 'if' to 'assert' in my version because the check could only
> fail due to an internal error:
> http://lists.nongnu.org/archive/html/qemu-devel/2010-09/msg00239.html
Sorry I missed your posting. I am happy with your version too, ACK from me.
I did a pull before doing this patch and didn't see it, which is why I
posted my version, but your patch does the trick nicely.
Cheers,
Jes