diff options
| author | Michael Goulet <michael@errs.io> | 2022-02-10 23:42:28 -0800 |
|---|---|---|
| committer | Michael Goulet <michael@errs.io> | 2022-02-15 09:17:09 -0800 |
| commit | 764839320c247b813528533c79d4b25a4f55f5fd (patch) | |
| tree | 67546778c9f2648771d9d54d98451706c1a740a1 | |
| parent | 5b2291cfa6ae9374ca22cf5d0aafde1a3f23ea57 (diff) | |
| download | rust-764839320c247b813528533c79d4b25a4f55f5fd.tar.gz rust-764839320c247b813528533c79d4b25a4f55f5fd.zip | |
rename some variables in gat wfcheck
| -rw-r--r-- | compiler/rustc_typeck/src/check/wfcheck.rs | 43 |
1 files changed, 18 insertions, 25 deletions
diff --git a/compiler/rustc_typeck/src/check/wfcheck.rs b/compiler/rustc_typeck/src/check/wfcheck.rs index 9a269e02448..1864101467b 100644 --- a/compiler/rustc_typeck/src/check/wfcheck.rs +++ b/compiler/rustc_typeck/src/check/wfcheck.rs @@ -275,30 +275,27 @@ pub fn check_trait_item(tcx: TyCtxt<'_>, def_id: LocalDefId) { /// fn next<'a>(&'a mut self) -> Option<Self::Item<'a>>; /// } /// ``` -fn check_gat_where_clauses( - tcx: TyCtxt<'_>, - trait_item: &hir::TraitItem<'_>, - encl_trait_def_id: DefId, -) { - let item = tcx.associated_item(trait_item.def_id); +fn check_gat_where_clauses(tcx: TyCtxt<'_>, gat_hir: &hir::TraitItem<'_>, gat_def_id: DefId) { + let gat_item = tcx.associated_item(gat_def_id); + let gat_def_id = gat_hir.def_id; // If the current trait item isn't a type, it isn't a GAT - if !matches!(item.kind, ty::AssocKind::Type) { + if !matches!(gat_item.kind, ty::AssocKind::Type) { return; } - let generics: &ty::Generics = tcx.generics_of(trait_item.def_id); + let gat_generics: &ty::Generics = tcx.generics_of(gat_def_id); // If the current associated type doesn't have any (own) params, it's not a GAT // FIXME(jackh726): we can also warn in the more general case - if generics.params.len() == 0 { + if gat_generics.params.len() == 0 { return; } - let associated_items: &ty::AssocItems<'_> = tcx.associated_items(encl_trait_def_id); + let associated_items: &ty::AssocItems<'_> = tcx.associated_items(gat_def_id); let mut clauses: Option<FxHashSet<ty::Predicate<'_>>> = None; // For every function in this trait... // In our example, this would be the `next` method for item in associated_items.in_definition_order().filter(|item| matches!(item.kind, ty::AssocKind::Fn)) { - let id = hir::HirId::make_owner(item.def_id.expect_local()); + let item_hir_id = hir::HirId::make_owner(item.def_id.expect_local()); let param_env = tcx.param_env(item.def_id.expect_local()); // Get the signature using placeholders. In our example, this would @@ -314,11 +311,11 @@ fn check_gat_where_clauses( let function_clauses = gather_gat_bounds( tcx, param_env, - id, + item_hir_id, sig.output(), &wf_tys, - trait_item.def_id, - generics, + gat_def_id, + gat_generics, ); if let Some(function_clauses) = function_clauses { @@ -347,7 +344,7 @@ fn check_gat_where_clauses( let clauses = clauses.unwrap_or_default(); debug!(?clauses); if !clauses.is_empty() { - let param_env = tcx.param_env(trait_item.def_id); + let param_env = tcx.param_env(gat_def_id); let mut clauses: Vec<_> = clauses .into_iter() @@ -355,7 +352,7 @@ fn check_gat_where_clauses( ty::PredicateKind::RegionOutlives(ty::OutlivesPredicate(a, b)) => { !region_known_to_outlive( tcx, - trait_item.hir_id(), + gat_hir.hir_id(), param_env, &FxHashSet::default(), a, @@ -365,7 +362,7 @@ fn check_gat_where_clauses( ty::PredicateKind::TypeOutlives(ty::OutlivesPredicate(a, b)) => { !ty_known_to_outlive( tcx, - trait_item.hir_id(), + gat_hir.hir_id(), param_env, &FxHashSet::default(), a, @@ -383,21 +380,17 @@ fn check_gat_where_clauses( if !clauses.is_empty() { let plural = if clauses.len() > 1 { "s" } else { "" }; let mut err = tcx.sess.struct_span_err( - trait_item.span, - &format!("missing required bound{} on `{}`", plural, trait_item.ident), + gat_hir.span, + &format!("missing required bound{} on `{}`", plural, gat_hir.ident), ); let suggestion = format!( "{} {}", - if !trait_item.generics.where_clause.predicates.is_empty() { - "," - } else { - " where" - }, + if !gat_hir.generics.where_clause.predicates.is_empty() { "," } else { " where" }, clauses.join(", "), ); err.span_suggestion( - trait_item.generics.where_clause.tail_span_for_suggestion(), + gat_hir.generics.where_clause.tail_span_for_suggestion(), &format!("add the required where clause{}", plural), suggestion, Applicability::MachineApplicable, |
