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

[basic.scope.class] CWG 2331: is redundant #1169

Closed
tkoeppe opened this issue Dec 3, 2016 · 5 comments
Closed

[basic.scope.class] CWG 2331: is redundant #1169

tkoeppe opened this issue Dec 3, 2016 · 5 comments
Assignees
@jensmaurer
Labels
cwg Issue must be reviewed by CWG. not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking.

Comments

@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 3, 2016

The first four paragraphs of [basic.scope.class] (3.3.7) are somewhat reduntant. As @zygoloid noted:

  • The normative paragraphs are 4 and 2.
  • Paragraph 1 is subsumed by p4.
  • Paragraph 3 follows from existing rules for name hiding.

We felt that this was too much to reword editorially though and would like CWG guidance. We could produce a pull request and let CWG decide whether the change is editorial, and if not the proposed wording could be moved into a Core issue.
@tkoeppe tkoeppe added the cwg Issue must be reviewed by CWG. label Dec 3, 2016
@jensmaurer
Copy link
Member

Did someone e-mail Mike Miller already?

@tkoeppe
Copy link
Contributor Author

tkoeppe commented Dec 6, 2016

@jensmaurer: Not yet. Could you perhaps? Will you be in the upcoming teleconference?

@jensmaurer
Copy link
Member

Sent e-mail to Mike Miller.

Yes, I will join the teleconference next Monday.

@jensmaurer jensmaurer self-assigned this Dec 20, 2016
@jensmaurer jensmaurer changed the title [basic.scope.class] is redundant [basic.scope.class] CWG 2331: is redundant Apr 13, 2018
@jensmaurer jensmaurer added the not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking. label Apr 13, 2018
@jensmaurer
Copy link
Member

jensmaurer commented Jul 21, 2018 •
edited

see CWG 2331

@zygoloid
Copy link
Member

zygoloid commented Nov 7, 2018

Closing now we are tracking this as a core issue.

@zygoloid zygoloid closed this as completed Nov 7, 2018
@zygoloid zygoloid mentioned this issue Mar 8, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jensmaurer jensmaurer

Labels
cwg Issue must be reviewed by CWG. not-editorial Issue is not deemed editorial; the editorial issue is kept open for tracking.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@zygoloid @tkoeppe @jensmaurer