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
[diff.basic] Fix format and inexact wording #789
Conversation
6f5c913
to
bbb0244
Compare
@FrankHB: Can you please rebase (and sorry for the force-pushes on master)? |
e140b61
to
167df7c
Compare
Though Annex C is informative, it's better to keep the terminology similar to normative text, which is easier to refer.
values for each \tcode{const}. | ||
This feature allows the user to put \tcode{const}objects in header files that are included | ||
in many compilation units. | ||
for each \tcode{const} 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.
The grammar doesn't work here: "provide explicit initializer for each const
object." Maybe "provide an explicit initializer[...]"?
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.
\Cpp, this feature urges programmers to provide an explicit initializer | ||
for each \tcode{const} object. | ||
This feature allows the user to put \tcode{const} objects in headers or source files that are included | ||
in more than one translation units. |
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.
Shouldn't it say "more than one unit", singular?
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 are right.
Signed-off-by: FrankHB <frankhb1989@gmail.com>
Because \tcode{const} objects may be used as values during translation in | ||
\Cpp, this feature urges programmers to provide an explicit initializer | ||
for each \tcode{const} object. | ||
This feature allows the user to put \tcode{const} objects in headers or source files that are included |
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.
Say just "objects in source files"; @zygoloid says that "users have no control over headers". Then I'll merge this.
Signed-off-by: FrankHB <frankhb1989@gmail.com>
Though Annex C is informative, it's better to keep the terminology similar to normative text, which is easier to refer.