Skip to content
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

LWG Poll 1: P2315R0 LWG Tentatively Ready issues #4523

Merged
merged 8 commits into from Mar 8, 2021

Conversation

burblebee
Copy link
Contributor

@burblebee burblebee commented Feb 25, 2021

Fixes #4506.
Fixes cplusplus/papers#999.

Notes:
*LWG3494: not applied (resolved by P2017R1 Conditionally borrowed ranges (Barry Revzin) cplusplus/papers#752).

source/ranges.tex Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
source/ranges.tex Outdated Show resolved Hide resolved
template<bool Const>
class elements_view<V, N>::@\exposid{iterator}@ { // \expos
using @\exposid{Base}@ = @\exposid{maybe-const}@<Const, V>; // \expos

iterator_t<@\exposid{Base}@> @\exposid{current_}@ = iterator_t<@\exposid{Base}@>();
iterator_t<@\exposid{Base}@> @\exposid{current_}@ = iterator_t<@\exposid{Base}@>(); // \expos
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

4e22857 should've been a "fixup:" commit, I think.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops, yes, it should have been. @tkoeppe can extract it when he merges if he cares enough (we used to be a lot more lax about fly-by fixes like this)

@jensmaurer jensmaurer added this to the post-2021-02 milestone Feb 25, 2021
mkurdej added a commit to llvm/llvm-project that referenced this pull request Mar 1, 2021
… papers) adopted in November 2020 and February 2021 virtual meetings.

Sources:
* cplusplus/draft#4380 (November 2020)
* cplusplus/draft#4523 (February 2021)
* https://github.com/cplusplus/draft/issues?q=is%3Aissue+LWG+Motion
source/ranges.tex Outdated Show resolved Hide resolved
morehouse pushed a commit to morehouse/llvm-project that referenced this pull request Mar 4, 2021
… papers) adopted in November 2020 and February 2021 virtual meetings.

Sources:
* cplusplus/draft#4380 (November 2020)
* cplusplus/draft#4523 (February 2021)
* https://github.com/cplusplus/draft/issues?q=is%3Aissue+LWG+Motion
@tkoeppe tkoeppe merged commit 2532f16 into master Mar 8, 2021
@jensmaurer jensmaurer deleted the motions-2021-02-lwg-1 branch June 14, 2021 21:27
CharlesHe023 pushed a commit to CharlesHe023/libcxx that referenced this pull request Aug 23, 2021
… papers) adopted in November 2020 and February 2021 virtual meetings.

Sources:
* cplusplus/draft#4380 (November 2020)
* cplusplus/draft#4523 (February 2021)
* https://github.com/cplusplus/draft/issues?q=is%3Aissue+LWG+Motion
GitOrigin-RevId: 11ef785cdd0299a719632ef8fbc34f54f3db0674
arichardson pushed a commit to CTSRD-CHERI/libcxx that referenced this pull request Sep 24, 2021
… papers) adopted in November 2020 and February 2021 virtual meetings.

Sources:
* cplusplus/draft#4380 (November 2020)
* cplusplus/draft#4523 (February 2021)
* https://github.com/cplusplus/draft/issues?q=is%3Aissue+LWG+Motion
conr2d pushed a commit to haderech/libcxx that referenced this pull request Oct 23, 2021
… papers) adopted in November 2020 and February 2021 virtual meetings.

Sources:
* cplusplus/draft#4380 (November 2020)
* cplusplus/draft#4523 (February 2021)
* https://github.com/cplusplus/draft/issues?q=is%3Aissue+LWG+Motion
mem-frob pushed a commit to draperlaboratory/hope-llvm-project that referenced this pull request Oct 7, 2022
… papers) adopted in November 2020 and February 2021 virtual meetings.

Sources:
* cplusplus/draft#4380 (November 2020)
* cplusplus/draft#4523 (February 2021)
* https://github.com/cplusplus/draft/issues?q=is%3Aissue+LWG+Motion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants