-
Notifications
You must be signed in to change notification settings - Fork 508
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
Allow using Box<[u8]>
/Box<str>
for bytes
/string
fields
#878
Conversation
I saw #429 but it's not what I wanted since it asks for two levels of indirection, I can probably easily add |
1525688
to
d8239f2
Compare
I tried to implement this in an external lib but it's inefficient if done naively (conversions to |
Please merge this, would be extremely helpful. |
d8239f2
to
fe3fa70
Compare
Rebased to fix conflicts, no actual changes made |
fe3fa70
to
d8f4c27
Compare
It seems like #979 is interesting for this PR. It would simplify the encoding needed for |
I'm not working with prost anymore and this PR is way out of date, someone interested can retake it if necessary :) |
This reduces the size of objects by one byte each time they're used, which can help quite a lot in some cases.
This is fully optional and the defaults have not changed.