Skip to content

doc: add basic C++ style guide #16090

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
addaleax wants to merge 6 commits into from
Closed

doc: add basic C++ style guide #16090

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
a4417a5
doc: add basic C++ style guide
addaleax Oct 8, 2017
39ab867
address nits
addaleax Oct 8, 2017
b6a028c
address more nits
addaleax Oct 8, 2017
66a546d
more nits
addaleax Oct 10, 2017
68176fe
[squash] cjihrig nits
addaleax Oct 12, 2017
8b289f1
[squash] capitalize Style Guide
addaleax Oct 13, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[squash] cjihrig nits
  • Loading branch information
@addaleax
addaleax committed Oct 12, 2017
commit 68176fe0d8dad14bf1c43fba7f963cc0558a0341
4 changes: 2 additions & 2 deletions CPP_STYLE_GUIDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ if (foo) {
}
```

(Braces are optional if the statement body only has one line.)
Braces are optional if the statement body only has one line.

`namespace`s receive no indentation on their own.

Expand Down Expand Up @@ -135,4 +135,4 @@ at the top level and not inside of nested calls.
A lot of code inside Node.js is written so that typechecking etc. is performed
in JavaScript.

(Using C++ `throw` is not allowed.)
Using C++ `throw` is not allowed.