nagai/kernel
3
0
Fork 0
forked from mirrors/linux
Code Pull requests Activity

Documentation/CodingStyle: flesh out if-else examples

Browse source 
There is a missing case for "Chapter 3: Placing Braces and Spaces".  We
often know we should not use braces where a single statement.  The first
case is:

	if (condition)
		action();

Another case is:

	if (condition)
		do_this();
	else
		do_that();

However, I can not find a description of the second case.

Signed-off-by: Harry Wei <harryxiyou@gmail.com>
Cc: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
Harry Wei 2011-03-22 16:35:01 -07:00 committed by Linus Torvalds
parent 0bc825d240
commit 38829dc9d7
1 changed files with 7 additions and 0 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

7
Documentation/CodingStyle
View file

@ -168,6 +168,13 @@ Do not unnecessarily use braces where a single statement will do.
if (condition) if (condition)
action(); action();
and
if (condition)
do_this();
else
do_that();
This does not apply if one branch of a conditional statement is a single This does not apply if one branch of a conditional statement is a single
statement. Use braces in both branches. statement. Use braces in both branches.