diff options
| author | Pieter Wuille <[email protected]> | 2017-01-10 11:58:27 -0800 |
|---|---|---|
| committer | Pieter Wuille <[email protected]> | 2017-01-10 11:58:27 -0800 |
| commit | 74994c6577a7ea129106e1ef0d5e30d87ca0499c (patch) | |
| tree | ad2702decab71a573c99f5f4d2de1bb2e027ef87 /doc/developer-notes.md | |
| parent | Merge #8811: rpc: Add support for JSON-RPC named arguments (diff) | |
| download | discoin-74994c6577a7ea129106e1ef0d5e30d87ca0499c.tar.xz discoin-74994c6577a7ea129106e1ef0d5e30d87ca0499c.zip | |
Improve style w.r.t. if
Diffstat (limited to 'doc/developer-notes.md')
| -rw-r--r-- | doc/developer-notes.md | 18 |
1 files changed, 13 insertions, 5 deletions
diff --git a/doc/developer-notes.md b/doc/developer-notes.md index ba03579e8..f8c34e060 100644 --- a/doc/developer-notes.md +++ b/doc/developer-notes.md @@ -11,9 +11,13 @@ gradually. - Braces on new lines for namespaces, classes, functions, methods. - Braces on the same line for everything else. - 4 space indentation (no tabs) for every block except namespaces. - - No indentation for public/protected/private or for namespaces. + - No indentation for `public`/`protected`/`private` or for `namespace`. - No extra spaces inside parenthesis; don't do ( this ) - - No space after function names; one space after if, for and while. + - No space after function names; one space after `if`, `for` and `while`. + - If an `if` only has a single-statement then-clause, it can appear + on the same line as the if, without braces. In every other case, + braces are required, and the then and else clauses must appear + correctly indented on a new line. - `++i` is preferred over `i++`. Block style example: @@ -22,14 +26,18 @@ namespace foo { class Class { - bool Function(char* psz, int n) + bool Function(const std::string& s, int n) { // Comment summarising what this section of code does for (int i = 0; i < n; ++i) { // When something fails, return early - if (!Something()) - return false; + if (!Something()) return false; ... + if (SomethingElse()) { + DoMore(); + } else { + DoLess(); + } } // Success return is usually at the end |