Skip to content

Use the uguid crate to replace the Guid struct and guid! macro #777

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

Merged
merged 1 commit into from
Apr 29, 2023

Conversation

nicholasbishop
Copy link
Member

The uguid crate is something I open sourced at work. It can hopefully serve as a useful bit of shared vocabulary in the ecosystem for crates that need Windows-style GUIDs.

This should be mostly transparent for the end user except that the from_values method has been removed. Usually the guid! macro is a more convenient choice anyway, but new or from_bytes can also be used if needed.

This allows us to delete a bit of code and some tests, as well as slightly reducing our dependency on proc macros as the uguid::guid! macro is a declarative macro.

Checklist

  • Sensible git history (for example, squash "typo" or "fix" commits). See the Rewriting History guide for help.
  • Update the changelog (if necessary)

This should be mostly transparent for the end user except that the `from_values`
method has been removed. Usually the `guid!` macro is a more convenient choice
anyway, but `new` or `from_bytes` can also be used if needed.

This allows us to delete a bit of code and some tests, as well as slightly
reducing our dependency on proc macros as the `uguid::guid!` macro is a
declarative macro.
@phip1611 phip1611 merged commit 1ab4840 into rust-osdev:main Apr 29, 2023
@JohnAZoidberg
Copy link
Contributor

Nice, I like it. I can use this to use the same code for my UEFI and non-uefi code. Like ESRT parsing.

@nicholasbishop nicholasbishop deleted the bishop-uguid-3 branch April 29, 2023 15:52
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.

3 participants