Commit Graph

2193 Commits (master)
 

Author SHA1 Message Date
Andrey Sikorin 728ed7c65f
Fix Adobe and CERT links (#1971) 3 years ago
tocic 4ec7e81ecb
Fix typos (#1970) 3 years ago
Sergey Zubkov 9d28e32999 C.87: fix missing access specifiers (closes #1957) 3 years ago
Sergey Zubkov 9a95f97f44 C.168: switch example to op+ to avoid side tracking about defaults (closes #1955) 3 years ago
Sergey Zubkov 0a7d70851c F.17: note about passing reference wrappers by value (issue #1948) 3 years ago
Daniel Kříž 6476b15070
SF.5: prefer relative import to absolute one (#1944)
Signed-off-by: Daniel Kříž <Daniel.kriz@protonmail.com>
3 years ago
Niels Dekker 1aafa24880
E.15 Clarify when a rethrow would slice to `std::exception` (#1949)
Avoided a potential misunderstanding that `throw e` would slice *any* `e` to `std::exception`.

The note about rethrowing appears to be written when the example of this item still did `catch (const exception& e)`, which was prior to commit d568d190f6, "Generalized E.15, closes #1848".
3 years ago
Nikolay Panov be1722ad0c
[#1946] T.65 Fix two typos (#1956)
Fixes #1946:
 - copy_trait type argument needs to be a container element type.
   Use Value_type<Iter> like in other examples.

 - Fix name typo in the definition of copy_helper.
3 years ago
Baiyan Huang 444704f703
C.168 Fix code example (#1954) 3 years ago
Nikolay Panov dca24a0379
T.21 Fix wrong return type (#1942)
Co-authored-by: Nikolay Panov <pn.interface@gmail.com>
3 years ago
bgloyer 6f27719b2b
C.64 Minor fix to make example compile (#1939) 4 years ago
Sergey Zubkov 4b8b2819b9 C.32: amend merged PR per call notes. It overlaps R.3 quite a bit 4 years ago
Sergey Zubkov 2b05a48a05 F.1 drop invalid line closes #1928 4 years ago
bgloyer cbf455407e
Clean up and add example for C.32 - raw pointers (#1909)
* Update CppCoreGuidelines.md

* Update CppCoreGuidelines.md

* Update isocpp.dic

* use snake casing

* sake case naming

* C 32 comments (#3)

* F.16 ("in" parameters): Move Matrix example to F.20 (return values) (#1922)

The `Matrix` example and the notes about assignment appear off-topic in rule F.16, as F.16 is specifically about "in" parameters.

With help from Sergey Zubkov.

* SL.io.50 (Avoid `endl`): Mention string streams (#1920)

Explicitly mentioned string streams as `endl` insertions into string streams do actually occur in the wild.

With help from Sergey Zubkov.

* Extended E.16 to include copy ctor for exception type, closes #1921

* Fix GitHub Actions build warnings, Marker style should be `*` (#1925)

* restored reference

* Added references to note

Co-authored-by: Niels Dekker <N.Dekker@lumc.nl>
Co-authored-by: Herb Sutter <herb.sutter@gmail.com>

Co-authored-by: Niels Dekker <N.Dekker@lumc.nl>
Co-authored-by: Herb Sutter <herb.sutter@gmail.com>
4 years ago
Ankur Satle f2e9b5c2ac
Issue #1931: C++20 wasn't included in "modern C++" (#1932)
* Issue #1931 - Include C++20 as 'modern C++'

* "modern C++" means C++11 and newer
4 years ago
Gábor Szeder 42c4cc6a75
E.19: clarify that `finally` is from the GSL (#1936)
When I stumbled upon E.19 "Use a `final_action` object to express
cleanup if no suitable resource handle is available" I was hopeful
that this `final_action`/`finally()` thing is from the STL, but, alas,
it isn't.

Make it clear that `finally` is a GSL construct.
4 years ago
Sven van Haastregt fde9ee5de1
C.9: Improve an inline link (#1933)
Avoid ending the sentence with a dangling "see".
4 years ago
Francisco Moretti 6d018fcf5f
C.166: Use markdown format and document title for link (#1929) 4 years ago
Niels Dekker 81dfb4814d
Fix GitHub Actions build warnings, Marker style should be `*` (#1925) 4 years ago
Herb Sutter d5907d6dd5 Extended E.16 to include copy ctor for exception type, closes #1921 4 years ago
Niels Dekker 9ead2c44b4
SL.io.50 (Avoid `endl`): Mention string streams (#1920)
Explicitly mentioned string streams as `endl` insertions into string streams do actually occur in the wild.

With help from Sergey Zubkov.
4 years ago
Niels Dekker a534f4a168
F.16 ("in" parameters): Move Matrix example to F.20 (return values) (#1922)
The `Matrix` example and the notes about assignment appear off-topic in rule F.16, as F.16 is specifically about "in" parameters.

With help from Sergey Zubkov.
4 years ago
Markus Hofbauer ddef6cdbae
Move F.60 to align with TOC (#1914) 4 years ago
Sergey Zubkov e30543d06a NL dropping parenthesized bit, per commit comment 4 years ago
Sergey Zubkov 697a78b4f4 NL: rules-> suggestsions (closes #1906) 4 years ago
Sergey Zubkov dc8f95d608 F.46: add note regarding the return of main (see #1896) 4 years ago
Sergey Zubkov 4a386f0a65 [SF.21] adjust per editors call 4 years ago
bgloyer 0db2b88c57
Added example for SF.21 (#1890)
* Example for SF.21

* Update isocpp.dic

* remove space

* revert isocpp.dic change
4 years ago
William So d9d985a9f6
[C.148] Fix 1 anchor link (#1904) 4 years ago
Sergey Zubkov 66bdbf1a90
modernizing concepts (#1883) 4 years ago
Werner Henze df6787f651
formatting typo (#1895)
Co-authored-by: Werner Henze <w.henze@avm.de>
4 years ago
Sandor DARGO afa6f73f1a
formatting: Updates to I.4 (#1891) 4 years ago
apenn-msft ead60212f3
conditionally unused parameters can be declared using maybe_unused attribute. (#1863)
* conditionally unused parameters can be declared using maybe_unused attribute.

* move to Notes and add example

* whitespace?

* ok got the whitespace

* adjust example
4 years ago
al-mission-2016 eb2f4d7756
code formating fix: +1 new line (#1888)
One more new line is necessary after the text "Return statements in a case label are also OK:" to make the code excerpt after it be formatted as code.
4 years ago
Michael B. Price e92886217c
Adding .DS_Store to the .gitignore file since MacOS loves touching those files. (#1886) 4 years ago
Michael B. Price 256e6a939f
Remove irrelevant source file mistakenly committed (#1885) 4 years ago
Sergey Zubkov e01a72ee93 make two more mentions of supported standards agree with the Abstract 4 years ago
bgloyer 43f2e34148
E.14 exception type examples for #1852 (#1857)
* E.14

* clean-up

* add final

* Added words to dictionary

* cleanup

* fix typo

* Update CppCoreGuidelines.md
4 years ago
Sergey Zubkov 369beb8a0c make other mentions of supported standards agree with the Abstract 4 years ago
Hyuk Myeong 1a57ff0226
Fixed typos (#1876) 4 years ago
Melroy van den Berg 5f393d53af
Promote C++20 (#1880)
The documentation still mentions C++17, while C++20 is already out for some time. So I updated the introduction paragraphs.
4 years ago
Sergey Zubkov 171fda3597 F.7: de-lowercase two lines 4 years ago
Ryan A. Pavlik ba12aca973
Fix broken cross-references. (#1878) 4 years ago
Sergey Zubkov 11cc4c4775 E.15: typo in title 4 years ago
Sergey Zubkov fdbaff2bf7 [CP.2] link CP.102 list of books and add Herlihy-Shavit
closes #1725
4 years ago
Sergey Zubkov ce5341b02d merge ES.74 into ES.6, leaving a redirect behind
closes #1858
4 years ago
Kevin Boyette dc6ea376c4
ES.48: fix wording in reasoning (#1859) 4 years ago
oleksandrkozlov b6d7ab64ad
C.130: drop a destructor in the D class (#1855)
To follow the Rule of Zero.
4 years ago
Sergey Zubkov 620cf6278e C.130: drop final from last commit per meeting notes 4 years ago
bgloyer 86a42c29fc
Update CppCoreGuidelines.md (#1851) 4 years ago