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

[dcl.attr.grammar] Clarify that only non-standard attributes can be ignored CWG2538 #5133

Closed
wants to merge 1 commit into from
Closed

[dcl.attr.grammar] Clarify that only non-standard attributes can be ignored CWG2538 #5133

wants to merge 1 commit into from

Conversation

jensmaurer
Copy link
Member

No description provided.

@jensmaurer jensmaurer added the not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking. label Dec 2, 2021
@jensmaurer
Copy link
Member Author

jensmaurer commented Dec 2, 2021 •
edited

This might need a CWG issue; see http://lists.isocpp.org/ext/2021/12/18597.php

Put on CWG agenda: https://wiki.edg.com/bin/view/Wg21telecons2021/CoreWorkingGroup

@jensmaurer jensmaurer added cwg Issue must be reviewed by CWG. and removed not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking. labels Dec 14, 2021
@AaronBallman
Copy link
Contributor

I agree that I think this needs a core issue; it has C compatibility concerns and I (personally) don't consider this to be editorial. Thanks for adding it to the CWG agenda!

@jensmaurer jensmaurer changed the title [dcl.attr.grammar] Clarify that only non-standard attributes can be ignored [dcl.attr.grammar] Clarify that only non-standard attributes can be ignored CWG2538 Feb 24, 2022
@jensmaurer
Copy link
Member Author

jensmaurer commented Feb 24, 2022 •
edited

Created CWG2538.

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 19, 2022

@jensmaurer What should we do with issues for which you have CWG issues? Keep both open and the latter "fixes" the former?

@jensmaurer
Copy link
Member Author

Good question. I think it makes sense to close the editorial issue once we have a core issue; both are publicly visible. Sometimes, the editorial issue contains a bit more (ongoing) discussion than the core issue, though.

@tkoeppe
Copy link
Contributor

tkoeppe commented Aug 20, 2022

OK. If it's helpful, we can certainly keep these issues open and then close them all at once.

@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Jun 28, 2023
@frederick-vs-ja
Copy link
Contributor

Superseded by #5990?

@jensmaurer jensmaurer mentioned this pull request Jul 28, 2023
Merged
@jensmaurer jensmaurer closed this Jul 28, 2023
@jensmaurer jensmaurer deleted the c58 branch July 28, 2023 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cwg Issue must be reviewed by CWG. needs rebase The pull request needs a git rebase to resolve merge conflicts.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@jensmaurer @AaronBallman @tkoeppe @frederick-vs-ja @wg21bot