Commit Graph

1931 Commits

Author SHA1 Message Date
beinhaerter
71a0419a1a C.145: add "see also" C.67 (#1460) 2019-08-01 11:19:16 -07:00
Chris Guzak
2895429f55 exception in SF.7 for issue 1440, allow std::literals (#1457)
* exception in SF.7 for issue 1440, allow std::literals

* PR feedback

* add UDLs to spelling dictionary

* put in alpha order
2019-08-01 11:16:46 -07:00
Werner Henze
e7cdce445a SF.10: remove double #include <vector> from example 2019-07-29 10:25:03 +01:00
Sergey Zubkov
d35d76c3e3 closes #1479 2019-07-23 09:10:19 -04:00
beinhaerter
af69b85e10 same capitalization for "Technical Specification" on all occurances (#1482) 2019-07-23 09:06:33 -04:00
beinhaerter
d46c678048 E.18: fix grammar (#1481) 2019-07-22 16:00:36 -04:00
cclauss
622568271a Travis CI: (#1469)
[Travis are now recommending removing the __sudo__ tag](https://blog.travis-ci.com/2018-11-19-required-linux-infrastructure-migration).

"_If you currently specify __sudo: false__ in your __.travis.yml__, we recommend removing that configuration_"
2019-07-09 17:07:32 -04:00
beinhaerter
dc72881cf3 ES.1: fixed wrong capitalization (#1468) 2019-07-09 17:06:58 -04:00
Tyler Young
5e636657d9 Fixes typo: "code based" -> "code bases" 2019-07-09 10:12:50 +01:00
beinhaerter
fbc9db56f0 R.3: "a" -> "an" (#1465) 2019-07-08 10:20:39 +01:00
hsutter
d8a6aaa40b Restored two more edits 2019-07-03 08:33:09 -07:00
hsutter
024f1a05dc Restored accidentally overwritten editorial changes 2019-07-02 14:29:25 -07:00
hsutter
514ac4487c Minor cleanup on the smart pointer concept, and addresses #1466 2019-07-02 12:56:31 -07:00
beinhaerter
85ae628765 Enum.2: typo/formatting (#1463) 2019-07-01 17:00:07 -04:00
beinhaerter
fb320822fc typo in C.181: 'it' -> 'is' (#1462) 2019-07-01 20:16:38 +01:00
beinhaerter
0a58ed2bc3 C.150/C.151: fix compiler syntax error in Example (#1461)
- remove unnecessary space in section Reason
2019-07-01 09:56:53 -04:00
beinhaerter
a70d903fde C.102: format note as text, not as code (#1459) 2019-07-01 09:55:50 -04:00
Jonathan Wakely
dbbc2589b2 Remove stray .eml files 2019-06-26 15:56:10 +01:00
hg
1cd545aed5 Remove extra backtick in ES.23 (#1453) 2019-06-25 13:30:38 -04:00
hsutter
8eea7cef63 Closes #1445 2019-06-20 11:54:25 -07:00
hsutter
a255fe23e0 Closes #1438 2019-06-20 11:40:26 -07:00
hsutter
10ff912f5c Closes #1432 2019-06-20 11:34:16 -07:00
Joshua T. Fisher
1d54448aeb Editing P.9: "Don't waste time or space" Example Text (#1439)
* Initial rewrite

* Fixed a couple of inaccuracies and minor grammar mistakes

Thanks to twitter user @lunasorcery for these changes!

* Added toLower to dictionary
2019-06-20 11:12:29 -07:00
shaneasd
d456ccf274 Fix comment in C.128 example (#1449) 2019-06-18 10:22:50 -04:00
beinhaerter
4b414458cf I.13: grammar and hint to std::string_view (#1443)
- "a C-style, zero-terminated strings" is wrong, it must be either "C-style, zero-terminated strings" or "a C-style, zero-terminated string"
- added hint to `std::string_view
2019-06-13 08:38:55 -04:00
beinhaerter
040ea419cc I.10: structured bindings is now available (#1442)
- C++17 is already available
- synchronize with F.21:
  - put quotes around "structured bindings"
  - remove link to proposal
2019-06-13 08:38:09 -04:00
Herb Sutter
8a707c5274
DO credit 2019-06-08 14:06:30 -07:00
Herb Sutter
5d65a37863
Added DO credit 2019-06-08 14:05:34 -07:00
alexcamposruiz
2d40c3ac2c Add example code for T.48 (#1422)
* Add example code for T.48

* Fix whitespace in end of line

* Use better syntax for concept constraint

* Revert "Use better syntax for concept constraint"

This reverts commit f071920d7f.
2019-05-30 11:21:43 -07:00
jkorinth
4c35d4c022 Fix C.120 good example (#1426)
C.120 has a good example which violates C.128 by specifying both virtual and override for methods.
closes #1425
2019-05-28 11:11:33 -04:00
Louis Cloete
6c92f514f4 Remove extra asterisk in example in C.60 (#1430)
closes #1429
2019-05-28 09:01:27 -04:00
Louis Cloete
29dedc49af Inserts a missing backtick in C.49 (#1428)
Below heading "Example, better still" there was a missing backtick after gsl::string_span
2019-05-20 08:14:08 +01:00
Sergey Zubkov
dbc554cbc5 update date 2019-05-02 14:53:28 -04:00
Herb Sutter
9948bdc157
Update ES.23 to allow = initialiization (#1416)
* Update ES.23 to allow = initialiization

* Silencing Travis

* Changed title back to original, it's fine

* Add note about explicit
2019-05-02 11:47:00 -07:00
Dave Smith
a9242c8dae Improve A.all (#1413)
* reword the A.1 rule title
* add candidate content for the A.1 rule
* make minor improvements to the A.2 note
* simplify wording in the first bullet of A.4's Reason
2019-05-02 11:18:07 -07:00
hsutter
fc27313b75 Adopting fixes from PR 1411 2019-05-02 11:12:08 -07:00
hsutter
959b556aac Restored cached_computation description 2019-05-02 11:06:03 -07:00
Dave Smith
1a9a35d2d9 Tighten up CP.1 (#1405)
* Tighten up CP.1

* balanced verb usage in first sentence
* changed third sentence to "libraries not using threads", as I
  believe this was the original author's intended meaning.
* clarified "this" in fourth sentence
* cut wordiness of "thanks to the magic of cut-and-paste", as it
  added no value
* changed "Example" heading to "Example, bad"
* added "bad:" comment above statics in the example
* added an explanatory sentence immediately after the example
* changed "works perfectly in a single-threaded" after example to
  "works as intended in a single threaded".  Also balanced the
  structure of the two comma separated phrases inside this sentence.
* strengthened parenthetical explanation in second bullet of "could
  be made safe" section

* Correct grammar mistake pointed out by @cubbimew

* Remove specific cache details in CP.1 per @hsutter's request
2019-05-02 11:05:35 -07:00
hsutter
41b5bac211 Revert "Fixed typo in ES.22"
This reverts commit 976ee508a4.
2019-04-27 07:47:59 -07:00
hsutter
976ee508a4 Fixed typo in ES.22 2019-04-27 07:45:45 -07:00
Florian Thake
385199cc90 Added bad and good example to NR.5 in CppCoreGuidelines.md (#1401)
* Added bad and goof example to NR.5 in CppCoreGuidelines.md

Added bad and good example to NR.5 Don’t: Don’t do substantive work in a constructor; instead use two-phase initialization.
I think it could be suitable.

* adjusted coding style

* removed extra space

* removed one more whitespace

* removed spaces before note to make it a blank line

* made Cleanup method from bad example return void

* some changes after review comments

- removed try catch
- removed defaulted dtor
- changed int to size_t, removed check for even.
- Expects() for invariant check
- typo

* spell check adjustment

* moved comment up for met the line length

* changed variablename in good example

... they were named same after removed the try catch scope

* changed afer comments

- changed check_size() function to a static member function 
- fixed comment mentioning the default contract violation behavior.
2019-04-25 11:21:50 -07:00
Kyle
0f57785d2b C.129 Small fixes (#1406)
* C.129 Fix typos and conjugation

I noticed some grammatical errors in this section and fixed them to match my interpretation of the author's intention.

* One more fix

Pluralization
2019-04-15 14:06:50 -04:00
Aleksander
6a6321fcbf ES.49: added lvalue in std::forward description (#1404) 2019-04-15 14:05:36 -04:00
Aleksander
9f5a67fda7 ES.42: index in an example was not declared (#1403) 2019-04-15 13:22:20 -04:00
Paul Romano
1f6042f66c Fix typo in some of the NL rules, "thus rule" -> "this rule" (#1402) 2019-04-10 10:45:57 -04:00
hsutter
35cfe0c984 Closes #1397 2019-04-04 11:28:59 -07:00
hsutter
9275e7da09 Fixed typo in previous commit, closes #1395 2019-04-04 11:27:40 -07:00
hsutter
85e6aef5d6 Closes #1395 2019-04-04 11:26:37 -07:00
hsutter
274d65818d Closes #1392 2019-04-04 11:16:34 -07:00
hsutter
4b7cd81ad0 Closes #1392 2019-04-04 11:16:05 -07:00