diff options
author | wm4 <wm4@nowhere> | 2014-03-13 00:42:50 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2014-03-13 00:42:50 +0100 |
commit | 7918034e151a49fa84622cb3a2753377b3b5597b (patch) | |
tree | ff8ea51474809ca753e478bcab1f0a795632f36b /DOCS | |
parent | 3bc78a84cd776387fbbfe120cdfaa5b56d329c61 (diff) | |
download | mpv-7918034e151a49fa84622cb3a2753377b3b5597b.tar.bz2 mpv-7918034e151a49fa84622cb3a2753377b3b5597b.tar.xz |
DOCS/coding-style: add an example and another rule
Diffstat (limited to 'DOCS')
-rw-r--r-- | DOCS/coding-style.md | 25 |
1 files changed, 25 insertions, 0 deletions
diff --git a/DOCS/coding-style.md b/DOCS/coding-style.md index 9c19cd9fc3..8174a92436 100644 --- a/DOCS/coding-style.md +++ b/DOCS/coding-style.md @@ -44,6 +44,31 @@ General formatting ``` - If the body of an if statement uses braces, the else branch should also use braces (and reverse). + + Example: + + ```C + if (a) { + // do something + something(); + something_else(); + } else { + one_line(); + } + ``` +- If an if condition spans multiple physical lines, then put the opening brace + for the if body on the next physical line. (Also, preferably always add a + brace, even if technically none is needed.) + + Example: + + ```C + if (very_long_condition_a && + very_long_condition_b) + { + code(); + } + ``` - Remove any trailing whitespace. - If the file you're editing uses formatting different from from what is described here, it's probably an old file from times when nobody followed a |