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
[streambuf.virt.get] use half-open ranges in underflow effects #1157
Conversation
Oh well. This seems to be a presentational improvement. Is that the only place in streambufs where we have that style of description? I wouldn't want to address only one of possibly N adjacent places. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jwakely: Ping - a comma is required.
characters starting at | ||
\tcode{gptr()}, | ||
characters in | ||
\range{gptr()}{egptr()} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You shall retain the final comma here, otherwise the "otherwise" would subsume the next bullet!
I like it. Looks good to merge once @tkoeppe's comment is addressed. |
9bab4d4
to
402b0ae
Compare
Rebased and added the comma. |
I should probably also look at
|
@jwakely: Yes, sounds good. We don't come past this section very often, so getting all the cleanup done in one go seems like a good idea |
Taking this part for C++17; I'd rather get this in than wait for more cleanups to be added to it. |
We should discuss this before merging, I'm ambivalent about it.