-
Notifications
You must be signed in to change notification settings - Fork 772
[ostream.formatted.print] Use \impldef macro. #6423
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
base: main
Are you sure you want to change the base?
[ostream.formatted.print] Use \impldef macro. #6423
Conversation
The text defines implementation-defined behavior, but it doesn't use the appropriate macro. So the text is not available in "Index of implementation-defined behavior".
See #5693 (comment) for how this macro is used. |
I think this is also similar to #6359 (comment). |
Thanks for the links, searching didn't show them. After reading the links I'm still not sure why this is not implementation-defined behavior. The way I read it is "Whether |
@mordante I think the point is that the macro expands to the words "implementation-defined", and the macro argument is the explanatory text that goes into the index. You can just build the PDF with your changes and check what you get (both in the text and in the index)! |
\tcode{os} is a stream that refers to a terminal capable of displaying Unicode | ||
which is determined in an implementation-defined manner, | ||
\tcode{os} is a stream that refers to \impldef{a terminal capable of displaying | ||
Unicode} which is determined in an implementation-defined manner, |
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.
Specifically, this appearance of the word "implementation-defined" must be replaced by \impldef{text to appear in the index}
The text defines implementation-defined behavior, but it doesn't use the appropriate macro. So the text is not available in "Index of implementation-defined behavior".