Thank you!
You are right. The idea was that the test also tests the const overloads of
begin() and end() of reverse_view. But the view concept requires movable.
Maybe, this should just be
static_assert(std::ranges::range);
instead?
The test in the patch now has the same static_assert twice.
Quick reminder: Can you (or anyone) please commit this? The updated patch is
attached in the last email.
Best,
Moritz
Am 03.03.21 um 20:26 schrieb Moritz Sichert:
Thanks for the review. I attached the updated patch.
Can you commit this for me or point me to what I should do next? This is my
Thanks for the review. I attached the updated patch.
Can you commit this for me or point me to what I should do next? This is my
first contribution here.
Best,
Moritz
Am 03.03.21 um 19:02 schrieb Patrick Palka:
On Wed, 3 Mar 2021, Moritz Sichert via Libstdc++ wrote:
std::ranges::reverse_vie
std::ranges::reverse_view uses make_reverse_iterator in its
implementation as described in [range.reverse.view]. This accidentally
allows ADL as an unqualified name is used in the call. According to
[contents], however, this should be treated as a qualified lookup into
the std namespace.
This lea
