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

EnumType and DomainSize #226

Closed
wants to merge 10 commits into from
Closed

EnumType and DomainSize #226

wants to merge 10 commits into from

Conversation

RipplB
Copy link
Contributor

@RipplB RipplB commented Oct 1, 2023

Fixes #122
And also introduces DomainSize: every type should now be able to tell the number of possible values it can have

@RipplB RipplB marked this pull request as ready for review October 1, 2023 13:20
@RipplB
Copy link
Contributor Author

RipplB commented Oct 1, 2023

@mondokm @szdan97

@szdan97 szdan97 requested a review from as3810t October 5, 2023 11:50
Copy link
Contributor

@as3810t as3810t left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes. I think this should integrate well with the existing code base.

@as3810t as3810t dismissed their stale review October 9, 2023 08:48

Reviewed again.

@RipplB RipplB closed this Jul 5, 2024
@RipplB
Copy link
Contributor Author

RipplB commented Jul 5, 2024

Development is continued in #272

@RipplB RipplB deleted the enumtype branch July 13, 2024 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Name collision checks for XSTS, enum type rework
3 participants