-
Notifications
You must be signed in to change notification settings - Fork 714
fix rustls-provider-example hpke no-std support #2269
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
fix rustls-provider-example hpke no-std support #2269
Conversation
be9d91c
to
e79fe79
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2269 +/- ##
=======================================
Coverage 94.59% 94.59%
=======================================
Files 104 104
Lines 24031 24031
=======================================
Hits 22731 22731
Misses 1300 1300 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should remove some things not needed for no-std
e79fe79
to
ba6520b
Compare
ba6520b
to
d0b217b
Compare
(with some very minor refactoring)
d0b217b
to
f521c0f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
Benchmark resultsInstruction countsSignificant differencesThere are no significant instruction count differences Other differencesClick to expand
Wall-timeSignificant differencesThere are no significant wall-time differences Other differencesClick to expand
Additional informationCheckout details:
|
(with some very minor refactoring)
should be finally ready now ... apologies for so much noise ... looks like codecov is complaining about 2 lines of code for
no-std
since I don't think we can unit-test this; I don't think this should be an issue since CI does already verify that we can compile this forno-std