[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] CODING_STYLE and if blocks
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] CODING_STYLE and if blocks |
Date: |
Fri, 23 Oct 2009 08:55:39 +0200 |
User-agent: |
Mutt/1.5.18 (2008-05-17) |
On Fri, Oct 23, 2009 at 01:47:04AM +0200, Alexander Graf wrote:
>
> On 23.10.2009, at 00:42, Anthony Liguori wrote:
>
>> Aurelien Jarno wrote:
>>> Hi all,
>>>
>>> I am currently reviewing the S390 patches which extensively use of
>>> code like:
>>>
>>> if (a == 5) printf("a was 5.\n");
>>> else if (a == 6) printf("a was 6.\n");
>>> else printf("a was something else entirely.\n");
>>>
>>> It is something currently allowed by the CODING_STYLE document
>>> (there is
>>> no "indented statement"), but I am not fully comfortable with it.
>>> Should
>>> we accept such code? Should we fix CODING_STYLE?
>>>
>>
>> I'd vote for fixing CODING_STYLE as that syntax makes my eyes hurt.
>>
>> While CODING_STYLE is there as a guideline, good taste should still
>> always prevail :-)
>
> I think Uli only wrote the code as is because CODING_STYLE told him to
> always use braces around one-liner statements. I don't see how
>
> if (a == 5)
> printf("a was 5.\n");
> else if (a == 6)
> printf("a was 6.\n");
> else
> printf("a was something else entirely.\n");
>
> would be not readable. In fact I tend to use that code style myself a
> lot in places where it makes sense like:
>
> if (r < 0)
> return r;
>
> It would really hurt my eyes to have braces on these simple ifs every
> single time.
>
It has been debated already a few times already, the argument against
this it that it make patches more difficulty readable.
--
Aurelien Jarno GPG: 1024D/F1BCDB73
address@hidden http://www.aurel32.net