diff options
Diffstat (limited to 'Documentation/CodingStyle')
-rw-r--r-- | Documentation/CodingStyle | 23 |
1 files changed, 7 insertions, 16 deletions
diff --git a/Documentation/CodingStyle b/Documentation/CodingStyle index fa6e25b94a54..c940239d9678 100644 --- a/Documentation/CodingStyle +++ b/Documentation/CodingStyle | |||
@@ -80,22 +80,13 @@ available tools. | |||
80 | The limit on the length of lines is 80 columns and this is a strongly | 80 | The limit on the length of lines is 80 columns and this is a strongly |
81 | preferred limit. | 81 | preferred limit. |
82 | 82 | ||
83 | Statements longer than 80 columns will be broken into sensible chunks. | 83 | Statements longer than 80 columns will be broken into sensible chunks, unless |
84 | Descendants are always substantially shorter than the parent and are placed | 84 | exceeding 80 columns significantly increases readability and does not hide |
85 | substantially to the right. The same applies to function headers with a long | 85 | information. Descendants are always substantially shorter than the parent and |
86 | argument list. Long strings are as well broken into shorter strings. The | 86 | are placed substantially to the right. The same applies to function headers |
87 | only exception to this is where exceeding 80 columns significantly increases | 87 | with a long argument list. However, never break user-visible strings such as |
88 | readability and does not hide information. | 88 | printk messages, because that breaks the ability to grep for them. |
89 | 89 | ||
90 | void fun(int a, int b, int c) | ||
91 | { | ||
92 | if (condition) | ||
93 | printk(KERN_WARNING "Warning this is a long printk with " | ||
94 | "3 parameters a: %u b: %u " | ||
95 | "c: %u \n", a, b, c); | ||
96 | else | ||
97 | next_statement; | ||
98 | } | ||
99 | 90 | ||
100 | Chapter 3: Placing Braces and Spaces | 91 | Chapter 3: Placing Braces and Spaces |
101 | 92 | ||