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

CWG Poll 3: P1857R3 Modules Dependency Discovery #3764

Merged
merged 1 commit into from Feb 25, 2020

Conversation

burblebee
Copy link
Contributor

@burblebee burblebee commented Feb 19, 2020

Fixes #3687.
Fixes cplusplus/papers#611.
Fixes cplusplus/nbballot#25.
Fixes cplusplus/nbballot#120.
Fixes cplusplus/nbballot#124.
Fixes cplusplus/nbballot#125.
Fixes cplusplus/nbballot#126.
Fixes cplusplus/nbballot#127.
Fixes cplusplus/nbballot#135.
Fixes cplusplus/nbballot#136.
Fixes cplusplus/nbballot#137.
Fixes cplusplus/nbballot#138.
Fixes cplusplus/nbballot#139.

Notes/issues/questions:

  • [lex.pptoken]/p4 Note says: "None have any observable spelling." What is this saying?
  • [diff.cpp17.basic] "The editor should consider a different section of annex C to move this to." Where to?
  • [basic.link] FYI, the grammar for private-module-fragment moved to [module.private.frag].
  • FYI, section [cpp.global.frag] is now [cpp.glob.frag].
  • FYI, section [module.global] is now [module.global.frag].

@jensmaurer jensmaurer added this to the post-2020-02 milestone Feb 20, 2020
@jensmaurer jensmaurer changed the title P1857R3 Modules Dependency Discovery (Motions 2020 02 cwg 3) CWG Poll 3: P1857R3 Modules Dependency Discovery Feb 21, 2020
Also fixes NB US 026, US 121, US 125, GB 126, US 127, US 128, US 136, US 137, US 138, US 139, and US 140 (C++20 CD).

[diff.cpp17.basic] Changed "m1" to "m2" in example.
[module.global.frag] Updated reference from [cpp.global.frag] to [cpp.pre].
[basic.link] Note: paragraph 3 (removed here) was modified by P1779R3 (CWG Poll 2).
[cpp.import] Replace non-existent import-directive with the intended pp-import.
Placed Annex C entry in [diff.cpp17.lex].
@zygoloid
Copy link
Member

Made a bunch more fixes and moved them to #3801 for separate merging.

@zygoloid zygoloid merged commit 960a7a9 into master Feb 25, 2020
@jensmaurer jensmaurer deleted the motions-2020-02-cwg-3 branch February 12, 2021 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment