diff options
| author | Ariel Ben-Yehuda <ariel.byd@gmail.com> | 2017-08-21 14:26:33 +0300 |
|---|---|---|
| committer | Ariel Ben-Yehuda <ariel.byd@gmail.com> | 2017-08-21 14:26:33 +0300 |
| commit | b47bcc2ee86debf37ff61b14d9b57361c7dd0ab8 (patch) | |
| tree | 735bd9aa6707238ecd58108f2f49c081aaf669b1 | |
| parent | e00263438c95e1e31eb76548b1e08b81e5f50a4a (diff) | |
| download | rust-b47bcc2ee86debf37ff61b14d9b57361c7dd0ab8.tar.gz rust-b47bcc2ee86debf37ff61b14d9b57361c7dd0ab8.zip | |
fix other cases of registering obligations in a snapshot
No test cases for these ones, but they would all ICE if they ever run with a non-empty set of obligations.
| -rw-r--r-- | src/librustc_typeck/check/coercion.rs | 6 | ||||
| -rw-r--r-- | src/librustc_typeck/check/regionck.rs | 10 |
2 files changed, 7 insertions, 9 deletions
diff --git a/src/librustc_typeck/check/coercion.rs b/src/librustc_typeck/check/coercion.rs index 0ec30fb26d6..e406ce845a6 100644 --- a/src/librustc_typeck/check/coercion.rs +++ b/src/librustc_typeck/check/coercion.rs @@ -883,8 +883,7 @@ impl<'a, 'gcx, 'tcx> FnCtxt<'a, 'gcx, 'tcx> { return self.commit_if_ok(|_| { self.at(cause, self.param_env) .lub(prev_ty, new_ty) - .map(|ok| self.register_infer_ok_obligations(ok)) - }); + }).map(|ok| self.register_infer_ok_obligations(ok)); } } @@ -897,8 +896,7 @@ impl<'a, 'gcx, 'tcx> FnCtxt<'a, 'gcx, 'tcx> { self.commit_if_ok(|_| { self.at(cause, self.param_env) .lub(prev_ty, new_ty) - .map(|ok| self.register_infer_ok_obligations(ok)) - }) + }).map(|ok| self.register_infer_ok_obligations(ok)) } } Ok(ok) => { diff --git a/src/librustc_typeck/check/regionck.rs b/src/librustc_typeck/check/regionck.rs index 73c243d27d1..1a61b8e29f2 100644 --- a/src/librustc_typeck/check/regionck.rs +++ b/src/librustc_typeck/check/regionck.rs @@ -1815,12 +1815,12 @@ impl<'a, 'gcx, 'tcx> RegionCtxt<'a, 'gcx, 'tcx> { // check whether this predicate applies to our current projection let cause = self.fcx.misc(span); match self.at(&cause, self.fcx.param_env).eq(outlives.0, ty) { - Ok(ok) => { - self.register_infer_ok_obligations(ok); - Ok(outlives.1) - } - Err(_) => { Err(()) } + Ok(ok) => Ok((ok, outlives.1)), + Err(_) => Err(()) } + }).map(|(ok, result)| { + self.register_infer_ok_obligations(ok); + result }); debug!("projection_bounds: region_result={:?}", |
