diff options
| author | bors <bors@rust-lang.org> | 2023-08-17 14:41:46 +0000 |
|---|---|---|
| committer | Philipp Krones <hello@philkrones.com> | 2023-08-17 18:06:36 +0200 |
| commit | 09d05c04da516d4f54ccf41e22b544bc8c425373 (patch) | |
| tree | 5b63d9f47c3a7b4e9ac5723e6ee54b5f32bd5993 /tests/codegen/trailing_zeros.rs | |
| parent | bd138e2ae11050ea9ca7a187345a1b606697a8db (diff) | |
| download | rust-09d05c04da516d4f54ccf41e22b544bc8c425373.tar.gz rust-09d05c04da516d4f54ccf41e22b544bc8c425373.zip | |
Auto merge of #11070 - y21:issue11065, r=flip1995
[`useless_conversion`]: only lint on paths to fn items and fix FP in macro Fixes #11065 (which is actually two issues: an ICE and a false positive) It now makes sure that the function call path points to a function-like item (and not e.g. a `const` like in the linked issue), so that calling `TyCtxt::fn_sig` later in the lint does not ICE (fixes https://github.com/rust-lang/rust-clippy/issues/11065#issuecomment-1616836099). It *also* makes sure that the expression is not part of a macro call (fixes https://github.com/rust-lang/rust-clippy/issues/11065#issuecomment-1616919639). ~~I'm not sure if there's a better way to check this other than to walk the parent expr chain and see if any of them are expansions.~~ (edit: it doesn't do this anymore) changelog: [`useless_conversion`]: fix ICE when call receiver is a non-fn item changelog: [`useless_conversion`]: don't lint if argument is a macro argument (fixes a FP) r? `@llogiq` (reviewed #10814, which introduced these issues)
Diffstat (limited to 'tests/codegen/trailing_zeros.rs')
0 files changed, 0 insertions, 0 deletions