Commit 38829dc9 authored Mar 22, 2011 by Harry Wei Committed by Linus Torvalds Mar 22, 2011

Documentation/CodingStyle: flesh out if-else examples



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>

parent 0bc825d2

Documentation/CodingStyle

+7 −0

Original line number	Diff line number	Diff line
		@@ -168,6 +168,13 @@ Do not unnecessarily use braces where a single statement will do.
		if (condition)
		action();

		and

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

		This does not apply if one branch of a conditional statement is a single
		statement. Use braces in both branches.