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
[ranges] Missed kebab-case markings for exposition-only names. #3562
Conversation
4216331
to
0d2aaac
Compare
Some of the suggestions in the original PR are missing a fix here. |
I thought I had them all covered. Anything specific? |
Ok, I hope I got them all this time. |
Oh, the index question is still open. I'll fix the spelling, but leave the italics for another day (needs more gymnastics with the indexing macros). And yes, the question about the inconsistent initialization with the istream iterator stuff is disturbing. |
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.
2 changes. Otherwise, LGTM (other than \placeholder
vs \placeholdernc
).
@@ -5009,8 +5009,8 @@ | |||
\pnum | |||
Let \tcode{\placeholder{inner-range}} be: |
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.
Let \tcode{\placeholder{inner-range}} be: | |
Let \exposid{inner-range} be: |
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.
This is really a placeholder for some code, not an exposition-only member. Leaving as-is.
Fixup for #3559.