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.

Sign up for GitHub

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

Jump to bottom

P2874R2 Mandating Annex D #6327

Merged
merged 1 commit into from Jul 21, 2023
Merged

P2874R2 Mandating Annex D #6327

merged 1 commit into from Jul 21, 2023

Conversation

AlisdairM
Copy link
Contributor

@AlisdairM AlisdairM commented Jun 20, 2023 •
edited by jensmaurer

Applies all changes to replace 'requires' clauses in Annex D with more modern wording, per LWG Poll 19 at Varna 2023 meeting, paper P2874R2.

Fixes #6305
Fixes cplusplus/papers#1557

@AlisdairM AlisdairM mentioned this pull request Jun 20, 2023
Closed
@tkoeppe
Copy link
Contributor

tkoeppe commented Jun 20, 2023

You should assign the tracking issue #6305 to yourself to make sure nobody duplicates this work!

@jensmaurer jensmaurer added this to the post-2023-06 milestone Jun 20, 2023
@jensmaurer
Copy link
Member

Please fix xrefdelta.tex:

incorrect entries in xrefdelta.tex:
missing depr.res.on.required
Error: Process completed with exit code 1.

@jensmaurer jensmaurer added the changes requested Changes to the wording or approach have been requested and not yet applied. label Jun 20, 2023
@jwakely
Copy link
Member

jwakely commented Jun 21, 2023

The changes to the actual text look OK to me, once the error is fixed.

@AlisdairM
Copy link
Contributor Author

Sorry, under the weather and slow to reply.
The reason I did not immediately fix the removed stable tag entry is that I am not sure how best to proceed. That index is relative to C++20, so should we be resetting it for C++23, or continuing to expand the reference since C++20?

@jensmaurer
Copy link
Member

The editor should reset the stable tag reference to C++23. @tkoeppe

@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 21, 2023

I'll update xrefdelta after the motions, thanks!

@AlisdairM @tkoeppe
P2874R2 Mandating Annex D
3741ce0 
Replaces "Requires:" elements in Annex D with more modern wording.
@tkoeppe tkoeppe merged commit ae15824 into cplusplus:main Jul 21, 2023
2 checks passed
@tkoeppe tkoeppe removed the changes requested Changes to the wording or approach have been requested and not yet applied. label Jul 21, 2023
@AlisdairM AlisdairM deleted the p2874_mandating_annex_D branch November 14, 2023 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JohelEGP JohelEGP JohelEGP approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
post-2023-06
Development

Successfully merging this pull request may close these issues.

[2023-06 LWG Motion 19] P2874R2 Mandating Annex D P2874 R2 Mandating Annex D
5 participants
@AlisdairM @tkoeppe @jensmaurer @jwakely @JohelEGP