-
Notifications
You must be signed in to change notification settings - Fork 777
LWG parts of P2996R13 Reflection for C++26 #8004
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
base: main
Are you sure you want to change the base?
Conversation
Forward-looking comment for the library part: The section labels for the library should be "meta.refl.*" (because shorter without loss of expression) and for the Also, "define_aggregate" could maybe become "def.aggr". |
b89923b
to
e87c239
Compare
%FIXME: I see the same in <algorithm> with a // see \ref{initializer.list.syn} comment | ||
#include <initializer_list> |
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 we should add // see \ref{initializer.list.syn}
for consistency. Currently, such comment is added whenever <initializer_list>
is included in the synopsis.
b0fb1eb
to
8b92b03
Compare
8b92b03
to
a13cfe0
Compare
@Eisenwave Do you already want review feedback, or is this still in "draft"? |
Thanks! |
0579faf
to
5c723b4
Compare
Hmm, it has occurred to me that this is a pretty big change. Everybody expected it and no one is surprised, but when you're |
a396eed
to
8846eea
Compare
8846eea
to
ec34c7e
Compare
I was gonna point it out if you didn't.. |
Closes #7930 once the core part is also complete.
Also closes cplusplus/papers#1668.