[committed] libstdc++: Implement proposed resolution for LWG 3532

Message ID 20210430150717.943210-1-ppalka@redhat.com
State New
Headers show
Series
  • [committed] libstdc++: Implement proposed resolution for LWG 3532
Related show

Commit Message

H.J. Lu via Gcc-patches April 30, 2021, 3:07 p.m.
Tested on x86_64-pc-linux-gnu, committed as "obvious".

libstdc++-v3/ChangeLog:

	* include/std/ranges (split_view::_InnerIter::operator++):
	Depend on _Base instead of _Vp directly, as per LWG 3532.
---
 libstdc++-v3/include/std/ranges | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.31.1.362.g311531c9de

Comments

H.J. Lu via Gcc-patches April 30, 2021, 5 p.m. | #1
On 30/04/21 11:07 -0400, Patrick Palka via Libstdc++ wrote:
>Tested on x86_64-pc-linux-gnu, committed as "obvious".


Thanks. OK for 11 and 10 too.

>libstdc++-v3/ChangeLog:

>

>	* include/std/ranges (split_view::_InnerIter::operator++):

>	Depend on _Base instead of _Vp directly, as per LWG 3532.

>---

> libstdc++-v3/include/std/ranges | 2 +-

> 1 file changed, 1 insertion(+), 1 deletion(-)

>

>diff --git a/libstdc++-v3/include/std/ranges b/libstdc++-v3/include/std/ranges

>index 73d3e008a46..4975d5c630b 100644

>--- a/libstdc++-v3/include/std/ranges

>+++ b/libstdc++-v3/include/std/ranges

>@@ -2898,7 +2898,7 @@ namespace views::__adaptor

> 	  constexpr decltype(auto)

> 	  operator++(int)

> 	  {

>-	    if constexpr (forward_range<_Vp>)

>+	    if constexpr (forward_range<_Base>)

> 	      {

> 		auto __tmp = *this;

> 		++*this;

>-- 

>2.31.1.362.g311531c9de

>

Patch

diff --git a/libstdc++-v3/include/std/ranges b/libstdc++-v3/include/std/ranges
index 73d3e008a46..4975d5c630b 100644
--- a/libstdc++-v3/include/std/ranges
+++ b/libstdc++-v3/include/std/ranges
@@ -2898,7 +2898,7 @@  namespace views::__adaptor
 	  constexpr decltype(auto)
 	  operator++(int)
 	  {
-	    if constexpr (forward_range<_Vp>)
+	    if constexpr (forward_range<_Base>)
 	      {
 		auto __tmp = *this;
 		++*this;