Skip to content

Fix P.5 first example #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 18, 2015
Merged

Fix P.5 first example #3

merged 1 commit into from
Sep 18, 2015

Conversation

Loghorn
Copy link
Contributor

@Loghorn Loghorn commented Sep 18, 2015

The shift instruction in the example should be <<= instead of >>=

The shift instruction in the example should be <<= instead of >>=
AndrewPardoe added a commit that referenced this pull request Sep 18, 2015
@AndrewPardoe AndrewPardoe merged commit 050b46b into isocpp:master Sep 18, 2015
@AndrewPardoe
Copy link
Contributor

Thank you! Good catch.

@AndrewPardoe AndrewPardoe mentioned this pull request Jun 27, 2017
hsutter added a commit that referenced this pull request Jul 13, 2022
* 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 <[email protected]>
Co-authored-by: Herb Sutter <[email protected]>

Co-authored-by: Niels Dekker <[email protected]>
Co-authored-by: Herb Sutter <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants