StarMirror/CppCoreGuidelines
1
0
Fork 0
mirror of https://github.com/isocpp/CppCoreGuidelines.git synced 2024-03-22 13:30:58 +08:00
Code Issues Projects Releases Wiki Activity
1,859 Commits 14 Branches 4 Tags 216 MiB
9efcaf07f7
Commit Graph

1859 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Anthony Cassidy
9efcaf07f7
Removed anti-CamelCase comments. (#1632) 
The comments were:
// ???Must we suffer CaMelcAse???
and
// ugly CaMeLcAsEvArIaBlE
 2020-06-04 11:19:47 -07:00
xtofl
4867a5c2d3
Use '<thing> template' i.s.o. 'template <thing>' (#1631) 
* Use '<thing> template' i.s.o. 'template <thing>'

The word 'template' is often wrongly used as an adjective to the 'thing'
it becomes when instantiated.

* Expand succinct formulation for readability

Co-authored-by: Kristoffel Pirard <kristoffel.pirard@vanhool.com>
 2020-06-04 11:18:37 -07:00
rico-chet
1936cb6b44
[NL.{11,15}] Bring rules in order (#1629) 
The rules NL.11 (Make literals readable) and NL.15 (Use spaces
sparingly) are placed in inverted order in the text flow, swap them.
 2020-06-02 15:23:14 -04:00
Sergey Zubkov
4625c984e9 bump date 2020-05-28 15:14:02 -04:00
Sergey Zubkov
24c3993e97 fix CI, fix heading and add comment requested at today's editor's call 2020-05-28 15:06:52 -04:00
Arthur O'Dwyer
d56d67969c
Expand CP.61 to talk about the general "factory" pattern. (#1621) 2020-05-28 15:04:34 -04:00
Arthur O'Dwyer
e8e0d103c0
CP.1: Simplify example, show good example, expand on rationale (#1615) 
"Make your code thread-safe" usually means "don't use global state."
Advice to replace global state with `thread_local` state is usually misguided.
https://quuxplusone.github.io/blog/2018/11/14/fiber-local-storage/
 2020-05-28 11:57:18 -07:00
Hyuk Myeong
19f2672b9e
Adjust the spacing for consistency (#1626) 
* Adjust the spacing for consistency

* Adjust the spacing for consistency-2
 2020-05-28 11:41:32 -07:00
rico-chet
9960c7dd7f
[SF.10] Fix annotation of the _bad_ example (#1627) 
The _bad_ example wasn't annotated as such and a perfectly fine function
name was marked bad.

Annotate the example as bad and remove the misleading function name
annotation.
 2020-05-28 11:37:50 -07:00
Moritz 'Morty' Strübe
f604a6b5d6
[F.36] Typo (missing s) (#1623) 2020-05-20 11:34:11 -04:00
Arthur O'Dwyer
6e0f8ace0b
Tweak one of ES.10's examples for verisimilitude. (#1611) 
For some reason the trailing `e = 3` seemed more of a red flag
than anything else about this line. Let's imply that the programmer
is trying to make some constants for hexadecimal translation.
 2020-05-14 11:43:48 -07:00
Dale Visser
46dadd0b16
Modified slightly impolite language (#1601) 
* dumb -> inefficient
 2020-05-07 11:28:14 -07:00
Yaro Shkvorets
97797870ad
code typo in R.20 (#1619) 2020-05-06 08:41:10 -04:00
Sergey Zubkov
986402298a travis CI fixes 2020-04-22 09:35:16 -04:00
Arthur O'Dwyer
d45496ff5e
You should never =delete a destructor! (#1605) 2020-04-20 08:59:37 -04:00
Sergey Zubkov
9928c95f97 typos, wording, links (closes #1529) 2020-04-02 11:26:53 -04:00
hsutter
b4bb11a833 Tweak to close #1588 2020-03-26 11:47:45 -07:00
shaneasd
040b141acf
Clarify ES.48 enforcement rule for void cast of [[nodiscard]] (#1588) 2020-03-26 11:47:19 -07:00
hsutter
1ca5545ce1 Followup to close #1586 2020-03-26 11:43:27 -07:00
Akash Mozumdar
6f01c706a8
Actually detect negative sizes by following ES.106 (#1586) 
* Actually detect negative sizes by following ES.106

And don't use senseless one letter names

* fix grammar

Co-Authored-By: Johel Ernesto Guerrero Peña <johelegp@gmail.com>

Co-authored-by: Johel Ernesto Guerrero Peña <johelegp@gmail.com>
 2020-03-26 11:38:07 -07:00
Sylwester Fraczek
d2be4cba57
F.21 remove ; in function definition (#1592) 2020-03-24 12:22:45 -04:00
Sergey Zubkov
ab91388efd
fix mismatched anchors, closes #1569 (#1581) 2020-03-19 11:34:50 -07:00
Akash Mozumdar
aea393c85c
C.180 example bugfix (#1587) 
new[] needs delete[]
 2020-03-16 08:27:00 -04:00
Adnn
2f0c2e5874
C.22 Fixes a mistake in the code example not making a deep copy (#1573) 2020-02-27 20:18:08 +01:00
hsutter
fc383d6641 Updated #1570 
Typo
 2020-02-27 11:16:46 -08:00
Chris Guzak
63ceef65e3
Add SF.12, 3rd try (#1570) 
* add SF.12

* add incscope to isocpp.dic to get the CI build to pass

* expand INCLUDES, update dictionary for the ci build to pass

* pr feedback

* in the same directory

* update based on feedback

* 3rd try

* PR feedback

* update lable

* Update CppCoreGuidelines.md

* Update CppCoreGuidelines.md
 2020-02-27 20:16:10 +01:00
hsutter
6388b4d1ea Followup to PR 1564 
Changed the two anchors back (anchors need to stay stable and don't
display visibly anyway)
Switched "non-public" back to hyphernated (to use hyphenation
consistently)
 2020-02-27 11:14:04 -08:00
Hyuk Myeong
ab0255b371
Add and remove hyphens to words that begin with non (#1564) 
* Add hyphens to words that begins with a non

* Remove hyphens in nonpublic
 2020-02-27 20:10:24 +01:00
roroberto
1429d9ec71
Added example to CP.61: Use an async() to spawn a concurrent task (#1557) 
Added example to CP.61: Use an async() to spawn a concurrent task
 2020-01-30 14:45:04 -05:00
Herb Sutter
6e72adc0fe
Casting away const isn't undefined behavior, but modifying a constant is (#1509) 
* Casting away const isn't undefined behavior, but modifying a constant is

You can cast away `const` as much as you like, as long as you never write to variable.

* rewording for clarification
 2020-01-30 11:28:19 -08:00
hsutter
2eb8ea9279 Tweaked #1516 one more time 2020-01-30 11:27:07 -08:00
hsutter
fd857db583 Tweak to #1516 per editors call 2020-01-30 11:24:43 -08:00
kuzkry
745b0664c2
clarify that GSL semi-specification of some types does not reflect actual support of them (#1516) 2020-01-30 11:23:29 -08:00
hsutter
dc8e49314f Tweaked PR #1553 per editors call 2020-01-30 11:21:00 -08:00
Kim Viggedal
153da17417
Merge I.2 and R.6 seem to address the same issue, (#1553) 
thus I propose removing R.6 and moving some of its contents into I.2.
 2020-01-30 11:19:22 -08:00
hsutter
9a4db6c1c4 Adopt the change suggested in PR #1556, just making the casing consistent the other way (lowercase) 2020-01-16 11:52:25 -08:00
hsutter
b01d9e0d23 Updated MISRA entry with link, updating PR #1554 2020-01-16 11:45:48 -08:00
Mark Nelson
605a48ae03 Added two coding references (#1554) 
* Added two commonly used coding guideline documents

* Update CppCoreGuidelines.md
 2020-01-16 11:44:50 -08:00
hsutter
f92c1c2eb3 Tweaked #1551 
To adjust example comment and Enforcement per PR #1551 comments
 2020-01-16 11:28:21 -08:00
Baptiste Thémine
164c0b3b5d Add code examples to CppCoreGuidelines ES.77 (#1551) 2020-01-16 11:26:23 -08:00
Hyuk Myeong
5c065f850c Fix typos (#1549) 
* R.37 : it callees -> its callees
* ES.65 : a different part from the original paper
 2020-01-16 11:05:11 -08:00
Sergey Zubkov
c68a395830 [I.27] fix move ctor in the pimpl example (closes #1471) 2020-01-15 16:38:44 -05:00
Sergey Zubkov
df94731290 update LLVM coding standards description, closes #1502 2020-01-15 16:34:23 -05:00
Bernd Ritter
778b44c2c9 Corrected link to noexcept anchor (#1559) 2020-01-09 13:01:02 -05:00
Sergey Zubkov
282ff88e57 fixes to pass travis checks 2019-12-20 16:37:06 -05:00
Sergey Zubkov
9e9950353a typos noted in comments to 2fdc696 2019-12-20 16:26:14 -05:00
shaneasd
1e371f7f86 Fix trivial recent typos (#1550) 2019-12-20 16:23:25 -05:00
Bjarne Stroustrup
ad30d9ee16 Update CppCoreGuidelines.md 
Clarified the enforcement of C.121 according to #1368
 2019-12-08 17:36:43 -05:00
Bjarne Stroustrup
7db3ad84ff Update CppCoreGuidelines.md 
REsolving #1500 with an explanatory note in the introduction.
 2019-12-08 15:09:15 -05:00
Bjarne Stroustrup
2fdc6960b4 Update CppCoreGuidelines.md 
Adding rule to resolve #1424
 2019-12-08 14:54:19 -05:00