Skip to content

Commit

Permalink
project: add a recursion limit to "tail-recursive" projections
Browse files Browse the repository at this point in the history
Fixes #21946
Fixes #23992
Fixes #25945
  • Loading branch information
arielb1 authored and Ariel Ben-Yehuda committed Oct 27, 2015
1 parent 0152a93 commit 867fd0a
Show file tree
Hide file tree
Showing 3 changed files with 53 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/librustc/middle/traits/project.rs
Original file line number Diff line number Diff line change
Expand Up @@ -343,7 +343,8 @@ pub fn normalize_projection_type<'a,'b,'tcx>(
projection_ty: projection_ty,
ty: ty_var
});
let obligation = Obligation::with_depth(cause, depth+1, projection.to_predicate());
let obligation = Obligation::with_depth(
cause, depth + 1, projection.to_predicate());
Normalized {
value: ty_var,
obligations: vec!(obligation)
Expand Down Expand Up @@ -382,7 +383,7 @@ fn opt_normalize_projection_type<'a,'b,'tcx>(
obligations);

if projected_ty.has_projection_types() {
let mut normalizer = AssociatedTypeNormalizer::new(selcx, cause, depth);
let mut normalizer = AssociatedTypeNormalizer::new(selcx, cause, depth+1);
let normalized_ty = normalizer.fold(&projected_ty);

debug!("normalize_projection_type: normalized_ty={:?} depth={}",
Expand Down
22 changes: 22 additions & 0 deletions src/test/compile-fail/issue-21946.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
// Copyright 2015 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.

trait Foo {
type A;
}

struct FooStruct;

impl Foo for FooStruct {
//~^ ERROR overflow evaluating the requirement `<FooStruct as Foo>::A`
type A = <FooStruct as Foo>::A;
}

fn main() {}
28 changes: 28 additions & 0 deletions src/test/run-pass/issue-23992.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
// Copyright 2015 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 struct Outer<T: Trait>(T);
pub struct Inner<'a> { value: &'a bool }

pub trait Trait {
type Error;
fn ready(self) -> Self::Error;
}

impl<'a> Trait for Inner<'a> {
type Error = Outer<Inner<'a>>;
fn ready(self) -> Outer<Inner<'a>> { Outer(self) }
}

fn main() {
let value = true;
let inner = Inner { value: &value };
assert_eq!(inner.ready().0.value, &value);
}

0 comments on commit 867fd0a

Please sign in to comment.