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 more things const #535

Merged
merged 1 commit into from Oct 28, 2021
Merged

Make more things const #535

merged 1 commit into from Oct 28, 2021

Conversation

KodrAus
Copy link
Member

@KodrAus KodrAus commented Oct 28, 2021

The set of things you can write in const fn has been steadily growing. This PR makes some more Uuid functions const. We can't do the as_fields methods yet without copying because we can't transmute a slice into a [u8; 8], but that should change in the future. For now I've just left them as-is.

@KodrAus
Copy link
Member Author

@KodrAus KodrAus commented Oct 28, 2021

This might need another MSRV bump to support. Depending on how recent that is I might end up backing it out.

@KodrAus
Copy link
Member Author

@KodrAus KodrAus commented Oct 28, 2021

Ah great! Looks like everything works in 1.46.

@KodrAus KodrAus merged commit 07f920f into uuid-rs:main Oct 28, 2021
18 checks passed
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.

None yet

1 participant