Skip to content

Commit

Permalink
auto merge of #19813 : barosl/rust/autoderef-type-inference-ice, r=pn…
Browse files Browse the repository at this point in the history
…kfelix

`check::autoderef()` returns a `ty_err` when it fails to infer the type. `probe::probe()` should respect this failure and fail together to prevent further corruption.

Fixes #19692.
Fixes #19583.
Fixes #19297.
  • Loading branch information
bors committed Dec 24, 2014
2 parents e64a819 + d9f3067 commit 8730d64
Show file tree
Hide file tree
Showing 2 changed files with 26 additions and 5 deletions.
13 changes: 8 additions & 5 deletions src/librustc_typeck/check/method/probe.rs
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,10 @@ pub fn probe<'a, 'tcx>(fcx: &FnCtxt<'a, 'tcx>,
// it ride, although it's really not great, and in fact could I
// think cause spurious errors. Really though this part should
// take place in the `fcx.infcx().probe` below.
let steps = create_steps(fcx, span, self_ty);
let steps = match create_steps(fcx, span, self_ty) {
Some(steps) => steps,
None => return Err(NoMatch(Vec::new())),
};

// Create a list of simplified self types, if we can.
let mut simplified_steps = Vec::new();
Expand Down Expand Up @@ -160,7 +163,7 @@ pub fn probe<'a, 'tcx>(fcx: &FnCtxt<'a, 'tcx>,
fn create_steps<'a, 'tcx>(fcx: &FnCtxt<'a, 'tcx>,
span: Span,
self_ty: Ty<'tcx>)
-> Vec<CandidateStep<'tcx>> {
-> Option<Vec<CandidateStep<'tcx>>> {
let mut steps = Vec::new();

let (fully_dereferenced_ty, dereferences, _) =
Expand All @@ -179,11 +182,11 @@ fn create_steps<'a, 'tcx>(fcx: &FnCtxt<'a, 'tcx>,
adjustment: AutoUnsizeLength(dereferences, len),
});
}
_ => {
}
ty::ty_err => return None,
_ => (),
}

return steps;
Some(steps)
}

impl<'a,'tcx> ProbeContext<'a,'tcx> {
Expand Down
18 changes: 18 additions & 0 deletions src/test/compile-fail/issue-19692.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
// Copyright 2014 The Rust Project Developers. See the COPYRIGHT
// file at the top-level directory of this distribution and at
// http://rust-lang.org/COPYRIGHT.
//
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.

struct Homura;

fn akemi(homura: Homura) {
let Some(ref madoka) = Some(homura.kaname()); //~ ERROR does not implement any method
madoka.clone(); //~ ERROR the type of this value must be known
}

fn main() { }

0 comments on commit 8730d64

Please sign in to comment.