-
-
Notifications
You must be signed in to change notification settings - Fork 279
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
split off BorrowDecode from Decode in bincode_derive #432
Conversation
Codecov Report
@@ Coverage Diff @@
## trunk #432 +/- ##
==========================================
- Coverage 71.04% 70.55% -0.49%
==========================================
Files 41 42 +1
Lines 2811 2839 +28
==========================================
+ Hits 1997 2003 +6
- Misses 814 836 +22
Continue to review full report at Codecov.
|
…DO for cow implementation specialization
I've taken the liberty to also re-export the |
14b6b89
to
b196fde
Compare
We've discussed splitting off
Decode
fromBorrowDecode
derive macro before. This PR actualizes that.This should offer people a solution to issues where the type implements
BorrowDecode
while people are explicitly requestingDecode
. It does not fix the issue aroundCow<T>
always needingDecode
, we can't fix that until specialization implementation.