Skip to content
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

Editorial review of Clause 27 [input.output] #891

Merged
merged 6 commits into from Aug 8, 2016

Conversation

tkoeppe
Copy link
Contributor

@tkoeppe tkoeppe commented Aug 6, 2016

This is @AlisdairM's work, just repackaged.

@@ -5916,6 +5916,19 @@
\postcondition
\tcode{rdbuf() == sb}.

\indexlibrary{\idxcode{basic_ostream}!constructor}%
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AlisdairM: Same question here, how come the synopsis does not list the move constructor?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a mis-application of http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2005/n1862.html after the Oxford 2007 meeting that no-one had noticed yet! I think the declaration in the class definition could be added editorially, given it was present in the original paper.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, and the other half of the mystery resolved! LWG 911 made these members protected, which is why they show up at the /end/ of the class definition - but the declarations are present and correct.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahhh - thanks! I added reference comments to the synopsis, too.

@tkoeppe tkoeppe merged commit 2ab1a39 into cplusplus:master Aug 8, 2016
@tkoeppe tkoeppe deleted the alisdair27 branch August 8, 2016 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants