Browse Source

CodingStyle: start flamewar about use of braces

Signed-off-by: Oliver Neukum <oneukum@suse.de>
Cc: Tilman Schmidt <tilman@imap.cc>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Oliver Neukum 18 years ago
parent
commit
e659ba4a0d
1 changed files with 15 additions and 0 deletions
  1. 15 0
      Documentation/CodingStyle

+ 15 - 0
Documentation/CodingStyle

@@ -160,6 +160,21 @@ supply of new-lines on your screen is not a renewable resource (think
 25-line terminal screens here), you have more empty lines to put
 25-line terminal screens here), you have more empty lines to put
 comments on.
 comments on.
 
 
+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();
+}
+
 		3.1:  Spaces
 		3.1:  Spaces
 
 
 Linux kernel style for use of spaces depends (mostly) on
 Linux kernel style for use of spaces depends (mostly) on