TeamHeptaMirrors/gcc
1
0
Fork 1
mirror of https://github.com/gcc-mirror/gcc.git synced 2026-05-06 23:14:49 +02:00
Code Issues Packages Projects Releases Wiki Activity

libstdc++: Use ranges::iter_move in ranges::unique [PR120789]

Browse Source 
PR libstdc++/120789

libstdc++-v3/ChangeLog:

	* include/bits/ranges_algo.h (__unique_fn::operator()): Use
	ranges::iter_move(iter) instead of std::move(*iter).
	* testsuite/25_algorithms/unique/120789.cc: New test.

Reviewed-by: Tomasz Kamiński <tkaminsk@redhat.com>
Reviewed-by: Jonathan Wakely <jwakely@redhat.com>
This commit is contained in:
Patrick Palka
2025-07-01 13:43:09 -04:00
parent f5282f7402
commit 6f69a68998
2 changed files with 37 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
libstdc++-v3/include/bits/ranges_algo.h
View File

@@ -1454,7 +1454,7 @@ namespace ranges
if (!std::__invoke(__comp,
std::__invoke(__proj, *__dest),
std::__invoke(__proj, *__first)))
*++__dest = std::move(*__first);
*++__dest = ranges::iter_move(__first);
return {++__dest, __first};
}

36
libstdc++-v3/testsuite/25_algorithms/unique/120789.cc Normal file
View File

@@ -0,0 +1,36 @@
// PR libstdc++/120789 - ranges::unique should use ranges::iter_move
// { dg-do compile { target c++20 } }
#include <algorithm>
struct A
{
bool operator==(const A&) const;
};
struct B
{
B(B&&) = delete;
B& operator=(const A&) const;
operator A() const;
bool operator==(const B&) const;
};
struct I
{
using value_type = A;
using difference_type = int;
B operator*() const;
I& operator++();
I operator++(int);
bool operator==(const I&) const;
friend A iter_move(const I&);
};
void
test01()
{
std::ranges::subrange<I, I> r;
auto [begin, end] = std::ranges::unique(r);
}