Commit Graph

1747 Commits (es23-updates)
 

Author SHA1 Message Date
Stephan Weller 2c3d9921b6 Replace erroneous public keyword by private in C.129. (#1166) 8 years ago
beinhaerter edf13aabc0 Note that C.46 does not apply for copy/move ctors (#1169)
* Note that C.46 does not apply for copy/move ctors

* moved Note section before Enforcement section
8 years ago
Gabriel Dos Reis 644e0f5b5e
Merge pull request #1164 from ChrisGuzak/ChrisGuzak-sizeof-not_max
use sizeof(buffer) instead of MAX
8 years ago
echeij 6ed138076d Fixed get_string example in F.21 (#1160)
In the 'all values as return value' example in F.21, the passed istream was not used in the function.
8 years ago
Chris Guzak 373f4c4808
use sizeof(buffer) instead of MAX 8 years ago
nnaka 09035eb1e4 Align comment block (#1162) 8 years ago
nnaka 9c4d4247b1 P.5 example read int array size mismatch (#1161)
Expand comment to clarify problem.
8 years ago
Andrew Pardoe b487f9790a subject-verb agreement in NL.2 8 years ago
hsutter 8d5d5e4aeb To address #1147
Changed `.length()` to `.size()` for `span` examples to track
standardization changes.
8 years ago
Andrew Pardoe beed3e97c0 update date 8 years ago
Ari Hietanen 6c55d4eaaf Bug fix C4 class Foobar (#1143)
* Fixed bug in example code of C4 Expecptions. In C4, the class function void Foobar::foo(double x) is supposed to call the overloaded void Foobar::foo(int x), but in the call foo(std::round(x)), std::round returns a double. Hence, it will get stuck in an infinite recursive loop. Added static_cast<int>(..) to enforce the call to right overload. Added also keyword public to be more consistent.

* Changed static_cast<int> to narrow_cast<int> following ES.46.

* Modified C4 Foobar class, s.t, std::lround(x) is now called in
void foo(double) and the overload is changed to void foo(long) from (int). Now there is
no need for conversions.
8 years ago
Tobias Mende da3b6b98bc Do Not Check For nullptr Explicitely In Examples (closes #1142) (#1144) 8 years ago
beinhaerter 9c98f9679c Copy-Paste-Typo in Comment for ES.87 sample (==0/!=0) (#1145) 8 years ago
Sergey Zubkov 01e59d5452
added ofstream to dictionary for travis CI 8 years ago
hsutter 10c0a8b156 Closes #1119
Make C.43 crisper -- the guideline is that default construction is
required for copyable types. A lot of the existing examples then just
fall out, without having to be presented as special cases. This was the
original intent and I think this new text helps make that clearer.
8 years ago
Andrew Pardoe d9f3149e11 Update references to C++17, also typo 8 years ago
Dipl.-Ing. Raoul Rubien, BSc 73dad7909d title page/In.sec: reordered and updated titles (#1135)
* title page: updated titles and unified titles' camel case to sentence case

* In.sec: reordered bullets to match document's workflow, updated titles

* updated several links and link names, modified names to title case

* redirected broken link to new anchor Rf-value-return at 'value return sematic rules' block

* updated links

* reverted unindended broken artistic prose

* updated links, modified "see also's" formatting to comply with the majority

* modified "see also's" formatting to comply with the majority

* fix spellcheck: 'Componentization -> Compartmentalization'

* updated link name
8 years ago
Bernardo Sulzbach eb534411a7 Should say repository instead of depository (#1138) 8 years ago
Dipl.-Ing. Raoul Rubien, BSc 888067d6ff fixed spellcheck/lint issues on running make in ./scripts/ (#1134) 8 years ago
Sergey Zubkov 25d9d12e15 travis CI fixes 8 years ago
hsutter f8f396ef3b Typo fix to NL.5 8 years ago
hsutter c5f67599d1 Added another example to NL.5 8 years ago
hsutter c10e5503e9 Closes #1104 8 years ago
Sergey Zubkov e3f753ed14 the copy ctor typo and comment fixes from #1125 8 years ago
Andrew Pardoe 15ca9d220c Fixing typo per #1130 & updating date 8 years ago
Herb Sutter c1e3cd01cd Add gsl::index, closes #1098 (#1115)
* Add gsl::index, closes #1098

And update examples throughout to use `index` as appropriate

* Actually adding `index` to the GSL.util section

* Added `sizeof` to whitelisted signed/unsigned comparisons

Same reason as container `.size()` -- better backward compatibility with
the existing standard
8 years ago
Oleg Artenii b677b69014 Removed useless "Currently (July 2016)" (#1123)
* Removed useless "Currently (July 2016)"

* replaced `GCC 6.1` with `GCC 6.1 or later`

* Correct usage of `and/or` in `GCC 6.1 _ later`
8 years ago
rknuus 2bdd1ae705 add example for R.24 (#1127)
* add example for R.24

taken from https://cpppatterns.com/patterns/weak-reference.html, where
it's published under CC0 Public Domain Dedication

* improve example for R.24
8 years ago
Alexey Porotnikov 9de66ec027 add "return" to the compound literal exclusions (#1113)
inhibits readability/brace warning for compound literals in return
statement
8 years ago
Andrew Pardoe 999f9dd0ce Adding "ptrdiff" to travis dictionary 8 years ago
Jonathan Wakely 5c3006b60f Change "untended" to "unintended" in C.9 8 years ago
Herb Sutter 7aac5313d9 Correct example comment 8 years ago
Herb Sutter e1cc6fedb0 In docs\gsl-intro, cleaned up stray notes into a "hints" section 8 years ago
Sergey Zubkov 59eb77027e travis CI fixes 8 years ago
Sergey Zubkov 7dc8aaa420 travis CI fixes 8 years ago
Bjarne Stroustrup 8f87a03588 ES.87
#1032 comment and example added.
To consider: should integer comparison be broken out as its own rule?
8 years ago
Bjarne Stroustrup 72b9b40a7c Some examples in Per.11
In response to #936 I added some text to Per.11. More would be welcome
8 years ago
Bjarne Stroustrup edbfc3b8ec Clarifying (I hope) text and example added
addressing #980 and #9777
8 years ago
Bjarne Stroustrup 0df2c26e9a C.129
#1052 added Jonathan's example
8 years ago
Bjarne Stroustrup ff42e7bbaf closing #547
(now?) we have CP.3 and CP.31. Also CP.mess.
Yes we need more work on CP.mess
8 years ago
Bjarne Stroustrup ecabc369bb simplifying ES.20
#488
8 years ago
Bjarne Stroustrup 7e5019378b Fix #493 array myth
Added comment about performance to SL.con.1
8 years ago
Andrew Pardoe 7cfb1d7961 update date 8 years ago
Mitch Phillips bd948ee839 Fix example code in F.20. (#1109) 8 years ago
michaelz85 044a4b277e fixed anchor name for T.21 (#1108) 8 years ago
michaelz85 7c11ac1acf fixed a word (should be 'that' not 'the') (#1107) 8 years ago
michaelz85 b5a20f10ea fixed error in code comment (#1105) 8 years ago
Oleg Artenii 352e94babe fixed minor typo: "on the on the" -> "on the" (#1103) 8 years ago
Andrew Pardoe d3a523eea8
Typos found at I.* (#1093)
* Typo: change word compute by inserted code .  is the name of a previous function.

* Typo: add missing comma

* Typo: minor period typo

* Typo: form lower to upper case

* Typos found at I.*

Typo: add missing comma

Typo: minor period typo

Typo: form lower to upper case

Typos found at I.*
8 years ago
Andrew Pardoe 3667b08ce3
Merge pull request #1095 from brenoguim/hotfix/public-deleted-methods
Add public access modifier to examples
8 years ago