Skip to content
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

[lex.separate][module.unit] move definitions of program and translation unit #7180

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AlisdairM
Copy link
Contributor

The definition of program at the top of [basic.link] should move to the front of [lex.separate] so that it is defined before its first usage, and also clarifies what the phases of translation produce.

Similarly, move the definition of the grammar production translation-unit to the top of the first clause to actually use it, [module.unit].

Finally, retitle [basic.link] as just Linkage, rather than prgrams and linkage.

@AlisdairM
Copy link
Contributor Author

Just to confirm that I looked at every reference for program and translation unit to see if they had a cross reference that should be updated. These terms occur quite frequently, but rarely have a cross-reference, and in those places that do have a cross reference, they are frequently cross referencing a concept in the whole sentence rather than a definition of either term.

@AlisdairM AlisdairM force-pushed the move_definition_of_program branch from 8c19937 to 86d979a Compare October 23, 2024 12:53
…on unit

The definition of program at the top of [basic.link] should move to
the front of [lex.separate] so that it is defined before its first
usage, and also clarifies that the phases of translation produce.

Similarly, move the definition of the grammar production translation-unit
to the top of the first clause to actually use it, [module.unit].

Finally, retitle [basic.link] as just Linkage, rather than
prgrams and linkage.
@AlisdairM
Copy link
Contributor Author

Note to CWG chair: this PR, if accepted, would seem to be making progress towards resolving http://wg21.link/cwg2670.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant