diff --git a/src/librustc/middle/typeck/check/vtable.rs b/src/librustc/middle/typeck/check/vtable.rs index cc8a2b82772..3f50a64e630 100644 --- a/src/librustc/middle/typeck/check/vtable.rs +++ b/src/librustc/middle/typeck/check/vtable.rs @@ -242,16 +242,17 @@ fn lookup_vtable(vcx: &VtableContext, // bounds to see if they include the trait we are looking for. let vtable_opt = match ty::get(ty).sty { ty::ty_param(param_ty {idx: n, ..}) => { - let type_param_bounds: &[@ty::TraitRef] = - vcx.param_env - .type_param_bounds - .get(n) - .trait_bounds - .as_slice(); - lookup_vtable_from_bounds(vcx, span, - type_param_bounds, - param_numbered(n), - trait_ref) + let env_bounds = &vcx.param_env.type_param_bounds; + if env_bounds.len() > n { + let type_param_bounds: &[@ty::TraitRef] = + env_bounds.get(n).trait_bounds.as_slice(); + lookup_vtable_from_bounds(vcx, span, + type_param_bounds, + param_numbered(n), + trait_ref) + } else { + None + } } ty::ty_self(_) => { diff --git a/src/librustc/util/ppaux.rs b/src/librustc/util/ppaux.rs index 3bb344548e1..60e0bd9e791 100644 --- a/src/librustc/util/ppaux.rs +++ b/src/librustc/util/ppaux.rs @@ -393,8 +393,9 @@ pub fn ty_to_str(cx: &ctxt, typ: t) -> ~str { ty_param(param_ty {idx: id, def_id: did}) => { let ident = match cx.ty_param_defs.borrow().find(&did.node) { Some(def) => token::get_ident(def.ident).get().to_str(), - // This should not happen... - None => format!("BUG[{:?}]", id) + // This can only happen when a type mismatch error happens and + // the actual type has more type parameters than the expected one. + None => format!("", id) }; if !cx.sess.verbose() { ident diff --git a/src/test/compile-fail/issue-13466.rs b/src/test/compile-fail/issue-13466.rs new file mode 100644 index 00000000000..e4621495f47 --- /dev/null +++ b/src/test/compile-fail/issue-13466.rs @@ -0,0 +1,22 @@ +// 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 or the MIT license +// , at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. + +// Regression test for #13466 + +pub fn main() { + // The expected arm type `Option` has one type parameter, while + // the actual arm `Result` has two. typeck should not be + // tricked into looking up a non-existing second type parameter. + let _x: uint = match Some(1u) { + //~^ ERROR mismatched types: expected `uint` but found `` + Ok(u) => u, //~ ERROR mismatched types: expected `std::option::Option` + Err(e) => fail!(e) //~ ERROR mismatched types: expected `std::option::Option` + }; +}