[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] hw/xics: Fix one-bit signed bitfields
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] hw/xics: Fix one-bit signed bitfields |
Date: |
Tue, 4 Sep 2012 21:53:48 +0100 |
On 4 September 2012 21:30, Stefan Weil <address@hidden> wrote:
> Report from smatch:
>
> xics.c:169:19: error: dubious one-bit signed bitfield
> xics.c:170:15: error: dubious one-bit signed bitfield
> xics.c:171:19: error: dubious one-bit signed bitfield
> xics.c:172:25: error: dubious one-bit signed bitfield
>
> Instead of replacing 'int' by 'unsigned', 'bool' was used because
> all 4 values are boolean values.
>
> Replacing 0 and 1 in the assignments by false and true
> looks better for those boolean values.
If we're changing this we should just drop the bitfield usage
entirely and use plain bool, I think.
-- PMM