Skip to content

Replace CRTP in builders with deducing this.#102

Merged
crud89 merged 21 commits intomainfrom
deducing-this
Nov 3, 2023
Merged

Replace CRTP in builders with deducing this.#102
crud89 merged 21 commits intomainfrom
deducing-this

Conversation

@crud89
Copy link
Copy Markdown
Owner

@crud89 crud89 commented Oct 25, 2023

Describe the pull request

This PR replaces many occurrences of the CRTP, especially in builders with the new C++23 feature deducing this. This simplifies builder architecture. Additionally, many builders now allow for compile-time evaluation, where possible, improving performance through optimization.

@crud89 crud89 added Priority: Medium A issue with normal priority. Type: Requirement labels Oct 25, 2023
@crud89 crud89 added this to the Alpha #04 milestone Oct 25, 2023
@crud89 crud89 self-assigned this Oct 25, 2023
@crud89 crud89 marked this pull request as ready for review November 3, 2023 19:20
@crud89 crud89 added the Build 🛠 Issues that involve the build process. label Nov 3, 2023
@crud89 crud89 merged commit 449c9cc into main Nov 3, 2023
@crud89 crud89 deleted the deducing-this branch November 3, 2023 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build 🛠 Issues that involve the build process. Priority: Medium A issue with normal priority.

Projects

Status: v0.4.1

Development

Successfully merging this pull request may close these issues.

1 participant