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

fix(#102): Fixed shift error on >64 bit enums #103

Merged
merged 2 commits into from
Feb 16, 2023

Conversation

mlgiraud
Copy link
Contributor

Large enums realistically will never be naturally exhaustive due to their size. The shift is now done as a checked shift and when the shift fails it will default to not naturally exhaustive, requiring the default attribute, even if it were exhaustive. (I want to meet the person who manages to write down 2^64 enum values or more)

Large enums realistically will never be naturally exhaustive due to their size. The shift is now done as a checked shift and when the shift fails it will default to not naturally exhaustive, requiring the default attribute, even if it were exhaustive. (I want to meet the person who manages to write down 2^64 enum values or more)
@illicitonion illicitonion merged commit d5a4caa into illicitonion:main Feb 16, 2023
@illicitonion
Copy link
Owner

Thanks!

@mlgiraud
Copy link
Contributor Author

Thank you for the quick merge. Could we maybe do a small bugfix release? 0.5.10?

@illicitonion
Copy link
Owner

Absolutely, I've just released 0.5.10 - thanks again for the contribution!

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.

2 participants