diff options
| author | Matthias Krüger <matthias.krueger@famsik.de> | 2022-11-15 01:40:44 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-11-15 01:40:44 +0100 |
| commit | 2c29b05fb24095b0bc2e792ffed3df7496cc2a59 (patch) | |
| tree | 9d888bb5c6cd877858063b5d1c4dd888dee7918f /src/tools | |
| parent | 0a3cb1ed8138900e8697d42988ee41250c3d69ad (diff) | |
| parent | 29fe28fcfc01f6b850744cc918dbc07a82978755 (diff) | |
| download | rust-2c29b05fb24095b0bc2e792ffed3df7496cc2a59.tar.gz rust-2c29b05fb24095b0bc2e792ffed3df7496cc2a59.zip | |
Rollup merge of #104383 - WaffleLapkin:rustc_undiagnostic_item, r=compiler-errors
Remove unused symbols and diagnostic items As the title suggests, this removes unused symbols from `sym::` and `#[rustc_diagnostic_item]` annotations that weren't mentioned anywhere. Originally I tried to use grep, to find symbols and item names that are never mentioned via `sym::name`, however this produced a lot of false positives (?), for example clippy matching on `Symbol::as_str` or macros "implicitly" adding `sym::`. I ended up fixing all these false positives (?) by hand, but tbh I'm not sure if it was worth it...
Diffstat (limited to 'src/tools')
| -rw-r--r-- | src/tools/clippy/clippy_utils/src/macros.rs | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/tools/clippy/clippy_utils/src/macros.rs b/src/tools/clippy/clippy_utils/src/macros.rs index 9a682fbe604..d13b34a66cc 100644 --- a/src/tools/clippy/clippy_utils/src/macros.rs +++ b/src/tools/clippy/clippy_utils/src/macros.rs @@ -199,12 +199,12 @@ pub fn first_node_in_macro(cx: &LateContext<'_>, node: &impl HirNode) -> Option< pub fn is_panic(cx: &LateContext<'_>, def_id: DefId) -> bool { let Some(name) = cx.tcx.get_diagnostic_name(def_id) else { return false }; matches!( - name.as_str(), - "core_panic_macro" - | "std_panic_macro" - | "core_panic_2015_macro" - | "std_panic_2015_macro" - | "core_panic_2021_macro" + name, + sym::core_panic_macro + | sym::std_panic_macro + | sym::core_panic_2015_macro + | sym::std_panic_2015_macro + | sym::core_panic_2021_macro ) } |
