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

Update Zero copy from 0.6.6 to 0.8.3 #360

Merged
merged 5 commits into from
Oct 11, 2024

Conversation

clundin25
Copy link
Contributor

  • Derive macros are now behind a feature flag derive.
  • AsBytes trait has been renamed to IntoBytes.
  • The
    Immutable trait has been added. This marker trait should be added to types that do not have interior mutation (E.g. don't use UnsafeCell or something similar).
  • The KnownLayout trait has been added. This marker trait indicates that zerocopy can reason about certain aspects of a type's layout.
  • The return signatures for some zerocopy methods have changed.
  • Use TryFromBytes for enums that have fewer states than their memory
    layout.

See upgrade guide for more information: google/zerocopy#1680.

Copy link

linux-foundation-easycla bot commented Oct 9, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

* Derive macros are now behind a feature flag `derive`.
* `AsBytes` trait has been renamed to `IntoBytes`.
* The
  [Immutable](https://docs.rs/zerocopy/0.8.3/zerocopy/trait.Immutable.html) trait has been added. This marker trait should be added to types that do not have interior mutation (E.g. don't use UnsafeCell or something similar).
* The [KnownLayout](https://docs.rs/zerocopy/0.8.3/zerocopy/trait.KnownLayout.html) trait has been added. This marker trait indicates that zerocopy can reason about certain aspects of a type's layout.
* The return signatures for some zerocopy methods have changed.
* Use `TryFromBytes` for enums that have fewer states than their memory
  layout.

See upgrade guide for more information: google/zerocopy#1680.
jhand2
jhand2 previously approved these changes Oct 10, 2024
Copy link
Collaborator

@jhand2 jhand2 left a comment

Choose a reason for hiding this comment

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

Nice!

@clundin25 clundin25 requested a review from jhand2 October 11, 2024 20:58
@clundin25
Copy link
Contributor Author

Opened #361 to track re-enabling the fuzzer build check.

@jhand2 jhand2 merged commit 7f172f1 into chipsalliance:main Oct 11, 2024
2 checks passed
@clundin25 clundin25 mentioned this pull request Oct 16, 2024
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