search

multiformats / rust-multiaddr

multiaddr implementation in rust
https://crates.io/crates/multiaddr
Other
88 stars 47 forks source link

Replace `Arc<Vec<u8>>` with `Arc<[u8]>` #117

Closed nazar-pc closed 1 month ago

nazar-pc commented 1 month ago

Dual indirection is completely unnecessary: https://github.com/multiformats/rust-multiaddr/blob/469b0deb9a64554c11e1d2227c78fe7c6c769aa7/src/lib.rs#L44-L46

jxs commented 1 month ago

Hi Nazar, thanks. Would you like to submit a PR addressing it?