Skip to content

dependent names #4793

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
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

dependent names #4793

wants to merge 3 commits into from

Conversation

jensmaurer
Copy link
Member

Fixes #4791

@jensmaurer
Copy link
Member Author

http://lists.isocpp.org/core/2021/08/11297.php

@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Aug 4, 2021
xmh0511
xmh0511 reviewed Aug 5, 2021
View reviewed changes
xmh0511
xmh0511 reviewed Aug 5, 2021
View reviewed changes
@jensmaurer
Copy link
Member Author

jensmaurer commented Aug 5, 2021 •
edited
Loading

Bad attempt (see below for a fix):

disambig

@xmh0511
Copy link
Contributor

xmh0511 commented Aug 5, 2021 •
edited
Loading

@jensmaurer For the second form, it may be nested-name-specifier templateopt simple-template-id. The first form may be nested-name-specifier identifier(nested-name-specifier always ends with ::). Except that, I think it is a big improvement for [temp.res.general-5]. Thanks.

@jensmaurer
Copy link
Member Author

jensmaurer commented Aug 5, 2021 •
edited
Loading

Right, fixed.

disambig

@xmh0511 xmh0511 mentioned this pull request Aug 31, 2021
Open
@jensmaurer jensmaurer added the cwg Issue must be reviewed by CWG. label Aug 31, 2021
@jensmaurer
Copy link
Member Author

Discussed by CWG https://wiki.edg.com/bin/view/Wg21telecons2021/IssuesProcessingTeleconference2021-08-24.
Needs an update using a new grammar term.

@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Sep 17, 2021
@wg21bot wg21bot added the needs rebase The pull request needs a git rebase to resolve merge conflicts. label Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xmh0511 xmh0511 xmh0511 left review comments

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.

[temp.res.general] p5: dependent qualified-id and using-declarator
3 participants
@jensmaurer @xmh0511 @wg21bot