-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Even more fixes for handling of macros #16443
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
base: master
Are you sure you want to change the base?
Conversation
962a33a to
5f63167
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.
There are a bunch of places where expr.span.ctxt() could've been reused -- highlighted just some of them
clippy_lints/src/methods/map_with_unused_argument_over_ranges.rs
Outdated
Show resolved
Hide resolved
01a1156 to
3142977
Compare
3142977 to
455a8c1
Compare
|
This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
Continuation #16337
Even more fixes for handling macros
changelog: [
manual_rotate] fix wrongly unmangled macroschangelog: [
map_with_unused_argument_over_ranges] fix wrongly unmangled macroschangelog: [
needless_bool] fix wrongly unmangled macroschangelog: [
manual_is_power_of_two] fix wrongly unmangled macroschangelog: [
manual_div_ceil] fix wrongly unmangled macroschangelog: [
implicit_saturating_sub] fix wrongly unmangled macroschangelog: [
range_minus_one] fix wrongly unmangled macroschangelog: [
range_plus_one] fix wrongly unmangled macroschangelog: [
manual_swap] fix wrongly unmangled macroschangelog: [
let_and_return] fix wrongly unmangled macros