diff options
author | Jeff King <peff@peff.net> | 2014-02-28 01:17:25 -0500 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-02-28 12:53:50 -0800 |
commit | f57b6cfdf7b395adc6b3c813109c2eb53e93809a (patch) | |
tree | aa23a72e18f2bbee611a8c1f621c8c7740fb2bd0 /Documentation/CodingGuidelines | |
parent | Git 1.8.5.5 (diff) | |
download | tgif-f57b6cfdf7b395adc6b3c813109c2eb53e93809a.tar.xz |
CodingGuidelines: mention C whitespace rules
We are fairly consistent about these, so most are covered by
"follow existing style", but it doesn't hurt to be explicit.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'Documentation/CodingGuidelines')
-rw-r--r-- | Documentation/CodingGuidelines | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/Documentation/CodingGuidelines b/Documentation/CodingGuidelines index ef67b53f72..ed432a80ca 100644 --- a/Documentation/CodingGuidelines +++ b/Documentation/CodingGuidelines @@ -126,6 +126,17 @@ For C programs: "char * string". This makes it easier to understand code like "char *string, c;". + - Use whitespace around operators and keywords, but not inside + parentheses and not around functions. So: + + while (condition) + func(bar + 1); + + and not: + + while( condition ) + func (bar+1); + - We avoid using braces unnecessarily. I.e. if (bla) { |