Commit Graph

1761 Commits (a255fe23e0f6ad07d419b35851eec7b4af6a92fd)
 

Author SHA1 Message Date
hsutter a255fe23e0 Closes #1438 7 years ago
hsutter 10ff912f5c Closes #1432 7 years ago
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
7 years ago
shaneasd d456ccf274 Fix comment in C.128 example (#1449) 7 years ago
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
7 years ago
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
7 years ago
Herb Sutter 8a707c5274
DO credit 7 years ago
Herb Sutter 5d65a37863
Added DO credit 7 years ago
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.
7 years ago
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
7 years ago
Louis Cloete 6c92f514f4 Remove extra asterisk in example in C.60 (#1430)
closes #1429
7 years ago
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
7 years ago
Sergey Zubkov dbc554cbc5 update date 7 years ago
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
7 years ago
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
7 years ago
hsutter fc27313b75 Adopting fixes from PR 1411 7 years ago
hsutter 959b556aac Restored cached_computation description 7 years ago
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
7 years ago
hsutter 41b5bac211 Revert "Fixed typo in ES.22"
This reverts commit 976ee508a4.
7 years ago
hsutter 976ee508a4 Fixed typo in ES.22 7 years ago
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.
7 years ago
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
7 years ago
Aleksander 6a6321fcbf ES.49: added lvalue in std::forward description (#1404) 7 years ago
Aleksander 9f5a67fda7 ES.42: index in an example was not declared (#1403) 7 years ago
Paul Romano 1f6042f66c Fix typo in some of the NL rules, "thus rule" -> "this rule" (#1402) 7 years ago
hsutter 35cfe0c984 Closes #1397 7 years ago
hsutter 9275e7da09 Fixed typo in previous commit, closes #1395 7 years ago
hsutter 85e6aef5d6 Closes #1395 7 years ago
hsutter 274d65818d Closes #1392 7 years ago
hsutter 4b7cd81ad0 Closes #1392 7 years ago
Taewon Park 7ddf721500 Fix wrong <a> tags in Pro.bounds and Pro.lifetime (#1399)
Replaced "href" attribute to "name" of <a> tags in Pro.bounds and Pro.lifetime sections, to make the anchors to them work correctly
7 years ago
Sergey Zubkov 73f37745b1 fix typo 7 years ago
Saad 08659db9e1 MD typo fix (#1396) 7 years ago
hsutter c072184052 Closes #1370 7 years ago
hsutter 142fc6ad33 Closes #1367 7 years ago
Amir Livneh e3da8a1fd9 Add example for E.28 (#1385) 7 years ago
Dave Smith c2a5785d7e Remove elements that don't add value in ES.84 (#1390) 7 years ago
Dave Smith 8301421762 Tighten up the intro to CP (#1391) 7 years ago
Amir Livneh a8c7b7c5a8 Fix grammar in T.41 (#1389) 7 years ago
Amir Livneh bd3f3d5d25 Fix grammar in Con.2 (#1388) 7 years ago
Amir Livneh 422a190f57 Fix typo in T.22 (#1387) 7 years ago
Amir Livneh aa25be7d6b Make sentence in T.41 complete (#1386) 7 years ago
Amir Livneh ba2dbc5edf Fix calls to malloc() with 2 arguments (#1377)
* Fix calls to malloc() with 2 arguments
7 years ago
Amir Livneh f67e91d295 Use `memoizes` instead of `mnemonizes` in the context of caching (#1383)
* Use `memoizes` instead of `mnemonizes` in the context of caching

While apparently, 'mnemonizes' is a word, I don't think it's the best choice here.

* Update isocpp.dic
7 years ago
Amir Livneh c1beff1941 Fix grammar in T.11 (#1380) 7 years ago
Amir Livneh 43f4390185 Fix grammar in T.13 (#1379) 7 years ago
Amir Livneh 8ff099d2d2 Fix grammar (#1378) 7 years ago
Amir Livneh 1a3e4040fa Remove extra space from README (#1376) 7 years ago
Amir Livneh 7d092e37aa Remove extra parenthesis (#1375) 7 years ago
Amir Livneh 7511b40996 Use 'an' instead of 'a' where appropriate (#1374) 7 years ago