[libcxx-commits] [libcxx] [libc++][string] P3044R2: sub-`string_view` from `string` (PR #147095)
Nikolas Klauser via libcxx-commits
libcxx-commits at lists.llvm.org
Mon Sep 22 00:09:41 PDT 2025
================
@@ -0,0 +1,131 @@
+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+// REQUIRES: std-at-least-c++26
+
+// <string>
+
+// constexpr basic_string_view<charT, traits> subview(size_type pos = 0,
+// size_type n = npos) const;
+
+#include <cassert>
+#include <concepts>
+#include <string>
+#include <string_view>
+#include <utility>
+
+#include "constexpr_char_traits.h"
+#include "make_string.h"
+#include "min_allocator.h"
+#include "test_allocator.h"
+#include "test_macros.h"
+#include "type_algorithms.h"
+
+#define CS(S) MAKE_CSTRING(CharT, S)
+
+template <typename CharT, typename TraitsT, typename AllocT>
+constexpr void test() {
+ std::basic_string<CharT, TraitsT, AllocT> s{CS("Hello cruel world!"), AllocT{}};
+
+ { // With a default position and a character length.
+
+ // Also check if subview() is a const-qualified.
+ assert(std::as_const(s).subview() == CS("Hello cruel world!"));
+
+ // Check it the return type of subview() is correct.
+ std::same_as<std::basic_string_view<CharT, TraitsT>> decltype(auto) sv = s.subview();
+ assert(sv == CS("Hello cruel world!"));
+ }
+
+ { // Check with different position and length.
+
+ // With a explict position and a character length.
+ assert(s.subview(6, 5) == CS("cruel"));
+
+ // From the beginning of the string with a explicit character length.
+ assert(s.subview(0, 5) == CS("Hello"));
+
+ // To the end of string with the default character length.
+ assert(s.subview(12) == CS("world!"));
+
+ // From the beginning to the end of the string with explicit values.
+ assert(s.subview(0, s.size()) == CS("Hello cruel world!"));
+ }
+
+ // Test if exceptions are thrown correctly.
+#ifndef TEST_HAS_NO_EXCEPTIONS
+ if (!std::is_constant_evaluated()) {
+ { // With a position that is out of range.
+ try {
+ std::ignore = s.subview(s.size() + 1);
+ assert(false);
+ } catch ([[maybe_unused]] const std::out_of_range& ex) {
+ LIBCPP_ASSERT(std::string(ex.what()) == "string_view::substr");
+ } catch (...) {
+ assert(false);
+ }
+ }
+
+ { // With a position that is out of range and a 0 character length.
+ try {
+ std::ignore = s.subview(s.size() + 1, 0);
+ assert(false);
+ } catch ([[maybe_unused]] const std::out_of_range& ex) {
+ LIBCPP_ASSERT(std::string(ex.what()) == "string_view::substr");
+ } catch (...) {
+ assert(false);
+ }
+ }
+
+ { // With a position that is out of range and a some character length.
+ try {
+ std::ignore = s.subview(s.size() + 1, 1);
+ assert(false);
+ } catch ([[maybe_unused]] const std::out_of_range& ex) {
+ LIBCPP_ASSERT(std::string(ex.what()) == "string_view::substr");
+ } catch (...) {
+ assert(false);
+ }
+ }
+ }
+#endif
+}
+
+template <typename CharT>
+constexpr void test() {
+ test<CharT, std::char_traits<CharT>, std::allocator<CharT>>();
+ test<CharT, std::char_traits<CharT>, min_allocator<CharT>>();
+ test<CharT, std::char_traits<CharT>, safe_allocator<CharT>>();
+ test<CharT, std::char_traits<CharT>, test_allocator<CharT>>();
+
+ test<CharT, constexpr_char_traits<CharT>, std::allocator<CharT>>();
+ test<CharT, constexpr_char_traits<CharT>, min_allocator<CharT>>();
+ test<CharT, constexpr_char_traits<CharT>, safe_allocator<CharT>>();
+ test<CharT, constexpr_char_traits<CharT>, test_allocator<CharT>>();
+}
+
+class Test {
+public:
+ template <typename CharT>
+ constexpr void operator()() const {
+ test<CharT>();
+ }
+};
+
+constexpr bool test() {
+ types::for_each(types::character_types(), Test{});
+
+ return true;
+}
----------------
philnik777 wrote:
```suggestion
constexpr bool test() {
types::for_each(types::character_types(), []<class CharT> { test<CharT>(); });
return true;
}
```
Same in the other test.
https://github.com/llvm/llvm-project/pull/147095
More information about the libcxx-commits
mailing list