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.

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

[basic.scope.class] Break up list #1137

Merged
merged 1 commit into from Nov 28, 2016
Merged

[basic.scope.class] Break up list #1137

merged 1 commit into from Nov 28, 2016

Conversation

tkoeppe
Copy link
Contributor

@tkoeppe tkoeppe commented Nov 28, 2016

No description provided.

@tkoeppe tkoeppe changed the title [basic.scope.class] Use unordered list and move example out of the last list item [basic.scope.class] Break up list Nov 28, 2016
@zygoloid zygoloid merged commit 0e26279 into cplusplus:master Nov 28, 2016
@tkoeppe tkoeppe deleted the scope branch November 28, 2016 17:59
@languagelawyer
Copy link
Contributor

languagelawyer commented Oct 19, 2019

The removal of

The following rules describe the scope of names declared in classes.

seems to change the meaning of the third and the fourth paragraphs: now they talk about all names whose scope covers a class or past the end of a class definition (e.g. a global variable declared before the class definition), not only names declared in classes.
This looks like a normative change which was introduced editorially.

@tkoeppe
Copy link
Contributor Author

tkoeppe commented Oct 20, 2019

Thanks, that's a good point. Perhaps we should insert "The potential scope of a declaration in a class that extends ..." to p4? @zygoloid?

@zygoloid
Copy link
Member

Yes, I agree.

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.

None yet

3 participants