diff options
author | fanquake <fanquake@gmail.com> | 2016-08-26 19:40:24 +0800 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2016-08-26 19:40:24 +0800 |
commit | ab53207b9cdb6422cbb515ca6acf1bf945651895 (patch) | |
tree | f1eef2fb5ff3cc607e46826cb65039280950c9d5 | |
parent | 12892dbb9fb6d6f46505d7b9caf8ae6905601c7f (diff) |
[trivial][doc] Mention ++i as preferred to i++ in dev notes
-rw-r--r-- | doc/developer-notes.md | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/doc/developer-notes.md b/doc/developer-notes.md index 95c46b05fe..70c0690ba3 100644 --- a/doc/developer-notes.md +++ b/doc/developer-notes.md @@ -14,6 +14,7 @@ gradually. - No indentation for public/protected/private or for namespaces. - No extra spaces inside parenthesis; don't do ( this ) - No space after function names; one space after if, for and while. + - `++i` is preferred over `i++`. Block style example: ```c++ @@ -24,7 +25,7 @@ class Class bool Function(char* psz, int n) { // Comment summarising what this section of code does - for (int i = 0; i < n; i++) { + for (int i = 0; i < n; ++i) { // When something fails, return early if (!Something()) return false; @@ -231,9 +232,9 @@ General Bitcoin Core - *Rationale*: Makes sure that they pass thorough testing, and that the tester will keep passing on the master branch. Otherwise all new pull requests will start failing the tests, resulting in confusion and mayhem - + - *Explanation*: If the test suite is to be updated for a change, this has to - be done first + be done first Wallet ------- |