[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 4/5] vmstateify tsc2005
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v2 4/5] vmstateify tsc2005 |
Date: |
Thu, 22 Sep 2016 17:00:42 +0100 |
On 22 September 2016 at 16:53, Dr. David Alan Gilbert
<address@hidden> wrote:
> * Peter Maydell (address@hidden) wrote:
>> These changes mean the code is now doing bitwise-logic on
>> bool variables, for instance:
>> s->busy &= s->enabled;
>
>> We also assign '0' and '1' to them rather than 'true' and 'false'.
>
> Yes, I went through and as far as I can tell they're always
> used as booleans already.
Yeah, but using bitwise ops on booleans is the kind of
thing that prompts compilers and static analysers and
linters to complain, so we should fix those expressions
if we're changing the type.
thanks
-- PMM