Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Investigate using bytemuck #498

Closed
jorgecarleitao opened this issue Oct 8, 2021 · 1 comment
Closed

Investigate using bytemuck #498

jorgecarleitao opened this issue Oct 8, 2021 · 1 comment
Labels
enhancement An improvement to an existing feature investigation Issues or PRs that are investigations. Prs may or may not be merged. no-changelog Issues whose changes are covered by a PR and thus should not be shown in the changelog

Comments

@jorgecarleitao
Copy link
Owner

Which would allow us to replace some of the unsafe calls by the more battle-tested version. I think that we would need to make NativeType implement Pod.

https://docs.rs/bytemuck/1.7.2/bytemuck/
@jorgecarleitao jorgecarleitao added enhancement An improvement to an existing feature investigation Issues or PRs that are investigations. Prs may or may not be merged. labels Oct 8, 2021
@jorgecarleitao jorgecarleitao added the no-changelog Issues whose changes are covered by a PR and thus should not be shown in the changelog label Jan 13, 2022
@jorgecarleitao
Copy link
Owner Author

Closed by #749

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement An improvement to an existing feature investigation Issues or PRs that are investigations. Prs may or may not be merged. no-changelog Issues whose changes are covered by a PR and thus should not be shown in the changelog
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jorgecarleitao