[libcxx-commits] [libcxx] 792fce7 - [libc++][ranges][NFC] Format a couple of tests as a prerequisite (#190514)
via libcxx-commits
libcxx-commits at lists.llvm.org
Sun Apr 5 05:21:04 PDT 2026
Author: Hristo Hristov
Date: 2026-04-05T15:20:58+03:00
New Revision: 792fce7cc3d68dc4b304f70df09995839a4b4319
URL: https://github.com/llvm/llvm-project/commit/792fce7cc3d68dc4b304f70df09995839a4b4319
DIFF: https://github.com/llvm/llvm-project/commit/792fce7cc3d68dc4b304f70df09995839a4b4319.diff
LOG: [libc++][ranges][NFC] Format a couple of tests as a prerequisite (#190514)
to https://llvm.org/PR190513 to avoid friction.
Added:
Modified:
libcxx/test/std/ranges/range.adaptors/range.filter/ctor.view_pred.pass.cpp
libcxx/test/std/ranges/range.adaptors/range.lazy.split/ctor.view.pass.cpp
Removed:
################################################################################
diff --git a/libcxx/test/std/ranges/range.adaptors/range.filter/ctor.view_pred.pass.cpp b/libcxx/test/std/ranges/range.adaptors/range.filter/ctor.view_pred.pass.cpp
index 3ccab93397147..8eaf6eba35709 100644
--- a/libcxx/test/std/ranges/range.adaptors/range.filter/ctor.view_pred.pass.cpp
+++ b/libcxx/test/std/ranges/range.adaptors/range.filter/ctor.view_pred.pass.cpp
@@ -19,7 +19,7 @@
#include "types.h"
struct Range : std::ranges::view_base {
- constexpr explicit Range(int* b, int* e) : begin_(b), end_(e) { }
+ constexpr explicit Range(int* b, int* e) : begin_(b), end_(e) {}
constexpr int* begin() const { return begin_; }
constexpr int* end() const { return end_; }
@@ -52,7 +52,7 @@ static_assert(!test_convertible<std::ranges::filter_view<Range, Pred>, Range, Pr
#else
-static_assert( test_convertible<std::ranges::filter_view<Range, Pred>, Range, Pred>(),
+static_assert(test_convertible<std::ranges::filter_view<Range, Pred>, Range, Pred>(),
"This constructor must not be explicit");
#endif // TEST_STD_VER >= 23
diff --git a/libcxx/test/std/ranges/range.adaptors/range.lazy.split/ctor.view.pass.cpp b/libcxx/test/std/ranges/range.adaptors/range.lazy.split/ctor.view.pass.cpp
index e7bf052a7e9ee..46af5498d59a0 100644
--- a/libcxx/test/std/ranges/range.adaptors/range.lazy.split/ctor.view.pass.cpp
+++ b/libcxx/test/std/ranges/range.adaptors/range.lazy.split/ctor.view.pass.cpp
@@ -20,18 +20,15 @@
struct ViewWithCounting : std::ranges::view_base {
int* times_copied = nullptr;
- int* times_moved = nullptr;
+ int* times_moved = nullptr;
constexpr ViewWithCounting(int& copies_ctr, int& moves_ctr) : times_copied(&copies_ctr), times_moved(&moves_ctr) {}
constexpr ViewWithCounting(const ViewWithCounting& rhs)
- : times_copied(rhs.times_copied)
- , times_moved(rhs.times_moved) {
+ : times_copied(rhs.times_copied), times_moved(rhs.times_moved) {
++(*times_copied);
}
- constexpr ViewWithCounting(ViewWithCounting&& rhs)
- : times_copied(rhs.times_copied)
- , times_moved(rhs.times_moved) {
+ constexpr ViewWithCounting(ViewWithCounting&& rhs) : times_copied(rhs.times_copied), times_moved(rhs.times_moved) {
++(*times_moved);
}
@@ -39,14 +36,14 @@ struct ViewWithCounting : std::ranges::view_base {
constexpr const char* end() const { return nullptr; }
constexpr ViewWithCounting& operator=(const ViewWithCounting&) = default;
- constexpr ViewWithCounting& operator=(ViewWithCounting&&) = default;
+ constexpr ViewWithCounting& operator=(ViewWithCounting&&) = default;
constexpr bool operator==(const ViewWithCounting&) const { return true; }
};
static_assert(std::ranges::forward_range<ViewWithCounting>);
static_assert(std::ranges::view<ViewWithCounting>);
-using View = ViewWithCounting;
+using View = ViewWithCounting;
using Pattern = ViewWithCounting;
// SFINAE tests.
@@ -58,7 +55,7 @@ static_assert(!test_convertible<std::ranges::lazy_split_view<View, Pattern>, Vie
#else
-static_assert( test_convertible<std::ranges::lazy_split_view<View, Pattern>, View, Pattern>(),
+static_assert(test_convertible<std::ranges::lazy_split_view<View, Pattern>, View, Pattern>(),
"This constructor must not be explicit");
#endif // TEST_STD_VER >= 23
More information about the libcxx-commits
mailing list