1 //! Error Reporting Code for the inference engine
3 //! Because of the way inference, and in particular region inference,
4 //! works, it often happens that errors are not detected until far after
5 //! the relevant line of code has been type-checked. Therefore, there is
6 //! an elaborate system to track why a particular constraint in the
7 //! inference graph arose so that we can explain to the user what gave
8 //! rise to a particular error.
10 //! The basis of the system are the "origin" types. An "origin" is the
11 //! reason that a constraint or inference variable arose. There are
12 //! different "origin" enums for different kinds of constraints/variables
13 //! (e.g., `TypeOrigin`, `RegionVariableOrigin`). An origin always has
14 //! a span, but also more information so that we can generate a meaningful
17 //! Having a catalog of all the different reasons an error can arise is
18 //! also useful for other reasons, like cross-referencing FAQs etc, though
19 //! we are not really taking advantage of this yet.
21 //! # Region Inference
23 //! Region inference is particularly tricky because it always succeeds "in
24 //! the moment" and simply registers a constraint. Then, at the end, we
25 //! can compute the full graph and report errors, so we need to be able to
26 //! store and later report what gave rise to the conflicting constraints.
30 //! Determining whether `T1 <: T2` often involves a number of subtypes and
31 //! subconstraints along the way. A "TypeTrace" is an extended version
32 //! of an origin that traces the types and other values that were being
33 //! compared. It is not necessarily comprehensive (in fact, at the time of
34 //! this writing it only tracks the root values being compared) but I'd
35 //! like to extend it to include significant "waypoints". For example, if
36 //! you are comparing `(T1, T2) <: (T3, T4)`, and the problem is that `T2
37 //! <: T4` fails, I'd like the trace to include enough information to say
38 //! "in the 2nd element of the tuple". Similarly, failures when comparing
39 //! arguments or return types in fn types should be able to cite the
40 //! specific position, etc.
44 //! Of course, there is still a LOT of code in typeck that has yet to be
45 //! ported to this system, and which relies on string concatenation at the
46 //! time of error detection.
48 use super::lexical_region_resolve::RegionResolutionError;
49 use super::region_constraints::GenericKind;
50 use super::{InferCtxt, RegionVariableOrigin, SubregionOrigin, TypeTrace, ValuePairs};
51 use crate::infer::{self, SuppressRegionErrors};
54 use crate::hir::def_id::DefId;
56 use crate::middle::region;
57 use crate::traits::{ObligationCause, ObligationCauseCode};
58 use crate::ty::error::TypeError;
59 use crate::ty::{self, subst::{Subst, SubstsRef}, Region, Ty, TyCtxt, TypeFoldable};
60 use errors::{Applicability, DiagnosticBuilder, DiagnosticStyledString};
62 use syntax_pos::{Pos, Span};
68 pub mod nice_region_error;
70 impl<'a, 'gcx, 'tcx> TyCtxt<'a, 'gcx, 'tcx> {
71 pub fn note_and_explain_region(
73 region_scope_tree: ®ion::ScopeTree,
74 err: &mut DiagnosticBuilder<'_>,
76 region: ty::Region<'tcx>,
79 let (description, span) = match *region {
80 ty::ReScope(scope) => {
82 let unknown_scope = || {
84 "{}unknown scope: {:?}{}. Please report a bug.",
88 let span = scope.span(self, region_scope_tree);
89 let tag = match self.hir().find(scope.node_id(self, region_scope_tree)) {
90 Some(Node::Block(_)) => "block",
91 Some(Node::Expr(expr)) => match expr.node {
92 hir::ExprKind::Call(..) => "call",
93 hir::ExprKind::MethodCall(..) => "method call",
94 hir::ExprKind::Match(.., hir::MatchSource::IfLetDesugar { .. }) => "if let",
95 hir::ExprKind::Match(.., hir::MatchSource::WhileLetDesugar) => "while let",
96 hir::ExprKind::Match(.., hir::MatchSource::ForLoopDesugar) => "for",
97 hir::ExprKind::Match(..) => "match",
100 Some(Node::Stmt(_)) => "statement",
101 Some(Node::Item(it)) => Self::item_scope_tag(&it),
102 Some(Node::TraitItem(it)) => Self::trait_item_scope_tag(&it),
103 Some(Node::ImplItem(it)) => Self::impl_item_scope_tag(&it),
105 err.span_note(span, &unknown_scope());
109 let scope_decorated_tag = match scope.data {
110 region::ScopeData::Node => tag,
111 region::ScopeData::CallSite => "scope of call-site for function",
112 region::ScopeData::Arguments => "scope of function body",
113 region::ScopeData::Destruction => {
114 new_string = format!("destruction scope surrounding {}", tag);
117 region::ScopeData::Remainder(first_statement_index) => {
118 new_string = format!(
119 "block suffix following statement {}",
120 first_statement_index.index()
125 self.explain_span(scope_decorated_tag, span)
128 ty::ReEarlyBound(_) | ty::ReFree(_) | ty::ReStatic => {
129 self.msg_span_from_free_region(region)
132 ty::ReEmpty => ("the empty lifetime".to_owned(), None),
134 ty::RePlaceholder(_) => (format!("any other region"), None),
136 // FIXME(#13998) RePlaceholder should probably print like
137 // ReFree rather than dumping Debug output on the user.
139 // We shouldn't really be having unification failures with ReVar
140 // and ReLateBound though.
141 ty::ReVar(_) | ty::ReLateBound(..) | ty::ReErased => {
142 (format!("lifetime {:?}", region), None)
145 // We shouldn't encounter an error message with ReClosureBound.
146 ty::ReClosureBound(..) => {
147 bug!("encountered unexpected ReClosureBound: {:?}", region,);
151 TyCtxt::emit_msg_span(err, prefix, description, span, suffix);
154 pub fn note_and_explain_free_region(
156 err: &mut DiagnosticBuilder<'_>,
158 region: ty::Region<'tcx>,
161 let (description, span) = self.msg_span_from_free_region(region);
163 TyCtxt::emit_msg_span(err, prefix, description, span, suffix);
166 fn msg_span_from_free_region(self, region: ty::Region<'tcx>) -> (String, Option<Span>) {
168 ty::ReEarlyBound(_) | ty::ReFree(_) => {
169 self.msg_span_from_early_bound_and_free_regions(region)
171 ty::ReStatic => ("the static lifetime".to_owned(), None),
172 ty::ReEmpty => ("an empty lifetime".to_owned(), None),
173 _ => bug!("{:?}", region),
177 fn msg_span_from_early_bound_and_free_regions(
179 region: ty::Region<'tcx>,
180 ) -> (String, Option<Span>) {
181 let cm = self.sess.source_map();
183 let scope = region.free_region_binding_scope(self);
184 let node = self.hir().as_local_hir_id(scope).unwrap_or(hir::DUMMY_HIR_ID);
185 let tag = match self.hir().find_by_hir_id(node) {
186 Some(Node::Block(_)) | Some(Node::Expr(_)) => "body",
187 Some(Node::Item(it)) => Self::item_scope_tag(&it),
188 Some(Node::TraitItem(it)) => Self::trait_item_scope_tag(&it),
189 Some(Node::ImplItem(it)) => Self::impl_item_scope_tag(&it),
192 let (prefix, span) = match *region {
193 ty::ReEarlyBound(ref br) => {
194 let mut sp = cm.def_span(self.hir().span_by_hir_id(node));
195 if let Some(param) = self.hir()
197 .and_then(|generics| generics.get_named(&br.name))
201 (format!("the lifetime {} as defined on", br.name), sp)
203 ty::ReFree(ty::FreeRegion {
204 bound_region: ty::BoundRegion::BrNamed(_, ref name),
207 let mut sp = cm.def_span(self.hir().span_by_hir_id(node));
208 if let Some(param) = self.hir()
210 .and_then(|generics| generics.get_named(&name))
214 (format!("the lifetime {} as defined on", name), sp)
216 ty::ReFree(ref fr) => match fr.bound_region {
218 format!("the anonymous lifetime #{} defined on", idx + 1),
219 self.hir().span_by_hir_id(node),
222 "an anonymous lifetime defined on".to_owned(),
223 self.hir().span_by_hir_id(node),
226 format!("the lifetime {} as defined on", region),
227 cm.def_span(self.hir().span_by_hir_id(node)),
232 let (msg, opt_span) = self.explain_span(tag, span);
233 (format!("{} {}", prefix, msg), opt_span)
237 err: &mut DiagnosticBuilder<'_>,
243 let message = format!("{}{}{}", prefix, description, suffix);
245 if let Some(span) = span {
246 err.span_note(span, &message);
252 fn item_scope_tag(item: &hir::Item) -> &'static str {
254 hir::ItemKind::Impl(..) => "impl",
255 hir::ItemKind::Struct(..) => "struct",
256 hir::ItemKind::Union(..) => "union",
257 hir::ItemKind::Enum(..) => "enum",
258 hir::ItemKind::Trait(..) => "trait",
259 hir::ItemKind::Fn(..) => "function body",
264 fn trait_item_scope_tag(item: &hir::TraitItem) -> &'static str {
266 hir::TraitItemKind::Method(..) => "method body",
267 hir::TraitItemKind::Const(..) | hir::TraitItemKind::Type(..) => "associated item",
271 fn impl_item_scope_tag(item: &hir::ImplItem) -> &'static str {
273 hir::ImplItemKind::Method(..) => "method body",
274 hir::ImplItemKind::Const(..)
275 | hir::ImplItemKind::Existential(..)
276 | hir::ImplItemKind::Type(..) => "associated item",
280 fn explain_span(self, heading: &str, span: Span) -> (String, Option<Span>) {
281 let lo = self.sess.source_map().lookup_char_pos_adj(span.lo());
283 format!("the {} at {}:{}", heading, lo.line, lo.col.to_usize() + 1),
289 impl<'a, 'gcx, 'tcx> InferCtxt<'a, 'gcx, 'tcx> {
290 pub fn report_region_errors(
292 region_scope_tree: ®ion::ScopeTree,
293 errors: &Vec<RegionResolutionError<'tcx>>,
294 suppress: SuppressRegionErrors,
297 "report_region_errors(): {} errors to start, suppress = {:?}",
302 if suppress.suppressed() {
306 // try to pre-process the errors, which will group some of them
307 // together into a `ProcessedErrors` group:
308 let errors = self.process_errors(errors);
311 "report_region_errors: {} errors after preprocessing",
315 for error in errors {
316 debug!("report_region_errors: error = {:?}", error);
318 if !self.try_report_nice_region_error(&error) {
319 match error.clone() {
320 // These errors could indicate all manner of different
321 // problems with many different solutions. Rather
322 // than generate a "one size fits all" error, what we
323 // attempt to do is go through a number of specific
324 // scenarios and try to find the best way to present
325 // the error. If all of these fails, we fall back to a rather
326 // general bit of code that displays the error information
327 RegionResolutionError::ConcreteFailure(origin, sub, sup) => {
328 if sub.is_placeholder() || sup.is_placeholder() {
329 self.report_placeholder_failure(region_scope_tree, origin, sub, sup)
332 self.report_concrete_failure(region_scope_tree, origin, sub, sup)
337 RegionResolutionError::GenericBoundFailure(origin, param_ty, sub) => {
338 self.report_generic_bound_failure(
347 RegionResolutionError::SubSupConflict(
355 if sub_r.is_placeholder() {
356 self.report_placeholder_failure(
363 } else if sup_r.is_placeholder() {
364 self.report_placeholder_failure(
372 self.report_sub_sup_conflict(
387 // This method goes through all the errors and try to group certain types
388 // of error together, for the purpose of suggesting explicit lifetime
389 // parameters to the user. This is done so that we can have a more
390 // complete view of what lifetimes should be the same.
391 // If the return value is an empty vector, it means that processing
392 // failed (so the return value of this method should not be used).
394 // The method also attempts to weed out messages that seem like
395 // duplicates that will be unhelpful to the end-user. But
396 // obviously it never weeds out ALL errors.
399 errors: &Vec<RegionResolutionError<'tcx>>,
400 ) -> Vec<RegionResolutionError<'tcx>> {
401 debug!("process_errors()");
403 // We want to avoid reporting generic-bound failures if we can
404 // avoid it: these have a very high rate of being unhelpful in
405 // practice. This is because they are basically secondary
406 // checks that test the state of the region graph after the
407 // rest of inference is done, and the other kinds of errors
408 // indicate that the region constraint graph is internally
409 // inconsistent, so these test results are likely to be
412 // Therefore, we filter them out of the list unless they are
413 // the only thing in the list.
415 let is_bound_failure = |e: &RegionResolutionError<'tcx>| match *e {
416 RegionResolutionError::GenericBoundFailure(..) => true,
417 RegionResolutionError::ConcreteFailure(..)
418 | RegionResolutionError::SubSupConflict(..) => false,
421 let mut errors = if errors.iter().all(|e| is_bound_failure(e)) {
426 .filter(|&e| !is_bound_failure(e))
431 // sort the errors by span, for better error message stability.
432 errors.sort_by_key(|u| match *u {
433 RegionResolutionError::ConcreteFailure(ref sro, _, _) => sro.span(),
434 RegionResolutionError::GenericBoundFailure(ref sro, _, _) => sro.span(),
435 RegionResolutionError::SubSupConflict(_, ref rvo, _, _, _, _) => rvo.span(),
440 /// Adds a note if the types come from similarly named crates
441 fn check_and_note_conflicting_crates(
443 err: &mut DiagnosticBuilder<'_>,
444 terr: &TypeError<'tcx>,
447 use hir::def_id::CrateNum;
448 use hir::map::DisambiguatedDefPathData;
449 use ty::print::Printer;
452 struct AbsolutePathPrinter<'a, 'gcx, 'tcx> {
453 tcx: TyCtxt<'a, 'gcx, 'tcx>,
456 struct NonTrivialPath;
458 impl<'gcx, 'tcx> Printer<'gcx, 'tcx> for AbsolutePathPrinter<'_, 'gcx, 'tcx> {
459 type Error = NonTrivialPath;
461 type Path = Vec<String>;
464 type DynExistential = !;
466 fn tcx<'a>(&'a self) -> TyCtxt<'a, 'gcx, 'tcx> {
472 _region: ty::Region<'_>,
473 ) -> Result<Self::Region, Self::Error> {
480 ) -> Result<Self::Type, Self::Error> {
484 fn print_dyn_existential(
486 _predicates: &'tcx ty::List<ty::ExistentialPredicate<'tcx>>,
487 ) -> Result<Self::DynExistential, Self::Error> {
494 ) -> Result<Self::Path, Self::Error> {
495 Ok(vec![self.tcx.original_crate_name(cnum).to_string()])
500 _trait_ref: Option<ty::TraitRef<'tcx>>,
501 ) -> Result<Self::Path, Self::Error> {
507 _print_prefix: impl FnOnce(Self) -> Result<Self::Path, Self::Error>,
508 _disambiguated_data: &DisambiguatedDefPathData,
510 _trait_ref: Option<ty::TraitRef<'tcx>>,
511 ) -> Result<Self::Path, Self::Error> {
516 print_prefix: impl FnOnce(Self) -> Result<Self::Path, Self::Error>,
517 disambiguated_data: &DisambiguatedDefPathData,
518 ) -> Result<Self::Path, Self::Error> {
519 let mut path = print_prefix(self)?;
520 path.push(disambiguated_data.data.as_interned_str().to_string());
523 fn path_generic_args(
525 print_prefix: impl FnOnce(Self) -> Result<Self::Path, Self::Error>,
526 _args: &[Kind<'tcx>],
527 ) -> Result<Self::Path, Self::Error> {
532 let report_path_match = |err: &mut DiagnosticBuilder<'_>, did1: DefId, did2: DefId| {
533 // Only external crates, if either is from a local
534 // module we could have false positives
535 if !(did1.is_local() || did2.is_local()) && did1.krate != did2.krate {
536 let abs_path = |def_id| {
537 AbsolutePathPrinter { tcx: self.tcx }
538 .print_def_path(def_id, &[])
541 // We compare strings because DefPath can be different
542 // for imported and non-imported crates
543 let same_path = || -> Result<_, NonTrivialPath> {
545 self.tcx.def_path_str(did1) == self.tcx.def_path_str(did2) ||
546 abs_path(did1)? == abs_path(did2)?
549 if same_path().unwrap_or(false) {
550 let crate_name = self.tcx.crate_name(did1.krate);
554 "Perhaps two different versions \
555 of crate `{}` are being used?",
563 TypeError::Sorts(ref exp_found) => {
564 // if they are both "path types", there's a chance of ambiguity
565 // due to different versions of the same crate
566 if let (&ty::Adt(exp_adt, _), &ty::Adt(found_adt, _))
567 = (&exp_found.expected.sty, &exp_found.found.sty)
569 report_path_match(err, exp_adt.did, found_adt.did);
572 TypeError::Traits(ref exp_found) => {
573 report_path_match(err, exp_found.expected, exp_found.found);
575 _ => (), // FIXME(#22750) handle traits and stuff
579 fn note_error_origin(
581 err: &mut DiagnosticBuilder<'tcx>,
582 cause: &ObligationCause<'tcx>,
583 exp_found: Option<ty::error::ExpectedFound<Ty<'tcx>>>,
586 ObligationCauseCode::MatchExpressionArmPattern { span, ty } => {
587 if ty.is_suggestable() { // don't show type `_`
588 err.span_label(span, format!("this match expression has type `{}`", ty));
590 if let Some(ty::error::ExpectedFound { found, .. }) = exp_found {
591 if ty.is_box() && ty.boxed_ty() == found {
592 if let Ok(snippet) = self.tcx.sess.source_map().span_to_snippet(span) {
595 "consider dereferencing the boxed value",
596 format!("*{}", snippet),
597 Applicability::MachineApplicable,
603 ObligationCauseCode::MatchExpressionArm {
610 hir::MatchSource::IfLetDesugar { .. } => {
611 let msg = "`if let` arms have incompatible types";
612 err.span_label(cause.span, msg);
614 hir::MatchSource::TryDesugar => {
615 if let Some(ty::error::ExpectedFound { expected, .. }) = exp_found {
616 let discrim_expr = self.tcx.hir().expect_expr_by_hir_id(discrim_hir_id);
617 let discrim_ty = if let hir::ExprKind::Call(_, args) = &discrim_expr.node {
618 let arg_expr = args.first().expect("try desugaring call w/out arg");
619 self.in_progress_tables.and_then(|tables| {
620 tables.borrow().expr_ty_opt(arg_expr)
623 bug!("try desugaring w/out call expr as discriminant");
627 Some(ty) if expected == ty => {
628 let source_map = self.tcx.sess.source_map();
630 source_map.end_point(cause.span),
631 "try removing this `?`",
633 Applicability::MachineApplicable,
641 let msg = "`match` arms have incompatible types";
642 err.span_label(cause.span, msg);
643 if prior_arms.len() <= 4 {
644 for sp in prior_arms {
645 err.span_label(*sp, format!(
646 "this is found to be of type `{}`",
650 } else if let Some(sp) = prior_arms.last() {
651 err.span_label(*sp, format!(
652 "this and all prior arms are found to be of type `{}`", last_ty,
657 ObligationCauseCode::IfExpression { then, outer, semicolon } => {
658 err.span_label(then, "expected because of this");
659 outer.map(|sp| err.span_label(sp, "if and else have incompatible types"));
660 if let Some(sp) = semicolon {
661 err.span_suggestion_short(
663 "consider removing this semicolon",
665 Applicability::MachineApplicable,
673 /// Given that `other_ty` is the same as a type argument for `name` in `sub`, populate `value`
674 /// highlighting `name` and every type argument that isn't at `pos` (which is `other_ty`), and
675 /// populate `other_value` with `other_ty`.
679 /// ^^^^--------^ this is highlighted
681 /// | this type argument is exactly the same as the other type, not highlighted
682 /// this is highlighted
684 /// -------- this type is the same as a type argument in the other type, not highlighted
688 value: &mut DiagnosticStyledString,
689 other_value: &mut DiagnosticStyledString,
691 sub: ty::subst::SubstsRef<'tcx>,
695 // `value` and `other_value` hold two incomplete type representation for display.
696 // `name` is the path of both types being compared. `sub`
697 value.push_highlighted(name);
700 value.push_highlighted("<");
703 // Output the lifetimes for the first type
704 let lifetimes = sub.regions()
706 let s = lifetime.to_string();
715 if !lifetimes.is_empty() {
716 if sub.regions().count() < len {
717 value.push_normal(lifetimes + &", ");
719 value.push_normal(lifetimes);
723 // Highlight all the type arguments that aren't at `pos` and compare the type argument at
724 // `pos` and `other_ty`.
725 for (i, type_arg) in sub.types().enumerate() {
727 let values = self.cmp(type_arg, other_ty);
728 value.0.extend((values.0).0);
729 other_value.0.extend((values.1).0);
731 value.push_highlighted(type_arg.to_string());
734 if len > 0 && i != len - 1 {
735 value.push_normal(", ");
737 //self.push_comma(&mut value, &mut other_value, len, i);
740 value.push_highlighted(">");
744 /// If `other_ty` is the same as a type argument present in `sub`, highlight `path` in `t1_out`,
745 /// as that is the difference to the other type.
747 /// For the following code:
750 /// let x: Foo<Bar<Qux>> = foo::<Bar<Qux>>();
753 /// The type error output will behave in the following way:
757 /// ^^^^--------^ this is highlighted
759 /// | this type argument is exactly the same as the other type, not highlighted
760 /// this is highlighted
762 /// -------- this type is the same as a type argument in the other type, not highlighted
766 mut t1_out: &mut DiagnosticStyledString,
767 mut t2_out: &mut DiagnosticStyledString,
769 sub: ty::subst::SubstsRef<'tcx>,
773 for (i, ta) in sub.types().enumerate() {
775 self.highlight_outer(&mut t1_out, &mut t2_out, path, sub, i, &other_ty);
778 if let &ty::Adt(def, _) = &ta.sty {
779 let path_ = self.tcx.def_path_str(def.did.clone());
780 if path_ == other_path {
781 self.highlight_outer(&mut t1_out, &mut t2_out, path, sub, i, &other_ty);
789 /// Adds a `,` to the type representation only if it is appropriate.
792 value: &mut DiagnosticStyledString,
793 other_value: &mut DiagnosticStyledString,
797 if len > 0 && pos != len - 1 {
798 value.push_normal(", ");
799 other_value.push_normal(", ");
803 /// For generic types with parameters with defaults, remove the parameters corresponding to
804 /// the defaults. This repeats a lot of the logic found in `ty::print::pretty`.
805 fn strip_generic_default_params(
808 substs: ty::subst::SubstsRef<'tcx>,
809 ) -> SubstsRef<'tcx> {
810 let generics = self.tcx.generics_of(def_id);
811 let mut num_supplied_defaults = 0;
812 let mut type_params = generics.params.iter().rev().filter_map(|param| match param.kind {
813 ty::GenericParamDefKind::Lifetime => None,
814 ty::GenericParamDefKind::Type { has_default, .. } => Some((param.def_id, has_default)),
815 ty::GenericParamDefKind::Const => None, // FIXME(const_generics:defaults)
818 let has_default = type_params.peek().map(|(_, has_default)| has_default);
819 *has_default.unwrap_or(&false)
822 let types = substs.types().rev();
823 for ((def_id, has_default), actual) in type_params.zip(types) {
827 if self.tcx.type_of(def_id).subst(self.tcx, substs) != actual {
830 num_supplied_defaults += 1;
833 let len = generics.params.len();
834 let mut generics = generics.clone();
835 generics.params.truncate(len - num_supplied_defaults);
836 substs.truncate_to(self.tcx, &generics)
839 /// Compares two given types, eliding parts that are the same between them and highlighting
840 /// relevant differences, and return two representation of those types for highlighted printing.
841 fn cmp(&self, t1: Ty<'tcx>, t2: Ty<'tcx>) -> (DiagnosticStyledString, DiagnosticStyledString) {
842 fn equals<'tcx>(a: &Ty<'tcx>, b: &Ty<'tcx>) -> bool {
843 match (&a.sty, &b.sty) {
844 (a, b) if *a == *b => true,
845 (&ty::Int(_), &ty::Infer(ty::InferTy::IntVar(_)))
846 | (&ty::Infer(ty::InferTy::IntVar(_)), &ty::Int(_))
847 | (&ty::Infer(ty::InferTy::IntVar(_)), &ty::Infer(ty::InferTy::IntVar(_)))
848 | (&ty::Float(_), &ty::Infer(ty::InferTy::FloatVar(_)))
849 | (&ty::Infer(ty::InferTy::FloatVar(_)), &ty::Float(_))
850 | (&ty::Infer(ty::InferTy::FloatVar(_)), &ty::Infer(ty::InferTy::FloatVar(_))) => {
857 fn push_ty_ref<'tcx>(
858 r: &ty::Region<'tcx>,
860 mutbl: hir::Mutability,
861 s: &mut DiagnosticStyledString,
863 let mut r = r.to_string();
869 s.push_highlighted(format!(
872 if mutbl == hir::MutMutable { "mut " } else { "" }
874 s.push_normal(ty.to_string());
877 match (&t1.sty, &t2.sty) {
878 (&ty::Adt(def1, sub1), &ty::Adt(def2, sub2)) => {
879 let sub_no_defaults_1 = self.strip_generic_default_params(def1.did, sub1);
880 let sub_no_defaults_2 = self.strip_generic_default_params(def2.did, sub2);
881 let mut values = (DiagnosticStyledString::new(), DiagnosticStyledString::new());
882 let path1 = self.tcx.def_path_str(def1.did.clone());
883 let path2 = self.tcx.def_path_str(def2.did.clone());
884 if def1.did == def2.did {
885 // Easy case. Replace same types with `_` to shorten the output and highlight
886 // the differing ones.
887 // let x: Foo<Bar, Qux> = y::<Foo<Quz, Qux>>();
890 // --- ^ type argument elided
892 // highlighted in output
893 values.0.push_normal(path1);
894 values.1.push_normal(path2);
896 // Avoid printing out default generic parameters that are common to both
898 let len1 = sub_no_defaults_1.len();
899 let len2 = sub_no_defaults_2.len();
900 let common_len = cmp::min(len1, len2);
901 let remainder1: Vec<_> = sub1.types().skip(common_len).collect();
902 let remainder2: Vec<_> = sub2.types().skip(common_len).collect();
903 let common_default_params = remainder1
906 .zip(remainder2.iter().rev())
907 .filter(|(a, b)| a == b)
909 let len = sub1.len() - common_default_params;
911 // Only draw `<...>` if there're lifetime/type arguments.
913 values.0.push_normal("<");
914 values.1.push_normal("<");
917 fn lifetime_display(lifetime: Region<'_>) -> String {
918 let s = lifetime.to_string();
925 // At one point we'd like to elide all lifetimes here, they are irrelevant for
926 // all diagnostics that use this output
930 // ^^ ^^ --- type arguments are not elided
932 // | elided as they were the same
933 // not elided, they were different, but irrelevant
934 let lifetimes = sub1.regions().zip(sub2.regions());
935 for (i, lifetimes) in lifetimes.enumerate() {
936 let l1 = lifetime_display(lifetimes.0);
937 let l2 = lifetime_display(lifetimes.1);
939 values.0.push_normal("'_");
940 values.1.push_normal("'_");
942 values.0.push_highlighted(l1);
943 values.1.push_highlighted(l2);
945 self.push_comma(&mut values.0, &mut values.1, len, i);
948 // We're comparing two types with the same path, so we compare the type
949 // arguments for both. If they are the same, do not highlight and elide from the
953 // ^ elided type as this type argument was the same in both sides
954 let type_arguments = sub1.types().zip(sub2.types());
955 let regions_len = sub1.regions().count();
956 for (i, (ta1, ta2)) in type_arguments.take(len).enumerate() {
957 let i = i + regions_len;
959 values.0.push_normal("_");
960 values.1.push_normal("_");
962 let (x1, x2) = self.cmp(ta1, ta2);
963 (values.0).0.extend(x1.0);
964 (values.1).0.extend(x2.0);
966 self.push_comma(&mut values.0, &mut values.1, len, i);
969 // Close the type argument bracket.
970 // Only draw `<...>` if there're lifetime/type arguments.
972 values.0.push_normal(">");
973 values.1.push_normal(">");
978 // let x: Foo<Bar<Qux> = foo::<Bar<Qux>>();
980 // ------- this type argument is exactly the same as the other type
982 if self.cmp_type_arg(
994 // let x: Bar<Qux> = y:<Foo<Bar<Qux>>>();
997 // ------- this type argument is exactly the same as the other type
998 if self.cmp_type_arg(
1010 // We couldn't find anything in common, highlight everything.
1011 // let x: Bar<Qux> = y::<Foo<Zar>>();
1013 DiagnosticStyledString::highlighted(t1.to_string()),
1014 DiagnosticStyledString::highlighted(t2.to_string()),
1019 // When finding T != &T, highlight only the borrow
1020 (&ty::Ref(r1, ref_ty1, mutbl1), _) if equals(&ref_ty1, &t2) => {
1021 let mut values = (DiagnosticStyledString::new(), DiagnosticStyledString::new());
1022 push_ty_ref(&r1, ref_ty1, mutbl1, &mut values.0);
1023 values.1.push_normal(t2.to_string());
1026 (_, &ty::Ref(r2, ref_ty2, mutbl2)) if equals(&t1, &ref_ty2) => {
1027 let mut values = (DiagnosticStyledString::new(), DiagnosticStyledString::new());
1028 values.0.push_normal(t1.to_string());
1029 push_ty_ref(&r2, ref_ty2, mutbl2, &mut values.1);
1033 // When encountering &T != &mut T, highlight only the borrow
1034 (&ty::Ref(r1, ref_ty1, mutbl1), &ty::Ref(r2, ref_ty2, mutbl2))
1035 if equals(&ref_ty1, &ref_ty2) =>
1037 let mut values = (DiagnosticStyledString::new(), DiagnosticStyledString::new());
1038 push_ty_ref(&r1, ref_ty1, mutbl1, &mut values.0);
1039 push_ty_ref(&r2, ref_ty2, mutbl2, &mut values.1);
1045 // The two types are the same, elide and don't highlight.
1047 DiagnosticStyledString::normal("_"),
1048 DiagnosticStyledString::normal("_"),
1051 // We couldn't find anything in common, highlight everything.
1053 DiagnosticStyledString::highlighted(t1.to_string()),
1054 DiagnosticStyledString::highlighted(t2.to_string()),
1061 pub fn note_type_err(
1063 diag: &mut DiagnosticBuilder<'tcx>,
1064 cause: &ObligationCause<'tcx>,
1065 secondary_span: Option<(Span, String)>,
1066 mut values: Option<ValuePairs<'tcx>>,
1067 terr: &TypeError<'tcx>,
1069 // For some types of errors, expected-found does not make
1070 // sense, so just ignore the values we were given.
1072 TypeError::CyclicTy(_) => {
1078 let (expected_found, exp_found, is_simple_error) = match values {
1079 None => (None, None, false),
1081 let (is_simple_error, exp_found) = match values {
1082 ValuePairs::Types(exp_found) => {
1084 exp_found.expected.is_primitive() && exp_found.found.is_primitive();
1086 (is_simple_err, Some(exp_found))
1090 let vals = match self.values_str(&values) {
1091 Some((expected, found)) => Some((expected, found)),
1093 // Derived error. Cancel the emitter.
1094 self.tcx.sess.diagnostic().cancel(diag);
1098 (vals, exp_found, is_simple_error)
1102 let span = cause.span(&self.tcx);
1104 diag.span_label(span, terr.to_string());
1105 if let Some((sp, msg)) = secondary_span {
1106 diag.span_label(sp, msg);
1109 if let Some((expected, found)) = expected_found {
1110 match (terr, is_simple_error, expected == found) {
1111 (&TypeError::Sorts(ref values), false, true) => {
1112 diag.note_expected_found_extra(
1116 &format!(" ({})", values.expected.sort_string(self.tcx)),
1117 &format!(" ({})", values.found.sort_string(self.tcx)),
1121 if let Some(exp_found) = exp_found {
1122 let (def_id, ret_ty) = match exp_found.found.sty {
1123 ty::FnDef(def, _) => {
1124 (Some(def), Some(self.tcx.fn_sig(def).output()))
1129 let exp_is_struct = match exp_found.expected.sty {
1130 ty::Adt(def, _) => def.is_struct(),
1134 if let (Some(def_id), Some(ret_ty)) = (def_id, ret_ty) {
1135 if exp_is_struct && &exp_found.expected == ret_ty.skip_binder() {
1136 let message = format!(
1137 "did you mean `{}(/* fields */)`?",
1138 self.tcx.def_path_str(def_id)
1140 diag.span_label(span, message);
1143 self.suggest_as_ref_where_appropriate(span, &exp_found, diag);
1146 diag.note_expected_found(&"type", expected, found);
1152 self.check_and_note_conflicting_crates(diag, terr, span);
1153 self.tcx.note_and_explain_type_err(diag, terr, span);
1155 // It reads better to have the error origin as the final
1157 self.note_error_origin(diag, &cause, exp_found);
1160 /// When encountering a case where `.as_ref()` on a `Result` or `Option` would be appropriate,
1162 fn suggest_as_ref_where_appropriate(
1165 exp_found: &ty::error::ExpectedFound<Ty<'tcx>>,
1166 diag: &mut DiagnosticBuilder<'tcx>,
1168 match (&exp_found.expected.sty, &exp_found.found.sty) {
1169 (ty::Adt(exp_def, exp_substs), ty::Ref(_, found_ty, _)) => {
1170 if let ty::Adt(found_def, found_substs) = found_ty.sty {
1171 let path_str = format!("{:?}", exp_def);
1172 if exp_def == &found_def {
1173 let opt_msg = "you can convert from `&Option<T>` to `Option<&T>` using \
1175 let result_msg = "you can convert from `&Result<T, E>` to \
1176 `Result<&T, &E>` using `.as_ref()`";
1177 let have_as_ref = &[
1178 ("std::option::Option", opt_msg),
1179 ("core::option::Option", opt_msg),
1180 ("std::result::Result", result_msg),
1181 ("core::result::Result", result_msg),
1183 if let Some(msg) = have_as_ref.iter()
1184 .filter_map(|(path, msg)| if &path_str == path {
1190 let mut show_suggestion = true;
1191 for (exp_ty, found_ty) in exp_substs.types().zip(found_substs.types()) {
1193 ty::Ref(_, exp_ty, _) => {
1194 match (&exp_ty.sty, &found_ty.sty) {
1198 (ty::Infer(_), _) => {}
1199 _ if ty::TyS::same_type(exp_ty, found_ty) => {}
1200 _ => show_suggestion = false,
1203 ty::Param(_) | ty::Infer(_) => {}
1204 _ => show_suggestion = false,
1207 if let (Ok(snippet), true) = (
1208 self.tcx.sess.source_map().span_to_snippet(span),
1211 diag.span_suggestion(
1214 format!("{}.as_ref()", snippet),
1215 Applicability::MachineApplicable,
1226 pub fn report_and_explain_type_error(
1228 trace: TypeTrace<'tcx>,
1229 terr: &TypeError<'tcx>,
1230 ) -> DiagnosticBuilder<'tcx> {
1232 "report_and_explain_type_error(trace={:?}, terr={:?})",
1236 let span = trace.cause.span(&self.tcx);
1237 let failure_code = trace.cause.as_failure_code(terr);
1238 let mut diag = match failure_code {
1239 FailureCode::Error0317(failure_str) => {
1240 struct_span_err!(self.tcx.sess, span, E0317, "{}", failure_str)
1242 FailureCode::Error0580(failure_str) => {
1243 struct_span_err!(self.tcx.sess, span, E0580, "{}", failure_str)
1245 FailureCode::Error0308(failure_str) => {
1246 struct_span_err!(self.tcx.sess, span, E0308, "{}", failure_str)
1248 FailureCode::Error0644(failure_str) => {
1249 struct_span_err!(self.tcx.sess, span, E0644, "{}", failure_str)
1252 self.note_type_err(&mut diag, &trace.cause, None, Some(trace.values), terr);
1258 values: &ValuePairs<'tcx>,
1259 ) -> Option<(DiagnosticStyledString, DiagnosticStyledString)> {
1261 infer::Types(ref exp_found) => self.expected_found_str_ty(exp_found),
1262 infer::Regions(ref exp_found) => self.expected_found_str(exp_found),
1263 infer::TraitRefs(ref exp_found) => self.expected_found_str(exp_found),
1264 infer::PolyTraitRefs(ref exp_found) => self.expected_found_str(exp_found),
1268 fn expected_found_str_ty(
1270 exp_found: &ty::error::ExpectedFound<Ty<'tcx>>,
1271 ) -> Option<(DiagnosticStyledString, DiagnosticStyledString)> {
1272 let exp_found = self.resolve_type_vars_if_possible(exp_found);
1273 if exp_found.references_error() {
1277 Some(self.cmp(exp_found.expected, exp_found.found))
1280 /// Returns a string of the form "expected `{}`, found `{}`".
1281 fn expected_found_str<T: fmt::Display + TypeFoldable<'tcx>>(
1283 exp_found: &ty::error::ExpectedFound<T>,
1284 ) -> Option<(DiagnosticStyledString, DiagnosticStyledString)> {
1285 let exp_found = self.resolve_type_vars_if_possible(exp_found);
1286 if exp_found.references_error() {
1291 DiagnosticStyledString::highlighted(exp_found.expected.to_string()),
1292 DiagnosticStyledString::highlighted(exp_found.found.to_string()),
1296 pub fn report_generic_bound_failure(
1298 region_scope_tree: ®ion::ScopeTree,
1300 origin: Option<SubregionOrigin<'tcx>>,
1301 bound_kind: GenericKind<'tcx>,
1304 self.construct_generic_bound_failure(region_scope_tree, span, origin, bound_kind, sub)
1308 pub fn construct_generic_bound_failure(
1310 region_scope_tree: ®ion::ScopeTree,
1312 origin: Option<SubregionOrigin<'tcx>>,
1313 bound_kind: GenericKind<'tcx>,
1315 ) -> DiagnosticBuilder<'a> {
1316 // Attempt to obtain the span of the parameter so we can
1317 // suggest adding an explicit lifetime bound to it.
1318 let type_param_span = match (self.in_progress_tables, bound_kind) {
1319 (Some(ref table), GenericKind::Param(ref param)) => {
1320 let table = table.borrow();
1321 table.local_id_root.and_then(|did| {
1322 let generics = self.tcx.generics_of(did);
1323 // Account for the case where `did` corresponds to `Self`, which doesn't have
1324 // the expected type argument.
1325 if !param.is_self() {
1326 let type_param = generics.type_param(param, self.tcx);
1327 let hir = &self.tcx.hir();
1328 hir.as_local_node_id(type_param.def_id).map(|id| {
1329 // Get the `hir::Param` to verify whether it already has any bounds.
1330 // We do this to avoid suggesting code that ends up as `T: 'a'b`,
1331 // instead we suggest `T: 'a + 'b` in that case.
1332 let mut has_bounds = false;
1333 if let Node::GenericParam(ref param) = hir.get(id) {
1334 has_bounds = !param.bounds.is_empty();
1336 let sp = hir.span(id);
1337 // `sp` only covers `T`, change it so that it covers
1338 // `T:` when appropriate
1339 let is_impl_trait = bound_kind.to_string().starts_with("impl ");
1340 let sp = if has_bounds && !is_impl_trait {
1344 .next_point(self.tcx.sess.source_map().next_point(sp)))
1348 (sp, has_bounds, is_impl_trait)
1358 let labeled_user_string = match bound_kind {
1359 GenericKind::Param(ref p) => format!("the parameter type `{}`", p),
1360 GenericKind::Projection(ref p) => format!("the associated type `{}`", p),
1363 if let Some(SubregionOrigin::CompareImplMethodObligation {
1370 return self.report_extra_impl_obligation(
1375 &format!("`{}: {}`", bound_kind, sub),
1379 fn binding_suggestion<'tcx, S: fmt::Display>(
1380 err: &mut DiagnosticBuilder<'tcx>,
1381 type_param_span: Option<(Span, bool, bool)>,
1382 bound_kind: GenericKind<'tcx>,
1385 let consider = format!(
1386 "consider adding an explicit lifetime bound {}",
1387 if type_param_span.map(|(_, _, is_impl_trait)| is_impl_trait).unwrap_or(false) {
1388 format!(" `{}` to `{}`...", sub, bound_kind)
1390 format!("`{}: {}`...", bound_kind, sub)
1393 if let Some((sp, has_lifetimes, is_impl_trait)) = type_param_span {
1394 let suggestion = if is_impl_trait {
1395 format!("{} + {}", bound_kind, sub)
1397 let tail = if has_lifetimes { " + " } else { "" };
1398 format!("{}: {}{}", bound_kind, sub, tail)
1400 err.span_suggestion_short(
1404 Applicability::MaybeIncorrect, // Issue #41966
1407 err.help(&consider);
1411 let mut err = match *sub {
1413 | ty::ReFree(ty::FreeRegion {
1414 bound_region: ty::BrNamed(..),
1417 // Does the required lifetime have a nice name we can print?
1418 let mut err = struct_span_err!(
1422 "{} may not live long enough",
1425 binding_suggestion(&mut err, type_param_span, bound_kind, sub);
1430 // Does the required lifetime have a nice name we can print?
1431 let mut err = struct_span_err!(
1435 "{} may not live long enough",
1438 binding_suggestion(&mut err, type_param_span, bound_kind, "'static");
1443 // If not, be less specific.
1444 let mut err = struct_span_err!(
1448 "{} may not live long enough",
1452 "consider adding an explicit lifetime bound for `{}`",
1455 self.tcx.note_and_explain_region(
1458 &format!("{} must be valid for ", labeled_user_string),
1466 if let Some(origin) = origin {
1467 self.note_region_origin(&mut err, &origin);
1472 fn report_sub_sup_conflict(
1474 region_scope_tree: ®ion::ScopeTree,
1475 var_origin: RegionVariableOrigin,
1476 sub_origin: SubregionOrigin<'tcx>,
1477 sub_region: Region<'tcx>,
1478 sup_origin: SubregionOrigin<'tcx>,
1479 sup_region: Region<'tcx>,
1481 let mut err = self.report_inference_failure(var_origin);
1483 self.tcx.note_and_explain_region(
1486 "first, the lifetime cannot outlive ",
1491 match (&sup_origin, &sub_origin) {
1492 (&infer::Subtype(ref sup_trace), &infer::Subtype(ref sub_trace)) => {
1493 debug!("report_sub_sup_conflict: var_origin={:?}", var_origin);
1494 debug!("report_sub_sup_conflict: sub_region={:?}", sub_region);
1495 debug!("report_sub_sup_conflict: sub_origin={:?}", sub_origin);
1496 debug!("report_sub_sup_conflict: sup_region={:?}", sup_region);
1497 debug!("report_sub_sup_conflict: sup_origin={:?}", sup_origin);
1498 debug!("report_sub_sup_conflict: sup_trace={:?}", sup_trace);
1499 debug!("report_sub_sup_conflict: sub_trace={:?}", sub_trace);
1500 debug!("report_sub_sup_conflict: sup_trace.values={:?}", sup_trace.values);
1501 debug!("report_sub_sup_conflict: sub_trace.values={:?}", sub_trace.values);
1503 if let (Some((sup_expected, sup_found)), Some((sub_expected, sub_found))) = (
1504 self.values_str(&sup_trace.values),
1505 self.values_str(&sub_trace.values),
1507 if sub_expected == sup_expected && sub_found == sup_found {
1508 self.tcx.note_and_explain_region(
1511 "...but the lifetime must also be valid for ",
1516 "...so that the {}:\nexpected {}\n found {}",
1517 sup_trace.cause.as_requirement_str(),
1518 sup_expected.content(),
1529 self.note_region_origin(&mut err, &sup_origin);
1531 self.tcx.note_and_explain_region(
1534 "but, the lifetime must be valid for ",
1539 self.note_region_origin(&mut err, &sub_origin);
1544 impl<'a, 'gcx, 'tcx> InferCtxt<'a, 'gcx, 'tcx> {
1545 fn report_inference_failure(
1547 var_origin: RegionVariableOrigin,
1548 ) -> DiagnosticBuilder<'tcx> {
1549 let br_string = |br: ty::BoundRegion| {
1550 let mut s = match br {
1551 ty::BrNamed(_, name) => name.to_string(),
1559 let var_description = match var_origin {
1560 infer::MiscVariable(_) => String::new(),
1561 infer::PatternRegion(_) => " for pattern".to_string(),
1562 infer::AddrOfRegion(_) => " for borrow expression".to_string(),
1563 infer::Autoref(_) => " for autoref".to_string(),
1564 infer::Coercion(_) => " for automatic coercion".to_string(),
1565 infer::LateBoundRegion(_, br, infer::FnCall) => {
1566 format!(" for lifetime parameter {}in function call", br_string(br))
1568 infer::LateBoundRegion(_, br, infer::HigherRankedType) => {
1569 format!(" for lifetime parameter {}in generic type", br_string(br))
1571 infer::LateBoundRegion(_, br, infer::AssocTypeProjection(def_id)) => format!(
1572 " for lifetime parameter {}in trait containing associated type `{}`",
1574 self.tcx.associated_item(def_id).ident
1576 infer::EarlyBoundRegion(_, name) => format!(" for lifetime parameter `{}`", name),
1577 infer::BoundRegionInCoherence(name) => {
1578 format!(" for lifetime parameter `{}` in coherence check", name)
1580 infer::UpvarRegion(ref upvar_id, _) => {
1581 let var_name = self.tcx.hir().name_by_hir_id(upvar_id.var_path.hir_id);
1582 format!(" for capture of `{}` by closure", var_name)
1584 infer::NLL(..) => bug!("NLL variable found in lexical phase"),
1591 "cannot infer an appropriate lifetime{} \
1592 due to conflicting requirements",
1599 Error0317(&'static str),
1600 Error0580(&'static str),
1601 Error0308(&'static str),
1602 Error0644(&'static str),
1605 impl<'tcx> ObligationCause<'tcx> {
1606 fn as_failure_code(&self, terr: &TypeError<'tcx>) -> FailureCode {
1607 use self::FailureCode::*;
1608 use crate::traits::ObligationCauseCode::*;
1610 CompareImplMethodObligation { .. } => Error0308("method not compatible with trait"),
1611 MatchExpressionArm { source, .. } => Error0308(match source {
1612 hir::MatchSource::IfLetDesugar { .. } => "`if let` arms have incompatible types",
1613 hir::MatchSource::TryDesugar => {
1614 "try expression alternatives have incompatible types"
1616 _ => "match arms have incompatible types",
1618 IfExpression { .. } => Error0308("if and else have incompatible types"),
1619 IfExpressionWithNoElse => Error0317("if may be missing an else clause"),
1620 MainFunctionType => Error0580("main function has wrong type"),
1621 StartFunctionType => Error0308("start function has wrong type"),
1622 IntrinsicType => Error0308("intrinsic has wrong type"),
1623 MethodReceiver => Error0308("mismatched method receiver"),
1625 // In the case where we have no more specific thing to
1626 // say, also take a look at the error code, maybe we can
1629 TypeError::CyclicTy(ty) if ty.is_closure() || ty.is_generator() => {
1630 Error0644("closure/generator type that references itself")
1632 _ => Error0308("mismatched types"),
1637 fn as_requirement_str(&self) -> &'static str {
1638 use crate::traits::ObligationCauseCode::*;
1640 CompareImplMethodObligation { .. } => "method type is compatible with trait",
1641 ExprAssignable => "expression is assignable",
1642 MatchExpressionArm { source, .. } => match source {
1643 hir::MatchSource::IfLetDesugar { .. } => "`if let` arms have compatible types",
1644 _ => "match arms have compatible types",
1646 IfExpression { .. } => "if and else have compatible types",
1647 IfExpressionWithNoElse => "if missing an else returns ()",
1648 MainFunctionType => "`main` function has the correct type",
1649 StartFunctionType => "`start` function has the correct type",
1650 IntrinsicType => "intrinsic has the correct type",
1651 MethodReceiver => "method receiver has the correct type",
1652 _ => "types are compatible",