Skip to content

dependent names#4793

Open
jensmaurer wants to merge 3 commits into
cplusplus:mainfrom
jensmaurer:c8
Open

dependent names#4793
jensmaurer wants to merge 3 commits into
cplusplus:mainfrom
jensmaurer:c8

Conversation

@jensmaurer

@jensmaurer jensmaurer commented Aug 4, 2021

Copy link
Copy Markdown
Member

Fixes #4791

@jensmaurer

jensmaurer commented Aug 4, 2021

Copy link
Copy Markdown
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
Comment thread source/templates.tex Outdated
xmh0511
xmh0511 reviewed Aug 5, 2021
View reviewed changes
Comment thread source/templates.tex Outdated
@jensmaurer

jensmaurer commented Aug 5, 2021 •
edited
Loading

Copy link
Copy Markdown
Member Author

Bad attempt (see below for a fix):

disambig

@xmh0511

xmh0511 commented Aug 5, 2021 •
edited
Loading

Copy link
Copy Markdown
Contributor

@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

jensmaurer commented Aug 5, 2021 •
edited
Loading

Copy link
Copy Markdown
Member Author

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

jensmaurer commented Sep 17, 2021

Copy link
Copy Markdown
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

1 more reviewer
@xmh0511 xmh0511 xmh0511 left review comments
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

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