-
Notifications
You must be signed in to change notification settings - Fork 13.3k
rustdoc: Cleanup two things in clean::types
#91510
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
Conversation
Rationale: * The name was confusing. * It was only used in one place. * That place didn't actually need all the functionality of `get_type`; rather, removing `get_type` makes that code clearer.
r? @CraftSpider (rust-highfive has picked a reviewer for you, use r? to override) |
Thanks! |
@bors: r+ rollup |
📌 Commit 51ca2cc has been approved by |
It's possible the search index change would have a slight perf effect, so @bors rollup=iffy |
☀️ Test successful - checks-actions |
Finished benchmarking commit (cafc458): comparison url. Summary: This benchmark run did not return any relevant changes. If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression |
Explanations are in the individual commits.