-
Notifications
You must be signed in to change notification settings - Fork 745
Tidy logging macros #2106
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
Tidy logging macros #2106
Conversation
Normalise the fact that logging macros live in `crate::log`, rather than a difference place depending on the `logging` feature.
Benchmark resultsInstruction countsSignificant differencesClick to expand
Other differencesClick to expand
Wall-timeSignificant differencesThere are no significant wall-time differences Other differencesClick to expand
Additional informationCheckout details:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2106 +/- ##
=======================================
Coverage 94.76% 94.76%
=======================================
Files 102 102
Lines 23514 23514
=======================================
Hits 22283 22283
Misses 1231 1231 ☔ View full report in Codecov by Sentry. |
I think this is pretty minor, merging with one approval. |
Normalise the fact that logging macros live in
crate::log
, rather than a difference place depending on thelogging
feature.ref rustls/hyper-rustls#285 (comment)