From 79c69112dc1dd5803af4e2682552c1ad2dde14ca Mon Sep 17 00:00:00 2001 From: Samuel Tardieu Date: Wed, 26 Mar 2025 20:38:58 +0100 Subject: Apply `collapsible_if` to Clippy itself Since Clippy uses the `let_chains` feature, there are many occasions to collapse `if` and `if let` statements. --- clippy_lints/src/missing_inline.rs | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) (limited to 'clippy_lints/src/missing_inline.rs') 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()); -- cgit 1.4.1-3-g733a5