mathieu/granite-rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Avoid cat_expr note when already in error state.

Browse source 
Also, to ensure we do not let the dropck get skipped, ICE if cat_expr
errors when *not* in error state.

This is not known to be a breaking change (i.e. I do not know of a
current case that causes the new ICE to be exercised).
This commit is contained in:
Felix S. Klock II 2015-02-24 15:22:37 +01:00
parent dccdde4007
commit 92bc3ea467
1 changed files with 12 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/librustc_typeck/check/regionck.rs
View file

@ -554,8 +554,12 @@ fn visit_expr(rcx: &mut Rcx, expr: &ast::Expr) {
expr.span); expr.span);
} }
Err(..) => { Err(..) => {
rcx.fcx.tcx().sess.span_note(expr.span, let tcx = rcx.fcx.tcx();
"cat_expr_unadjusted Errd during dtor check"); if tcx.sess.has_errors() {
// cannot run dropck; okay b/c in error state anyway.
} else {
tcx.sess.span_bug(expr.span, "cat_expr_unadjusted Errd");
}
} }
} }
} }
@ -571,8 +575,12 @@ fn visit_expr(rcx: &mut Rcx, expr: &ast::Expr) {
check_safety_of_rvalue_destructor_if_necessary(rcx, head_cmt, expr.span); check_safety_of_rvalue_destructor_if_necessary(rcx, head_cmt, expr.span);
} }
Err(..) => { Err(..) => {
rcx.fcx.tcx().sess.span_note(expr.span, let tcx = rcx.fcx.tcx();
"cat_expr Errd during dtor check"); if tcx.sess.has_errors() {
// cannot run dropck; okay b/c in error state anyway.
} else {
tcx.sess.span_bug(expr.span, "cat_expr Errd");
}
} }
} }