Skip to content

Commit 4549ef5

Browse files
committed
emit err when using trait objects in pat
1 parent fee1598 commit 4549ef5

File tree

4 files changed

+33
-17
lines changed

4 files changed

+33
-17
lines changed

src/librustc_mir_build/hair/pattern/const_to_pat.rs

+14-16
Original file line numberDiff line numberDiff line change
@@ -101,23 +101,21 @@ impl<'a, 'tcx> ConstToPat<'a, 'tcx> {
101101
cv.ty, structural
102102
);
103103
if let Some(non_sm_ty) = structural {
104-
let adt_def = match non_sm_ty {
105-
traits::NonStructuralMatchTy::Adt(adt_def) => adt_def,
106-
traits::NonStructuralMatchTy::Param => {
107-
bug!("use of constant whose type is a parameter inside a pattern")
104+
let msg = match non_sm_ty {
105+
traits::NonStructuralMatchTy::Adt(adt_def) => {
106+
let path = self.tcx().def_path_str(adt_def.did);
107+
format!(
108+
"to use a constant of type `{}` in a pattern, \
109+
`{}` must be annotated with `#[derive(PartialEq, Eq)]`",
110+
path, path,
111+
)
108112
}
109113
traits::NonStructuralMatchTy::Dynamic => {
110-
bug!("use of a trait object inside a pattern")
114+
format!("trait objects cannot be used in patterns")
115+
}
116+
traits::NonStructuralMatchTy::Param => {
117+
bug!("use of constant whose type is a parameter inside a pattern")
111118
}
112-
};
113-
let path = self.tcx().def_path_str(adt_def.did);
114-
115-
let make_msg = || -> String {
116-
format!(
117-
"to use a constant of type `{}` in a pattern, \
118-
`{}` must be annotated with `#[derive(PartialEq, Eq)]`",
119-
path, path,
120-
)
121119
};
122120

123121
// double-check there even *is* a semantic `PartialEq` to dispatch to.
@@ -148,13 +146,13 @@ impl<'a, 'tcx> ConstToPat<'a, 'tcx> {
148146

149147
if !ty_is_partial_eq {
150148
// span_fatal avoids ICE from resolution of non-existent method (rare case).
151-
self.tcx().sess.span_fatal(self.span, &make_msg());
149+
self.tcx().sess.span_fatal(self.span, &msg);
152150
} else {
153151
self.tcx().struct_span_lint_hir(
154152
lint::builtin::INDIRECT_STRUCTURAL_MATCH,
155153
self.id,
156154
self.span,
157-
|lint| lint.build(&make_msg()).emit(),
155+
|lint| lint.build(&msg).emit(),
158156
);
159157
}
160158
}

src/test/ui/const-generics/issues/issue-63322-forbid-dyn.stderr

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,6 @@ error[E0741]: the types of const generic parameters must derive `PartialEq` and
1212
LL | fn test<const T: &'static dyn A>() {
1313
| ^^^^^^^^^^^^^^ `&'static (dyn A + 'static)` doesn't derive both `PartialEq` and `Eq`
1414

15-
error: aborting due to previous error
15+
error: aborting due to previous error; 1 warning emitted
1616

1717
For more information about this error, try `rustc --explain E0741`.
+10
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
const F: &'static dyn Send = &7u32;
2+
3+
fn main() {
4+
let a: &dyn Send = &7u32;
5+
match a {
6+
F => panic!(),
7+
//~^ ERROR trait objects cannot be used in patterns
8+
_ => {}
9+
}
10+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
error: trait objects cannot be used in patterns
2+
--> $DIR/issue-70972-dyn-trait.rs:6:9
3+
|
4+
LL | F => panic!(),
5+
| ^
6+
7+
error: aborting due to previous error
8+

0 commit comments

Comments
 (0)