Remove some unsafe; stabilize zerocopy #833
Conversation
Remove a number of `unsafe` blocks, replacing them with uses of zerocopy. In order to do this, we stabilize zerocopy as a (non-optional) dependency. Closes uuid-rs#588
|
Thanks for the PR @joshlf! I'm not immediately in favour of adding a non-optional dependency on As an aside, it would be really nice to see some support for the common pattern of transmuting values of |
| @@ -194,58 +252,59 @@ const fn format_hyphenated(src: &[u8; 16], upper: bool) -> [u8; 36] { | |||
| #[inline] | |||
| fn encode_simple<'b>(src: &[u8; 16], buffer: &'b mut [u8], upper: bool) -> &'b mut str { | |||
| let buf = &mut buffer[..Simple::LENGTH]; | |||
| let dst = buf.as_mut_ptr(); | |||
| let buf: &mut [u8; Simple::LENGTH] = buf.try_into().unwrap(); | |||
|
No worries! Feel free to steal the non-zerocopy changes from this PR.
Hopefully the Safe Transmute APIs will support this once they land. |
Remove a number of
unsafeblocks, replacing them with uses of zerocopy. In order to do this, we stabilize zerocopy as a (non-optional) dependency.Closes #588