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
[dcl.init.aggr]/1 Aggregate only cares about direct base classes CWG2610 #5634
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think indirect virtual base classes may be a problem (they have global constructor effects). Agreed with the private/protected changes.
This looks good to me now. @hubert-reinterpretcast , please have a final look. |
It's good enough. I have a mild preference for repeating "virtual" and reordering the bullets so that the two "no private or protected" bullets are grouped together. |
Except for virtual ones
c8af740
to
a0a2a3e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM; thanks!
@jensmaurer Does CWG consider this editorial? (Feel free to merge if so!) |
Fixes cplusplus/CWG#101
See also #2892