Skip to content
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.

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

Make GetSemantic(i) return a constexpr #266

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

robUx4
Copy link
Contributor

@robUx4 robUx4 commented Feb 24, 2024

This might be useful to do some static checks in the code.

Draft on top of #265 and #264

@robUx4 robUx4 added api-break breaks the API (e.g. programs using it will have to adjust their source code) abi-break breaks the ABI (e.g. programs linked against the library have to be recompiled) labels Feb 24, 2024
@robUx4
Copy link
Contributor Author

robUx4 commented Feb 24, 2024

Ultimately we should probably use a proper container, but only std::array seems usable in C++17.

@robUx4
Copy link
Contributor Author

robUx4 commented Mar 3, 2024

Rebased on top of #269

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
abi-break breaks the ABI (e.g. programs linked against the library have to be recompiled) api-break breaks the API (e.g. programs using it will have to adjust their source code)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant