qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] Drop braces around single statement rule


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PATCH] Drop braces around single statement rule
Date: Mon, 02 Aug 2010 11:32:05 -0500
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.11) Gecko/20100713 Lightning/1.0b1 Thunderbird/3.0.6

On 08/02/2010 11:29 AM, Blue Swirl wrote:
Yeah, I'm neglecting the fact that we're not consistent as maintainers and
I'm all for dropping it from CODING_STYLE.
I'd rather expand the document. For example, I like the approach libvirt takes:
http://libvirt.org/git/?p=libvirt.git;a=blob_plain;f=HACKING;hb=HEAD

Not specifically to braces, but they describe types and memory
allocation etc. which is FAQ stuff for us too.

I agree 100%. There are a lot of idioms in QEMU that would be good to document for people learning the code base.

But I think that's orthogonal to the discussion of whether we want to keep the if rule and whether we want to enforce all of coding style at commit time.

Regards,

Anthony Liguori




reply via email to

[Prev in Thread] Current Thread [Next in Thread]