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
P2502R2 std::generator: Synchronous Coroutine Generator for Ranges #5695
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.
After applying the suggested changes, replace all Rng
with R
and all rng
with r
.
The only uses of rng
in the standard come from InnerRng
and their related InnerConstRng
and inner_rng
, and [rand.req.urng].
R
is more common for ranges::range
and its refinements.
source/ranges.tex
Outdated
\pnum | ||
\returns | ||
An awaitable object of an unspecified type\iref{expr.await} | ||
which takes ownership of the generator \tcode{g.range}, |
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
which takes ownership of the generator \tcode{g.range}, | |
into which \tcode{g.range} is moved, |
communicates the same point, perhaps even more clearly?
f4af5b0
to
e75e167
Compare
abfcdf3
to
600b95a
Compare
37e68d6
to
52456b9
Compare
It is more customary to refer to ranges as just "R", not "Rng".
692a231
to
3dee233
Compare
Fixes #5619
Fixes cplusplus/papers#1151