Description
Currently, most of zerocopy's abstractions don't work in a const context for one reason or another. We should change that as much as possible.
This is a blocker for ICU4X using zerocopy.
Trait methods
One large subset of the API is the set of trait methods. Most or all of these don't rely on calling other trait methods, but effectively just use the trait bound to guarantee that the operation is sound. This entire subset in principle ought to be representable as bare functions, and many of them would likely be convertible into const fn
s. We could of course keep the trait methods as well, and just implement them in terms of the functions.
This approach has two big downsides:
- It would effectively duplicate a large subset of our API surface
- We would probably want to remove the bare functions in a future release once all of the trait methods could themselves be const (it's not clear to me how far off this is)
TryFromBytes
Our derive for TryFromBytes
automatically derives the is_bit_valid
method. In order to support const
TryFromBytes
, we would need to:
- Emit an inherent
#[doc(hidden)] pub fn __zerocopy_is_bit_valid(...) -> bool
- Rewrite our impl of
TryFromBytes::is_bit_valid
to call this function - Possibly provide an attribute to allow renaming that function for macro hygiene reasons
- Note: It's unclear whether this would work at all since other const code needs to be able to call the function at a well-known name
- If it doesn't work on our MSRV, we could just make it an optional
zerocopy-derive
feature (which is exposed viazerocopy
with a feature of the same or similar name)
Note a very fundamental limitation: There would be no way to support generic types (e.g., #[derive(TryFromBytes)] struct Foo<T>(T)
) because the only way to invoke the function would be to name the type it operates on (e.g., Foo::__zerocopy_is_bit_valid(...)
). This also means that the public API would need to be a macro in order to emit the appropriate code.