Skip to content
This repository was archived by the owner on May 28, 2025. It is now read-only.

Commit 757b726

Browse files
estebankestebank
committed
Use only one label for multiple unsatisfied bounds on type (typeck)
1 parent 3691ab8 commit 757b726

16 files changed

+56
-105
lines changed

‎compiler/rustc_hir_typeck/src/method/suggest.rs‎

Lines changed: 35 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ use crate::Expectation;
99
use crate::FnCtxt;
1010
use rustc_ast::ast::Mutability;
1111
use rustc_attr::parse_confusables;
12-
use rustc_data_structures::fx::{FxIndexMap, FxIndexSet};
12+
use rustc_data_structures::fx::{FxHashMap, FxIndexMap, FxIndexSet};
1313
use rustc_data_structures::unord::UnordSet;
1414
use rustc_errors::StashKey;
1515
use rustc_errors::{
@@ -547,7 +547,7 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
547547
);
548548
}
549549

550-
let mut bound_spans = vec![];
550+
let mut bound_spans: FxHashMap<Span, Vec<String>> = Default::default();
551551
let mut restrict_type_params = false;
552552
let mut unsatisfied_bounds = false;
553553
if item_name.name == sym::count && self.is_slice_ty(rcvr_ty, span) {
@@ -642,28 +642,34 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
642642
false
643643
};
644644
let mut bound_span_label = |self_ty: Ty<'_>, obligation: &str, quiet: &str| {
645-
let msg = format!(
646-
"doesn't satisfy `{}`",
647-
if obligation.len() > 50 { quiet } else { obligation }
648-
);
645+
let msg = format!("`{}`", if obligation.len() > 50 { quiet } else { obligation });
649646
match &self_ty.kind() {
650647
// Point at the type that couldn't satisfy the bound.
651-
ty::Adt(def, _) => bound_spans.push((self.tcx.def_span(def.did()), msg)),
648+
ty::Adt(def, _) => {
649+
bound_spans.entry(tcx.def_span(def.did())).or_default().push(msg)
650+
}
652651
// Point at the trait object that couldn't satisfy the bound.
653652
ty::Dynamic(preds, _, _) => {
654653
for pred in preds.iter() {
655654
match pred.skip_binder() {
656655
ty::ExistentialPredicate::Trait(tr) => {
657-
bound_spans.push((self.tcx.def_span(tr.def_id), msg.clone()))
656+
bound_spans
657+
.entry(tcx.def_span(tr.def_id))
658+
.or_default()
659+
.push(msg.clone());
658660
}
659661
ty::ExistentialPredicate::Projection(_)
660662
| ty::ExistentialPredicate::AutoTrait(_) => {}
661663
}
662664
}
663665
}
664666
// Point at the closure that couldn't satisfy the bound.
665-
ty::Closure(def_id, _) => bound_spans
666-
.push((tcx.def_span(*def_id), format!("doesn't satisfy `{quiet}`"))),
667+
ty::Closure(def_id, _) => {
668+
bound_spans
669+
.entry(tcx.def_span(*def_id))
670+
.or_default()
671+
.push(format!("`{quiet}`"));
672+
}
667673
_ => {}
668674
}
669675
};
@@ -1170,9 +1176,25 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
11701176

11711177
self.suggest_unwrapping_inner_self(&mut err, source, rcvr_ty, item_name);
11721178

1173-
bound_spans.sort();
1174-
bound_spans.dedup();
1175-
for (span, msg) in bound_spans.into_iter() {
1179+
#[allow(rustc::potential_query_instability)] // We immediately sort the resulting Vec.
1180+
let mut bound_spans: Vec<(Span, Vec<String>)> = bound_spans
1181+
.into_iter()
1182+
.map(|(span, mut bounds)| {
1183+
bounds.sort();
1184+
bounds.dedup();
1185+
(span, bounds)
1186+
})
1187+
.collect();
1188+
bound_spans.sort_by_key(|(span, _)| *span);
1189+
for (span, bounds) in bound_spans {
1190+
if !tcx.sess.source_map().is_span_accessible(span) {
1191+
continue;
1192+
}
1193+
let msg = match &bounds[..] {
1194+
[bound] => format!("doesn't satisfy {bound}"),
1195+
[bounds @ .., last] => format!("doesn't satisfy {} or {last}", bounds.join(", ")),
1196+
[] => unreachable!(),
1197+
};
11761198
err.span_label(span, msg);
11771199
}
11781200

‎tests/ui/box/unit/unique-object-noncopyable.stderr‎

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,11 @@
11
error[E0599]: the method `clone` exists for struct `Box<dyn Foo>`, but its trait bounds were not satisfied
22
--> $DIR/unique-object-noncopyable.rs:24:16
33
|
4-
LL | trait Foo {
5-
| ---------
6-
| |
7-
| doesn't satisfy `dyn Foo: Clone`
8-
| doesn't satisfy `dyn Foo: Sized`
4+
LL | trait Foo {
5+
| --------- doesn't satisfy `dyn Foo: Clone` or `dyn Foo: Sized`
96
...
10-
LL | let _z = y.clone();
11-
| ^^^^^ method cannot be called on `Box<dyn Foo>` due to unsatisfied trait bounds
12-
--> $SRC_DIR/alloc/src/boxed.rs:LL:COL
13-
::: $SRC_DIR/alloc/src/boxed.rs:LL:COL
14-
|
15-
= note: doesn't satisfy `Box<dyn Foo>: Clone`
7+
LL | let _z = y.clone();
8+
| ^^^^^ method cannot be called on `Box<dyn Foo>` due to unsatisfied trait bounds
169
|
1710
= note: the following trait bounds were not satisfied:
1811
`dyn Foo: Sized`

‎tests/ui/box/unit/unique-pinned-nocopy.stderr‎

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,11 @@
11
error[E0599]: the method `clone` exists for struct `Box<R>`, but its trait bounds were not satisfied
22
--> $DIR/unique-pinned-nocopy.rs:12:16
33
|
4-
LL | struct R {
5-
| -------- doesn't satisfy `R: Clone`
4+
LL | struct R {
5+
| -------- doesn't satisfy `R: Clone`
66
...
7-
LL | let _j = i.clone();
8-
| ^^^^^ method cannot be called on `Box<R>` due to unsatisfied trait bounds
9-
--> $SRC_DIR/alloc/src/boxed.rs:LL:COL
10-
::: $SRC_DIR/alloc/src/boxed.rs:LL:COL
11-
|
12-
= note: doesn't satisfy `Box<R>: Clone`
7+
LL | let _j = i.clone();
8+
| ^^^^^ method cannot be called on `Box<R>` due to unsatisfied trait bounds
139
|
1410
= note: the following trait bounds were not satisfied:
1511
`R: Clone`

‎tests/ui/derives/deriving-with-repr-packed-2.stderr‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,7 @@ LL | pub struct Foo<T>(T, T, T);
88
| doesn't satisfy `Foo<NonCopy>: Clone`
99
LL |
1010
LL | struct NonCopy;
11-
| --------------
12-
| |
13-
| doesn't satisfy `NonCopy: Clone`
14-
| doesn't satisfy `NonCopy: Copy`
11+
| -------------- doesn't satisfy `NonCopy: Clone` or `NonCopy: Copy`
1512
...
1613
LL | _ = x.clone();
1714
| ^^^^^ method cannot be called on `Foo<NonCopy>` due to unsatisfied trait bounds

‎tests/ui/derives/issue-91550.stderr‎

Lines changed: 4 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,7 @@ error[E0599]: the method `insert` exists for struct `HashSet<Value>`, but its tr
22
--> $DIR/issue-91550.rs:8:8
33
|
44
LL | struct Value(u32);
5-
| ------------
6-
| |
7-
| doesn't satisfy `Value: Eq`
8-
| doesn't satisfy `Value: Hash`
9-
| doesn't satisfy `Value: PartialEq`
5+
| ------------ doesn't satisfy `Value: Eq`, `Value: Hash` or `Value: PartialEq`
106
...
117
LL | hs.insert(Value(0));
128
| ^^^^^^
@@ -26,10 +22,7 @@ error[E0599]: the method `use_eq` exists for struct `Object<NoDerives>`, but its
2622
--> $DIR/issue-91550.rs:26:9
2723
|
2824
LL | pub struct NoDerives;
29-
| --------------------
30-
| |
31-
| doesn't satisfy `NoDerives: Eq`
32-
| doesn't satisfy `NoDerives: PartialEq`
25+
| -------------------- doesn't satisfy `NoDerives: Eq` or `NoDerives: PartialEq`
3326
LL |
3427
LL | struct Object<T>(T);
3528
| ---------------- method `use_eq` not found for this struct
@@ -57,12 +50,7 @@ error[E0599]: the method `use_ord` exists for struct `Object<NoDerives>`, but it
5750
--> $DIR/issue-91550.rs:27:9
5851
|
5952
LL | pub struct NoDerives;
60-
| --------------------
61-
| |
62-
| doesn't satisfy `NoDerives: Eq`
63-
| doesn't satisfy `NoDerives: Ord`
64-
| doesn't satisfy `NoDerives: PartialEq`
65-
| doesn't satisfy `NoDerives: PartialOrd`
53+
| -------------------- doesn't satisfy `NoDerives: Eq`, `NoDerives: Ord`, `NoDerives: PartialEq` or `NoDerives: PartialOrd`
6654
LL |
6755
LL | struct Object<T>(T);
6856
| ---------------- method `use_ord` not found for this struct
@@ -94,12 +82,7 @@ error[E0599]: the method `use_ord_and_partial_ord` exists for struct `Object<NoD
9482
--> $DIR/issue-91550.rs:28:9
9583
|
9684
LL | pub struct NoDerives;
97-
| --------------------
98-
| |
99-
| doesn't satisfy `NoDerives: Eq`
100-
| doesn't satisfy `NoDerives: Ord`
101-
| doesn't satisfy `NoDerives: PartialEq`
102-
| doesn't satisfy `NoDerives: PartialOrd`
85+
| -------------------- doesn't satisfy `NoDerives: Eq`, `NoDerives: Ord`, `NoDerives: PartialEq` or `NoDerives: PartialOrd`
10386
LL |
10487
LL | struct Object<T>(T);
10588
| ---------------- method `use_ord_and_partial_ord` not found for this struct

‎tests/ui/generic-associated-types/issue-119942-unsatisified-gat-bound-during-assoc-ty-selection.stderr‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,6 @@ LL | enum Node<T, P: PointerFamily> {
2626
...
2727
LL | let mut list = RcNode::<i32>::new();
2828
| ^^^ doesn't have a size known at compile-time
29-
--> $SRC_DIR/core/src/ops/deref.rs:LL:COL
30-
|
31-
= note: doesn't satisfy `_: Sized`
3229
|
3330
note: trait bound `Node<i32, RcFamily>: Sized` was not satisfied
3431
--> $DIR/issue-119942-unsatisified-gat-bound-during-assoc-ty-selection.rs:4:18

‎tests/ui/generic-associated-types/method-unsatisfied-assoc-type-predicate.rs‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,7 @@ impl<T: X<Y<i32> = i32>> M for T {}
1717

1818
struct S;
1919
//~^ NOTE method `f` not found for this
20-
//~| NOTE doesn't satisfy `<S as X>::Y<i32> = i32`
21-
//~| NOTE doesn't satisfy `S: M`
20+
//~| NOTE doesn't satisfy `<S as X>::Y<i32> = i32` or `S: M`
2221

2322
impl X for S {
2423
type Y<T> = bool;

‎tests/ui/generic-associated-types/method-unsatisfied-assoc-type-predicate.stderr‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,11 @@
11
error[E0599]: the method `f` exists for struct `S`, but its trait bounds were not satisfied
2-
--> $DIR/method-unsatisfied-assoc-type-predicate.rs:28:7
2+
--> $DIR/method-unsatisfied-assoc-type-predicate.rs:27:7
33
|
44
LL | struct S;
55
| --------
66
| |
77
| method `f` not found for this struct
8-
| doesn't satisfy `<S as X>::Y<i32> = i32`
9-
| doesn't satisfy `S: M`
8+
| doesn't satisfy `<S as X>::Y<i32> = i32` or `S: M`
109
...
1110
LL | a.f();
1211
| ^ method cannot be called on `S` due to unsatisfied trait bounds

‎tests/ui/iterators/vec-on-unimplemented.stderr‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,6 @@ error[E0599]: `Vec<bool>` is not an iterator
33
|
44
LL | vec![true, false].map(|v| !v).collect::<Vec<_>>();
55
| ^^^ `Vec<bool>` is not an iterator; try calling `.into_iter()` or `.iter()`
6-
--> $SRC_DIR/alloc/src/vec/mod.rs:LL:COL
7-
|
8-
= note: doesn't satisfy `Vec<bool>: Iterator`
96
|
107
= note: the following trait bounds were not satisfied:
118
`Vec<bool>: Iterator`

‎tests/ui/mismatched_types/issue-36053-2.stderr‎

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,7 @@ error[E0599]: the method `count` exists for struct `Filter<Fuse<Once<&str>>, {cl
2121
LL | once::<&str>("str").fuse().filter(|a: &str| true).count();
2222
| --------- ^^^^^ method cannot be called due to unsatisfied trait bounds
2323
| |
24-
| doesn't satisfy `<_ as FnOnce<(&&str,)>>::Output = bool`
25-
| doesn't satisfy `_: FnMut<(&&str,)>`
26-
--> $SRC_DIR/core/src/iter/adapters/filter.rs:LL:COL
27-
|
28-
= note: doesn't satisfy `_: Iterator`
24+
| doesn't satisfy `<_ as FnOnce<(&&str,)>>::Output = bool` or `_: FnMut<(&&str,)>`
2925
|
3026
= note: the following trait bounds were not satisfied:
3127
`<{closure@$DIR/issue-36053-2.rs:7:39: 7:48} as FnOnce<(&&str,)>>::Output = bool`

0 commit comments

Comments
 (0)