-
Notifications
You must be signed in to change notification settings - Fork 770
P2976R1 Freestanding Library: algorithm, numeric, and random #7706
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
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'm not an editor, but here's my review of the paper vs. the latex.
void uninitialized_default_construct(ExecutionPolicy&& exec, // freestanding-deleted, | ||
NoThrowForwardIterator first, // see \ref{algorithms.parallel.overloads} |
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 don't think this comment is aligned with the other // freestandings
. Same for others in this file.
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.
This seems fine, do you have anything specific in mind?
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.
No objections to the status quo.
7beb045
to
bdced58
Compare
bdced58
to
9f05d3f
Compare
@@ -1601,10 +1602,12 @@ | |||
\ref{iterators} & Iterators library & \tcode{<iterator>} \\ \rowsep | |||
\ref{ranges} & Ranges library & \tcode{<ranges>} \\ \rowsep | |||
\ref{algorithms} & Algorithms library & \tcode{<algorithm>}, \tcode{<numeric>} \\ \rowsep | |||
\ref{execpol} & Execution policies & \tcode{<execpol>} \\ \rowsep |
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.
Oops, the header should be <execution>
. I'll open a PR for this.
Fixes #7669.
Fixes cplusplus/papers#1644.