[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] CODING_STYLE: {} as in linux kernel
From: |
Jamie Lokier |
Subject: |
Re: [Qemu-devel] [PATCH] CODING_STYLE: {} as in linux kernel |
Date: |
Wed, 7 Oct 2009 10:11:27 +0100 |
User-agent: |
Mutt/1.5.13 (2006-08-11) |
Michael S. Tsirkin wrote:
> 4. Block structure
>
> -Every indented statement is braced; even if the block contains just one
> -statement. The opening brace is on the line that contains the control
> -flow statement that introduces the new block; the closing brace is on the
> -same line as the else keyword, or on a line by itself if there is no else
> -keyword. Example:
> +If an indented block contains just one statement, it is not braced. This
> +matches the Linux coding style. The opening brace of a block is on the line
> +that contains the control flow statement that introduces the new block; the
> +closing brace is on the same line as the else keyword, or on a line by itself
> +if there is no else keyword. Example:
>
> - if (a == 5) {
> + if (a == 5)
> printf("a was 5.\n");
> - } else if (a == 6) {
> + else if (a == 6) {
> printf("a was 6.\n");
> - } else {
> + printf("multiply by 7 to get the answer.\n");
> + } else
> printf("a was something else entirely.\n");
> - }
>
> An exception is the opening brace for a function; for reasons of tradition
> and clarity it comes on a line by itself:
> @@ -75,4 +75,5 @@ and clarity it comes on a line by itself:
>
> Rationale: a consistent (except for functions...) bracing style reduces
> ambiguity and avoids needless churn when lines are added or removed.
> +This matches the linux coding style.
Actually the above does not match the Linux coding style, and it's ugly too.
This is the Linux style:
Do not unnecessarily use braces where a single statement will do.
if (condition)
action();
This does not apply if one branch of a conditional statement
is a single statement. Use braces in both branches.
if (condition) {
do_this();
do_that();
} else {
otherwise();
}
-- Jamie