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
[over] Replace \term with \defnx to add index entries #1881
Conversation
source/overloading.tex
Outdated
An | ||
\term{implicit conversion sequence} | ||
\indextext{sequence!implicit conversion}% | ||
An \defnx{implicit conversion sequence}{sequence!implicit conversion} |
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.
Yuck. Can we index this as "conversion sequence!implicit" rather than "sequence!implicit conversion"? Also, a \see from "implicit conversion sequence" would be really useful.
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.
Fixed.
source/overloading.tex
Outdated
@@ -2484,18 +2475,18 @@ | |||
\pnum | |||
This subclause defines a partial ordering of implicit conversion | |||
sequences based on the relationships | |||
\term{better conversion sequence} | |||
\defnx{better conversion sequence}{conversion!sequence!better} |
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.
I think we should keep "conversion sequence" separate from "conversion" in the index, since they have quite different sets of rules.
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.
Fixed.
Addressed @zygoloid's review comments. |
Partially addresses #329.