Skip to content

[dcl.dcl][stmt.stmt] Remove tautonyms from top level stable labels #7179

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

Closed
wants to merge 1 commit into from

Conversation

AlisdairM
Copy link
Contributor

@AlisdairM AlisdairM commented Jul 28, 2024

Per comment on issue #5315.
Thanks to Jonathan Wakely for the notion of tautonyms.

Quickly force-pushed a correction to the comment identifying the PR number.

Copy link
Member

@jensmaurer jensmaurer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like that.

@jensmaurer
Copy link
Member

One check-output.sh error remains:

2024-07-28T21:03:49.8853617Z incorrect entries in xrefdelta.tex:
2024-07-28T21:03:49.8867284Z spurious diff.cpp23.dcl.dcl

@AlisdairM
Copy link
Contributor Author

Silly mistake corrected --- there is no diff.cpp23 in the C++23 standard itself!

@AlisdairM
Copy link
Contributor Author

Has this missed the clause revisions for C++26?

@tkoeppe
Copy link
Contributor

tkoeppe commented Oct 16, 2024

I'm not sure about this change yet, let's discuss at the meeting.

@AlisdairM
Copy link
Contributor Author

Squashed, rebased, and force-pushed for good measure.

Per comment on issue cplusplus#5315.
Thanks to Jonathan Wakely for the notion of tautonyms.
@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 17, 2024

Let's go with Jen's approach and leave the diff labels alone. We're causing enough upset as it is.

@tkoeppe tkoeppe closed this Dec 17, 2024
@AlisdairM AlisdairM deleted the remove_tautonyms branch December 17, 2024 23:14
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.

3 participants