Skip to content

refactor: [WIP] convert slices from CRTP to C++20 concepts #2866

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
sbstndb wants to merge 7 commits into
base: master
Choose a base branch
from
Open

refactor: [WIP] convert slices from CRTP to C++20 concepts #2866

sbstndb wants to merge 7 commits into from

Conversation

@sbstndb
Copy link
Contributor

@sbstndb sbstndb commented Oct 21, 2025

Description

Convert slices from CRTP to C++20 concepts

Checklist

  • The title and commit message(s) are descriptive.
  • Small commits made to fix your PR have been squashed to avoid history pollution.
  • Tests have been added for new features or bug fixes.
  • API of new functions and classes are documented.
@sbstndb
refactor: convert slices from CRTP to C++20 concepts
d3d4245 
Use if constexpr with auto return types for slice helper functions.
@sbstndb
Copy link
Contributor Author

sbstndb commented Oct 21, 2025

It seems that some conversions are still missing. This PR needs a little more work before it can be reviewed and merged.
@sbstndb
refactor: use xtl::integral_concept in slices
73e4823 
Replace xtl::is_integral<T>::value trait with xtl::integral_concept for consistency with existing C++20 concepts.
@sbstndb
style: apply clang-format to slices
ab17eef 
Add braces to if constexpr blocks for consistency.
@sbstndb sbstndb changed the title refactor: convert slices from CRTP to C++20 concepts refactor: [WIP] convert slices from CRTP to C++20 concepts Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@sbstndb