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
[intro.defs] Remove cross-references to main part #4191
Conversation
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.
Maybe extend the commit message to say "cross-references from the Terms and Definitions to the main part".
read\iref{conv.lval} or | ||
modify (\ref{expr.ass}, \ref{expr.post.incr}, \ref{expr.pre.incr}) | ||
the value of an object |
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.
Can we add some text to the note below to retain these useful xrefs? Perhaps add between the first and second sentence:
Reads of scalar objects are described in \ref{conv.lval} and modifications of scalar objects are describred in \ref{expr.ass}, \ref{expr.post.incr}, and \ref{expr.pre.incr}.
I think it's important to the use of this definition that we know what these four cases are.
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/intro.tex
Outdated
templates that take two template arguments | ||
and are declared in header \libheader{iosfwd} |
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.
Can we keep the two conditions the other way around? It makes more sense to me to start with all templates in <iosfwd>
and then filter by "takes two arguments" than to start with the universe of all templates taking two arguments and then filter by "declared in <iosfwd>
".
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.
Definitions of terms cannot refer to subclauses in the main body of the standard. Partially addresses ISO/CS 016 (C++20 DIS)
Partially addresses ISO/CS 016 (C++20 DIS)
Fixes cplusplus/nbballot#397