[libvirt] [PATCH 02/15] Remove unneeded curly brackets around oneline code blocks in src/conf/

Martin Kletzander mkletzan at redhat.com
Fri Nov 7 15:59:27 UTC 2014

On Fri, Nov 07, 2014 at 09:58:11AM -0500, Laine Stump wrote:
>On 11/07/2014 05:09 AM, Eric Blake wrote:
>> On 11/06/2014 05:38 PM, Martin Kletzander wrote:
>>> @@ -138,9 +138,9 @@ virDevicePCIAddressEqual(virDevicePCIAddress *addr1,
>>>      if (addr1->domain == addr2->domain &&
>>>          addr1->bus == addr2->bus &&
>>>          addr1->slot == addr2->slot &&
>>> -        addr1->function == addr2->function) {
>>> +        addr1->function == addr2->function)
>>>          return true;
>>> -    }
>> Conversions like this are a little harder to read (that is, any time the
>> 'if' condition extends over multiple lines, but there is exactly four
>> space indentation of the condition, it's visually hard to see where the
>> multi-line condition ends and the loop body begins).  The opening { is
>> hard to see either way, but seeing the closing } is my visual clue that
>> "yes, the last line of this blob of code is not part of the 'if' but the
>> one-line body".  I'm not opposed to your removal of {}, but wonder if we
>> should rethink our style to recommend keeping the {} if the 'if'
>> expression is multiline.
>I've always thought this, and used to do it that way until a few
>reviewers complained. So I for one would welcome our new "use braces if
>condition is multiple lines" overlords :-)
>(I also agree that this is all an exercise in futility unless there is a
>matching syntax-check rule to enforce it.)

Well, I used to place opening curly brackets on their own new lines as
we do with functions (pre-libvirt) and this is something I had to get
used to.

Anyway, I'd be more than OK with changing contributor guidelines and
saying that we require curly brackets if either a) the body or b) the
condition itself spans over multiple lines.  syntax-check for that is
*way* easier ;)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20141107/625aa80d/attachment-0001.sig>

More information about the libvir-list mailing list