|
Size: 1596
Comment:
|
Size: 2187
Comment:
|
| Deletions are marked like this. | Additions are marked like this. |
| Line 1: | Line 1: |
| [[http://clang.llvm.org/extra/clang-tidy/index.html|Clang-Tidy — Extra Clang Tools 3.9 documentation]] Use: {{{ clang-tidy -checks='*' main.cpp -- -std=c++11 }}} [[http://clang.llvm.org/docs/ClangFormat.html|ClangFormat]]: auto-formats C++ code * [[https://github.com/andrewseidl/githook-clang-format|Git hook running clang-format]]. Don't use this! |
|
| Line 22: | Line 10: |
| == Formatting == [[http://clang.llvm.org/docs/ClangFormat.html|ClangFormat]]: auto-formats C++ code * [[https://github.com/andrewseidl/githook-clang-format|Git hook running clang-format]]. Don't use this! |
|
| Line 24: | Line 18: |
| * [[http://clang.llvm.org/extra/clang-tidy/index.html|Clang-Tidy]]: C/C++ linting tool, à la Google's cpplint. {{{ clang-tidy -checks='*' *.cpp -- -std=c++11 }}} |
|
| Line 39: | Line 38: |
== Stuff I never remember == Beware unncessary copies w/ `auto`; default semantics is to make a copy. {{{#!highlight c++ numbers=off // Typically there's no reason to copy. for (const auto &Val : Container) { observe(Val); } for (auto &Val : Container) { Val.change(); } // Remove the reference if you really want a new copy. for (auto Val : Container) { Val.change(); saveSomewhere(Val); } // Copy pointers, but make it clear that they're pointers. for (const auto *Ptr : Container) { observe(*Ptr); } for (auto *Ptr : Container) { Ptr->change(); } }}} |
C++ style guides
Formatting
ClangFormat: auto-formats C++ code
Git hook running clang-format. Don't use this!
Static analysis tools
Clang-Tidy: C/C++ linting tool, à la Google's cpplint.
clang-tidy -checks='*' *.cpp -- -std=c++11
Clang Static Analyzer: sets CC and CXX to use alternate compiler to do analysis
# Highest severity warnings only (i.e. memory leaks, etc)
cppcheck --enable=all *.cpp
# Everything except style-related linting
cppcheck --enable=warning,performance,portability,information,missingInclude \
--std=c++11 --library=std.cfg --verbose --quiet \
*.cpp
Overview articles:
Stuff I never remember
Beware unncessary copies w/ auto; default semantics is to make a copy.
// Typically there's no reason to copy.
for (const auto &Val : Container) { observe(Val); }
for (auto &Val : Container) { Val.change(); }
// Remove the reference if you really want a new copy.
for (auto Val : Container) { Val.change(); saveSomewhere(Val); }
// Copy pointers, but make it clear that they're pointers.
for (const auto *Ptr : Container) { observe(*Ptr); }
for (auto *Ptr : Container) { Ptr->change(); }
