isocpp / isocpp/CppCoreGuidelines
P.3 needs to be updated now the Ranges TS is part of the standard
Open
@hsutter is already working on this.
Since Jan 27, 2022.
- Dominant language
- CSS
- Stars
- 45.3k
- Forks
- 5.6k
- PR merge metrics
- No merged PRs in 30d
Description
I propose that the second example is removed:
for_each(par, v, [](int x) { /* do something with the value of x */ });
as passing an ExecutionPolicy is not possible in the final version
To avoid confusion with (std::)for_each, I would also suggest that ranges::for_each is used.
Related: #1254
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.