From 4c3a710a05e31504857f8bebe2ca8197460fd7dd Mon Sep 17 00:00:00 2001 From: Christopher Di Bella Date: Wed, 10 Jan 2024 19:28:25 +0000 Subject: [PATCH 1/2] [libcxx] renames local variable to avoid shadowing werror Due to the inclusion of a header, a global type is was being shadowed, which upset GCC. --- .../range.join/range.join.sentinel/ctor.other.pass.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libcxx/test/std/ranges/range.adaptors/range.join/range.join.sentinel/ctor.other.pass.cpp b/libcxx/test/std/ranges/range.adaptors/range.join/range.join.sentinel/ctor.other.pass.cpp index fb1e8eb1ebefa6..96bcf6b2c4c913 100644 --- a/libcxx/test/std/ranges/range.adaptors/range.join/range.join.sentinel/ctor.other.pass.cpp +++ b/libcxx/test/std/ranges/range.adaptors/range.join/range.join.sentinel/ctor.other.pass.cpp @@ -71,10 +71,10 @@ constexpr bool test() { BufferView, sentinel_wrapper>, bidirectional_iterator, sentinel_wrapper>>; using JoinView = std::ranges::join_view; - using sentinel = std::ranges::sentinel_t; + using sentinel_t = std::ranges::sentinel_t; using const_sentinel = std::ranges::sentinel_t; - static_assert(!std::constructible_from); - static_assert(!std::constructible_from); + static_assert(!std::constructible_from); + static_assert(!std::constructible_from); } return true; } From 4fd5e776772500297f3e609373d15bd726076b84 Mon Sep 17 00:00:00 2001 From: Louis Dionne Date: Wed, 10 Jan 2024 17:27:22 -0500 Subject: [PATCH 2/2] Apply suggestions from code review Co-authored-by: Tacet <4922191+AdvenamTacet@users.noreply.github.com> --- .../range.join/range.join.sentinel/ctor.other.pass.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libcxx/test/std/ranges/range.adaptors/range.join/range.join.sentinel/ctor.other.pass.cpp b/libcxx/test/std/ranges/range.adaptors/range.join/range.join.sentinel/ctor.other.pass.cpp index 96bcf6b2c4c913..8e78c3732e2080 100644 --- a/libcxx/test/std/ranges/range.adaptors/range.join/range.join.sentinel/ctor.other.pass.cpp +++ b/libcxx/test/std/ranges/range.adaptors/range.join/range.join.sentinel/ctor.other.pass.cpp @@ -72,9 +72,9 @@ constexpr bool test() { bidirectional_iterator, sentinel_wrapper>>; using JoinView = std::ranges::join_view; using sentinel_t = std::ranges::sentinel_t; - using const_sentinel = std::ranges::sentinel_t; - static_assert(!std::constructible_from); - static_assert(!std::constructible_from); + using const_sentinel_t = std::ranges::sentinel_t; + static_assert(!std::constructible_from); + static_assert(!std::constructible_from); } return true; }