[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules |
Date: |
Tue, 17 Aug 2010 11:39:00 -0700 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.7) Gecko/20100720 Fedora/3.1.1-1.fc13 Thunderbird/3.1.1 |
On 08/17/2010 01:09 AM, Jes Sorensen wrote:
> On 08/12/10 19:50, Blue Swirl wrote:
>> +While using "bool" is good for readability, it comes with minor caveats:
>> + - Don't use "bool" in places where the type size must be constant across
>> + all systems, like public interfaces and on-the-wire protocols.
>> + - Don't compare a bool variable against the literal, "true",
>> + since a value with a logical non-false value need not be "1".
>> + I.e., don't write "if (seen == true) ...". Rather, write "if (seen)...".
>
> I'd strongly discourage the use of bool in any code.
I strongly disagree. The use of "bool", even if you ignore stdbool.h
and do "typedef int bool", is valuable documentation in the code.
r~
- [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Blue Swirl, 2010/08/12
- [Qemu-devel] Re: [PATCH 2/5] CODING_STYLE: add C type rules, Blue Swirl, 2010/08/13
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Jes Sorensen, 2010/08/17
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Blue Swirl, 2010/08/17
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, malc, 2010/08/17
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Jes Sorensen, 2010/08/17
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, malc, 2010/08/17
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Jes Sorensen, 2010/08/17
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Anthony Liguori, 2010/08/17
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, malc, 2010/08/17
Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules,
Richard Henderson <=
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Jes Sorensen, 2010/08/17
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Avi Kivity, 2010/08/18
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Jes Sorensen, 2010/08/19
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Avi Kivity, 2010/08/19
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Jes Sorensen, 2010/08/19
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Avi Kivity, 2010/08/19
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, malc, 2010/08/19
- Re: [Qemu-devel] [PATCH 2/5] CODING_STYLE: add C type rules, Avi Kivity, 2010/08/19
[Qemu-devel] Re: [PATCH 2/5] CODING_STYLE: add C type rules, Paolo Bonzini, 2010/08/18
[Qemu-devel] Re: [PATCH 2/5] CODING_STYLE: add C type rules, Jes Sorensen, 2010/08/18