diff options
| author | Samuel Tardieu <sam@rfc1149.net> | 2025-03-26 20:38:58 +0100 |
|---|---|---|
| committer | Samuel Tardieu <sam@rfc1149.net> | 2025-03-27 14:40:44 +0100 |
| commit | 79c69112dc1dd5803af4e2682552c1ad2dde14ca (patch) | |
| tree | bcb0ea10bee9603dc677ca29cb7af6d4fa62eb03 /clippy_lints/src/missing_inline.rs | |
| parent | cd701524701bbc3443a007b52edcee6550abb0be (diff) | |
Apply `collapsible_if` to Clippy itself
Since Clippy uses the `let_chains` feature, there are many occasions to collapse `if` and `if let` statements.
Diffstat (limited to 'clippy_lints/src/missing_inline.rs')
| -rw-r--r-- | clippy_lints/src/missing_inline.rs | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/clippy_lints/src/missing_inline.rs b/clippy_lints/src/missing_inline.rs index f49e03ea765..1f613171b46 100644 --- a/clippy_lints/src/missing_inline.rs +++ b/clippy_lints/src/missing_inline.rs @@ -160,12 +160,13 @@ impl<'tcx> LateLintPass<'tcx> for MissingInline { AssocItemContainer::Impl => cx.tcx.impl_trait_ref(container_id).map(|t| t.skip_binder().def_id), }; - if let Some(trait_def_id) = trait_def_id { - if trait_def_id.is_local() && !cx.effective_visibilities.is_exported(impl_item.owner_id.def_id) { - // If a trait is being implemented for an item, and the - // trait is not exported, we don't need #[inline] - return; - } + if let Some(trait_def_id) = trait_def_id + && trait_def_id.is_local() + && !cx.effective_visibilities.is_exported(impl_item.owner_id.def_id) + { + // If a trait is being implemented for an item, and the + // trait is not exported, we don't need #[inline] + return; } let attrs = cx.tcx.hir_attrs(impl_item.hir_id()); |
