Split range adaptors

[bartholomaios]

beman::str_split::split_when_view is a range adaptor based on std::ranges::split_view that splits a view on delimiters found by the provided search function. The search function takes the remaining sub-range to search as input, and returns the nested sub-range around the next delimiter.

// Custom delimiter search operation.
class any_seq_of {
public:
    constexpr explicit any_seq_of(std::string_view delimiters)
        : delimiters_(delimiters)
    {
    }

    template<class V>
        requires std::constructible_from<std::string_view, V>
    V operator()(V view)
    {
        auto str = std::string_view(view);
        auto begin_pos = str.find_first_of(delimiters_);
        if (begin_pos == str.npos) {
            return {std::ranges::end(view), std::ranges::end(view)};
        }
        auto end_pos = str.find_first_not_of(delimiters_, begin_pos + 1);
        return {std::ranges::begin(view) + begin_pos, std::ranges::begin(view) + end_pos};
    }

private:
    std::string_view delimiters_;
};

std::vector<std::string> make_words() {
    return "Alice \t was beginning to\tget \tvery    tired"sv
            | bss::views::split_when(any_seq_of(" \t"sv))
            | std::ranges::to<std::vector<std::string>>();
}

I've also implemented std::range::split_view as beman::str_split::split_view. I haven't written range adaptors before, so this was a good learning exercise.

beman::str_split::split_when_view still has all the same ergonomic issues that std::ranges::split_view has W.R.T working with strings. Nonetheless, I still think this is a useful starting point for range-based splitting.

I've not made an attempt to use modules yet. Everything is header based. I do want to support modules, I just haven't take the time to learn how to use them.

[github-actions]

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

pre-commit

---

[pre-commit] _{reported by reviewdog 🐶}

str_split/tests/beman/str_split/split_when.tests.cpp

Lines 58 to 60 in ba5f322

	auto words = "Alice \t was beginning to\tget \tvery tired"sv
	| bss::views::split_when(any_seq_of(" \t"sv))
	| std::ranges::to<std::vector<std::string>>();

[github-actions]

[pre-commit] _{reported by reviewdog 🐶}

Suggested change

	auto name = "Alice's"sv;
	auto name = "Alice's"sv;

[github-actions]

[pre-commit] _{reported by reviewdog 🐶}

Suggested change

	auto parts = "A"sv
	| bss::views::split(""sv)
	| std::ranges::to<std::vector<std::string>>();
	auto parts = "A"sv | bss::views::split(""sv) | std::ranges::to<std::vector<std::string>>();

[github-actions]

[pre-commit] _{reported by reviewdog 🐶}

Suggested change

	auto parts = "Alice"sv
	| bss::views::split(""sv)
	| std::ranges::to<std::vector<std::string>>();
	auto parts = "Alice"sv | bss::views::split(""sv) | std::ranges::to<std::vector<std::string>>();

[github-actions]

[pre-commit] _{reported by reviewdog 🐶}

Suggested change

	auto parts = "Down the Rabbit-Hole."sv
	| bss::views::split("the"sv)
	| std::ranges::to<std::vector<std::string>>();
	auto parts = "Down the Rabbit-Hole."sv | bss::views::split("the"sv) | std::ranges::to<std::vector<std::string>>();

[github-actions]

[pre-commit] _{reported by reviewdog 🐶}

Suggested change

	auto parts = "Alice was beginning to get very tired"sv
	| bss::views::split(" t"sv)
	| std::ranges::to<std::vector<std::string>>();
	auto parts = "Alice was beginning to get very tired"sv | bss::views::split(" t"sv) |
	std::ranges::to<std::vector<std::string>>();

[github-actions]

[pre-commit] _{reported by reviewdog 🐶}

Suggested change

	V operator()(V view)
	{
	auto str = std::string_view(view);
	V operator()(V view) {
	auto str = std::string_view(view);

[github-actions]

[pre-commit] _{reported by reviewdog 🐶}

Suggested change

	private:
	private:

[github-actions]

[pre-commit] _{reported by reviewdog 🐶}

Suggested change

	}
	} // namespace

[github-actions]

[pre-commit] _{reported by reviewdog 🐶}

Suggested change

	auto words = ""sv
	| bss::views::split_when(any_seq_of(" \t"sv))
	| std::ranges::to<std::vector<std::string>>();
	auto words = ""sv | bss::views::split_when(any_seq_of(" \t"sv)) | std::ranges::to<std::vector<std::string>>();

[github-actions]

[pre-commit] _{reported by reviewdog 🐶}

Suggested change

	auto parts = "Down the Rabbit-Hole"sv
	| bss::views::split_when(any_seq_of("-."sv))
	| std::ranges::to<std::vector<std::string>>();
	auto parts = "Down the Rabbit-Hole"sv | bss::views::split_when(any_seq_of("-."sv)) |
	std::ranges::to<std::vector<std::string>>();

[coveralls]

Coverage Report for CI Build 27506021983

Warning

No base build found for commit f097c5d on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 96.226%

Details

• Patch coverage: 2 uncovered changes across 1 file (51 of 53 lines covered, 96.23%).

Uncovered Changes

File	Changed	Covered	%
include/beman/str_split/detail/split_iterator.hpp	15	13	86.67%
Total (4 files)	53	51	96.23%

Coverage Regressions

Requires a base build to compare against. How to fix this →

---

Coverage Stats

Relevant Lines:	53
Covered Lines:	51
Line Coverage:	96.23%
Coverage Strength:	22.21 hits per line

---

💛 - Coveralls

[github-actions]

[pre-commit] _{reported by reviewdog 🐶}

Suggested change

	auto words = "Alice \t was beginning to\tget \tvery tired"sv
	| bss::views::split_when(any_seq_of(" \t"sv))
	| std::ranges::to<std::vector<std::string>>();
	auto words = "Alice \t was beginning to\tget \tvery tired"sv | bss::views::split_when(any_seq_of(" \t"sv)) |
	std::ranges::to<std::vector<std::string>>();