[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH 2/2] CODING_STYLE: indent example
From: |
Wei Yang |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH 2/2] CODING_STYLE: indent example code as all others |
Date: |
Tue, 19 Feb 2019 22:05:44 +0000 |
User-agent: |
NeoMutt/20170113 (1.7.2) |
On Tue, Feb 19, 2019 at 06:38:54PM +0100, Philippe Mathieu-Daudé wrote:
>On 2/19/19 2:31 AM, Wei Yang wrote:
>> All the example code are indented with four spaces except this one.
>>
>> Fix this by adding four spaces here.
>>
>> Signed-off-by: Wei Yang <address@hidden>
>> ---
>> CODING_STYLE | 8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/CODING_STYLE b/CODING_STYLE
>> index 73f66ca185..27581d80c1 100644
>> --- a/CODING_STYLE
>> +++ b/CODING_STYLE
>> @@ -134,10 +134,10 @@ block to a separate function altogether.
>> When comparing a variable for (in)equality with a constant, list the
>> constant on the right, as in:
>>
>> -if (a == 1) {
>> - /* Reads like: "If a equals 1" */
>> - do_something();
>> -}
>> + if (a == 1) {
>> + /* Reads like: "If a equals 1" */
I don't get your point.
You mean to put it into multiline comments?
>
>I guess you found a bug in the documentation :)
>
>Since 8c06fbdf36bf4d the style asked is:
>
> We now require Linux-kernel-style multiline comments:
> /*
> * line one
> * line two
> */
>
>> + do_something();
>> + }
>>
>> Rationale: Yoda conditions (as in 'if (1 == a)') are awkward to read.
>> Besides, good compilers already warn users when '==' is mis-typed as '=',
>>
--
Wei Yang
Help you, Help me
- Re: [Qemu-trivial] [Qemu-devel] [PATCH 1/2] CODING_STYLE: specify the indent rule for multiline code, (continued)
Re: [Qemu-trivial] [Qemu-devel] [PATCH 1/2] CODING_STYLE: specify the indent rule for multiline code, Eric Blake, 2019/02/19
[Qemu-trivial] [PATCH 2/2] CODING_STYLE: indent example code as all others, Wei Yang, 2019/02/18
Re: [Qemu-trivial] [Qemu-devel] [PATCH 2/2] CODING_STYLE: indent example code as all others,
Wei Yang <=
Re: [Qemu-trivial] [Qemu-devel] [PATCH 2/2] CODING_STYLE: indent example code as all others, Eric Blake, 2019/02/19