Skip to content

Commit

Permalink
Fix an ICE with error types in a vec
Browse files Browse the repository at this point in the history
Closes #16783
  • Loading branch information
nrc committed Aug 27, 2014
1 parent f2b87e9 commit d13fe1c
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/librustc/middle/ty.rs
Expand Up @@ -3266,6 +3266,11 @@ pub fn adjust_ty(cx: &ctxt,
-> ty::t {
/*! See `expr_ty_adjusted` */

match get(unadjusted_ty).sty {
ty_err => return unadjusted_ty,
_ => {}
}

return match adjustment {
Some(adjustment) => {
match *adjustment {
Expand Down
15 changes: 15 additions & 0 deletions src/test/compile-fail/issue-16783.rs
@@ -0,0 +1,15 @@
// 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.

pub fn main() {
let x = [1, 2, 3];
//~^ ERROR cannot determine a type for this local variable: cannot determine the type of this
let y = x.as_slice();
}

25 comments on commit d13fe1c

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Aug 31, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from luqmana
at nrc@d13fe1c

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Aug 31, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging nick29581/rust/dst-bug-1 = d13fe1c into auto

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Aug 31, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nick29581/rust/dst-bug-1 = d13fe1c merged ok, testing candidate = fc46da11

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Aug 31, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from luqmana
at nrc@d13fe1c

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Aug 31, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging nick29581/rust/dst-bug-1 = d13fe1c into auto

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Aug 31, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nick29581/rust/dst-bug-1 = d13fe1c merged ok, testing candidate = 55a2654c

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Aug 31, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from luqmana
at nrc@d13fe1c

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Aug 31, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging nick29581/rust/dst-bug-1 = d13fe1c into auto

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Aug 31, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nick29581/rust/dst-bug-1 = d13fe1c merged ok, testing candidate = 1d74b34f

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from luqmana
at nrc@d13fe1c

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging nick29581/rust/dst-bug-1 = d13fe1c into auto

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nick29581/rust/dst-bug-1 = d13fe1c merged ok, testing candidate = b3847f99

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from luqmana
at nrc@d13fe1c

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging nick29581/rust/dst-bug-1 = d13fe1c into auto

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nick29581/rust/dst-bug-1 = d13fe1c merged ok, testing candidate = 2100899e

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from luqmana
at nrc@d13fe1c

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging nick29581/rust/dst-bug-1 = d13fe1c into auto

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nick29581/rust/dst-bug-1 = d13fe1c merged ok, testing candidate = 12b438c

@bors
Copy link
Contributor

@bors bors commented on d13fe1c Sep 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 12b438c

Please sign in to comment.