|
From: | Anthony Liguori |
Subject: | Re: [Qemu-devel] [PATCH] Add support for r6040 NIC |
Date: | Wed, 31 Aug 2011 08:30:49 -0500 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110516 Lightning/1.0b2 Thunderbird/3.1.10 |
On 08/31/2011 08:17 AM, malc wrote:
On Tue, 30 Aug 2011, Anthony Liguori wrote:On 08/30/2011 08:30 PM, malc wrote:On Tue, 30 Aug 2011, Anthony Liguori wrote:This won't even come close to passing checkpatch.plHave you actually tried?Sigh. I was hoping checkpatch.pl was more useful than it appears to be. At any rate, the patch doesn't follow CODING_STYLE.Where?
3. NamingVariables are lower_case_with_underscores; easy to type and read. Structured
type names are in CamelCase; harder to type but standing out. Scalar type names are lower_case_with_underscores_ending_with_a_t, like the POSIX uint64_t and family. Note that this last convention contradicts POSIX and is therefore likely to be changed. When wrapping standard library functions, use the prefix qemu_ to alert readers that they are seeing a wrapped version; otherwise avoid this prefix. Regards, Anthony Liguori
[..snip..]
[Prev in Thread] | Current Thread | [Next in Thread] |