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.

Sign up for GitHub

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

Jump to bottom

[range.istream.iterator] Fix typo #3661

Merged
merged 1 commit into from Feb 5, 2020
Merged

[range.istream.iterator] Fix typo #3661

merged 1 commit into from Feb 5, 2020

Conversation

pppalka
Copy link
Contributor

@pppalka pppalka commented Feb 5, 2020

The description of basic_istream_view::iterator::operator*() refers to a
nonexistent data member value_ of basic_istream_view instead of the data
member object_.

CaseyCarter
CaseyCarter reviewed Feb 5, 2020
View reviewed changes
Copy link
Contributor

@CaseyCarter CaseyCarter left a comment

Choose a reason for hiding this comment

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

The change itself is good, but the subclause stable name is more precisely [range.istream.iterator].

@pppalka pppalka changed the title [range.istream] Fix typo [range.istream.iterator] Fix typo Feb 5, 2020
[range.istream.iterator] Fix typo
7942c0b 
The description of basic_istream_view::iterator::operator*() refers to a
nonexistent data member value_ of basic_istream_view instead of the data
member object_.
@pppalka
Copy link
Contributor Author

pppalka commented Feb 5, 2020

Thanks, the subclause stable name in the commit message has been adjusted via force-push.

@jensmaurer jensmaurer merged commit 694d84c into cplusplus:master Feb 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CaseyCarter CaseyCarter CaseyCarter approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@pppalka @CaseyCarter @jensmaurer