summaryrefslogtreecommitdiffstats
path: root/DOCS
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2014-03-13 00:42:50 +0100
committerwm4 <wm4@nowhere>2014-03-13 00:42:50 +0100
commit7918034e151a49fa84622cb3a2753377b3b5597b (patch)
treeff8ea51474809ca753e478bcab1f0a795632f36b /DOCS
parent3bc78a84cd776387fbbfe120cdfaa5b56d329c61 (diff)
downloadmpv-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.md25
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