summaryrefslogtreecommitdiffstats
path: root/Documentation/CodingStyle
diff options
context:
space:
mode:
authorHarry Wei <jiaweiwei.xiyou@gmail.com>2011-03-22 16:35:01 -0700
committerLinus Torvalds <torvalds@linux-foundation.org>2011-03-22 17:44:16 -0700
commit38829dc9d7b46b195ab99d62c8c53c21a7adc36b (patch)
treeff38cc7aa7965af81e2d5ee5f21e2724489269ec /Documentation/CodingStyle
parent0bc825d240abcaf5ed6e9d59b44215b51718ef5b (diff)
downloadop-kernel-dev-38829dc9d7b46b195ab99d62c8c53c21a7adc36b.zip
op-kernel-dev-38829dc9d7b46b195ab99d62c8c53c21a7adc36b.tar.gz
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>
Diffstat (limited to 'Documentation/CodingStyle')
-rw-r--r--Documentation/CodingStyle7
1 files changed, 7 insertions, 0 deletions
diff --git a/Documentation/CodingStyle b/Documentation/CodingStyle
index 1cd3478..58b0bf9 100644
--- a/Documentation/CodingStyle
+++ b/Documentation/CodingStyle
@@ -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.
OpenPOWER on IntegriCloud