From b91a6fcd5d82da723d7af1bf6790fd93a7741e93 Mon Sep 17 00:00:00 2001 From: Guillaume Gomez Date: Tue, 26 Nov 2019 13:25:14 +0100 Subject: [PATCH] Replace Iterator::find calls with Iterator::any when better --- src/librustdoc/passes/collect_intra_doc_links.rs | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/src/librustdoc/passes/collect_intra_doc_links.rs b/src/librustdoc/passes/collect_intra_doc_links.rs index 3c9b2f2154b2..3c632462168b 100644 --- a/src/librustdoc/passes/collect_intra_doc_links.rs +++ b/src/librustdoc/passes/collect_intra_doc_links.rs @@ -84,17 +84,16 @@ impl<'a, 'tcx> LinkCollector<'a, 'tcx> { let ty_res = ty_res.map_id(|_| panic!("unexpected node_id")); match ty_res { Res::Def(DefKind::Enum, did) => { - let item = cx.tcx.inherent_impls(did) - .iter() - .flat_map(|imp| cx.tcx.associated_items(*imp)) - .find(|item| item.ident.name == variant_name); - if item.is_some() { + if cx.tcx.inherent_impls(did) + .iter() + .flat_map(|imp| cx.tcx.associated_items(*imp)) + .any(|item| item.ident.name == variant_name) { return Err(()); } match cx.tcx.type_of(did).kind { ty::Adt(def, _) if def.is_enum() => { if def.all_fields() - .find(|item| item.ident.name == variant_field_name).is_some() { + .any(|item| item.ident.name == variant_field_name) { Ok((ty_res, Some(format!("variant.{}.field.{}", variant_name, variant_field_name))))