neobytes-project/neobytes
neobytes-project/neobytes
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge #8602: [trivial][doc] Mention ++i as preferred over i++ in dev notes

Browse Source 
ab53207 [trivial][doc] Mention ++i as preferred to i++ in dev notes (fanquake)
This commit is contained in:
Wladimir J. van der Laan 2016-08-26 15:54:25 +02:00 committed by Alexander Block
parent 59f4df1d8b
commit 87c81ba6f9
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
doc/developer-notes.md
View File

@ -14,6 +14,7 @@ gradually.
- No indentation for public/protected/private or for namespaces. - No indentation for public/protected/private or for namespaces.
- No extra spaces inside parenthesis; don't do ( this ) - 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.
- `++i` is preferred over `i++`.
- Align pointers and references to the left i.e. use `type& var` and not `type &var`. - Align pointers and references to the left i.e. use `type& var` and not `type &var`.
Block style example: Block style example:
@ -25,7 +26,7 @@ class Class
bool Function(char* psz, int n, const string& s) bool Function(char* psz, int n, const string& s)
{ {
// Comment summarising what this section of code does // 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 // When something fails, return early
if (!Something()) if (!Something())
return false; return false;
@ -234,9 +235,9 @@ General Dash Core
- *Rationale*: Makes sure that they pass thorough testing, and that the tester will keep passing - *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 on the master branch. Otherwise all new pull requests will start failing the tests, resulting in
confusion and mayhem confusion and mayhem
- *Explanation*: If the test suite is to be updated for a change, this has to - *Explanation*: If the test suite is to be updated for a change, this has to
be done first be done first
Wallet Wallet
------- -------