Simplify visit_{foreign,trait}_item

Using an `if` seems like a better semantic fit and saves a few lines.
This commit is contained in:
LingMan 2020-12-08 15:37:39 +01:00
parent 5019791e2d
commit 7654b12112

View File

@ -423,15 +423,11 @@ impl<'v, 'k, 'tcx> ItemLikeVisitor<'v> for LifeSeeder<'k, 'tcx> {
} }
fn visit_trait_item(&mut self, trait_item: &hir::TraitItem<'_>) { fn visit_trait_item(&mut self, trait_item: &hir::TraitItem<'_>) {
match trait_item.kind { use hir::TraitItemKind::{Const, Fn};
hir::TraitItemKind::Const(_, Some(_)) if matches!(trait_item.kind, Const(_, Some(_)) | Fn(_, hir::TraitFn::Provided(_)))
| hir::TraitItemKind::Fn(_, hir::TraitFn::Provided(_)) => { && has_allow_dead_code_or_lang_attr(self.tcx, trait_item.hir_id, &trait_item.attrs)
if has_allow_dead_code_or_lang_attr(self.tcx, trait_item.hir_id, &trait_item.attrs) {
{ self.worklist.push(trait_item.hir_id);
self.worklist.push(trait_item.hir_id);
}
}
_ => {}
} }
} }
@ -440,17 +436,11 @@ impl<'v, 'k, 'tcx> ItemLikeVisitor<'v> for LifeSeeder<'k, 'tcx> {
} }
fn visit_foreign_item(&mut self, foreign_item: &hir::ForeignItem<'_>) { fn visit_foreign_item(&mut self, foreign_item: &hir::ForeignItem<'_>) {
match foreign_item.kind { use hir::ForeignItemKind::{Fn, Static};
hir::ForeignItemKind::Static(..) | hir::ForeignItemKind::Fn(..) => { if matches!(foreign_item.kind, Static(..) | Fn(..))
if has_allow_dead_code_or_lang_attr( && has_allow_dead_code_or_lang_attr(self.tcx, foreign_item.hir_id, &foreign_item.attrs)
self.tcx, {
foreign_item.hir_id, self.worklist.push(foreign_item.hir_id);
&foreign_item.attrs,
) {
self.worklist.push(foreign_item.hir_id);
}
}
_ => {}
} }
} }
} }