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

[class.abstract] The intro para seems like it should be a note - noth… #724

Merged
merged 1 commit into from May 30, 2016
Merged

[class.abstract] The intro para seems like it should be a note - noth… #724

merged 1 commit into from May 30, 2016

Conversation

faisalv
Copy link
Contributor

@faisalv faisalv commented May 16, 2016

…ing seems prescriptive about it.

@tkoeppe
Copy link
Contributor

tkoeppe commented May 16, 2016

@jwakely, @zygoloid: Please have a look.

@jwakely
Copy link
Member

jwakely commented May 16, 2016

LGTM but I'll leave @zygoloid to decide.

@zygoloid zygoloid merged commit e7ed06f into cplusplus:master May 30, 2016
@faisalv faisalv deleted the fv-abstract-class-intro branch May 31, 2016 00:53
FrankHB pushed a commit to FrankHB/draft that referenced this pull request Jul 9, 2016
@faisalv @FrankHB
[class.abstract] The intro para seems like it should be a note - noth…
52024b4 
…ing seems prescriptive about it. (cplusplus#724)
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@faisalv @tkoeppe @jwakely @zygoloid