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
P1754R1 Rename concepts to standard_case for C++20 #3099
Conversation
Fixed whitespace issues seen during review. Fixed additional inconsistencies in the Latex used for concept names. [concept.strictweakorder] Fix \indexlibrary for strict_weak_order. [ranges] Change ranges::view to ranges::views. Fixes #3015.
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.
Nice work! The only real problem I found was same_as
. I didn't comment on every one of them, but find&replace will find the others anyway.
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.
Just a few editorial corrections.
… naming scheme. * \placeholdernc{Advanceable} => \placeholdernc{advanceable} * \placeholdernc{same-impl} => \placeholdernc{same-as-impl} * \placeholdernc{Decrementable} => \placeholdernc{decrementable}
…tions for consistency. * concept.convertibleto => concept.convertible * concept.derivedfrom => concept.derived * concept.stricttotallyordered => concept.totallyordered
Thanks again for all the work on this, @burblebee, I really should have prepared this PR myself. |
@jwakely you marked this "changes requested" - I addressed all of your concerns afaik. Might you please have a quick look to make that go away? I don't like the red :) |
@zygoloid this motion is not complete - must be completed after all other motions are merged. |
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.
Requested changes look good, but there are conflicts to resolve now that other motions have been merged to master.
Rebasing this turned out to be more work than recreating the semi-mechanical renaming part, so I did that and kept all the other changes. This is superseded by #3149. |
No description provided.