In ty::normalize_ty, don't replace self_regions with None

Instead, replace with re_static. This was causing ty::subst to
fail when called from trans::type_of::type_of.

Already discussed with nmatsakis and it's a small change, so
no review.

Closes #3447
This commit is contained in:
Tim Chevalier 2012-10-15 17:18:45 -07:00
parent bbc46d527d
commit b38092e9a2
2 changed files with 12 additions and 7 deletions

View File

@ -1473,7 +1473,10 @@ fn subst(cx: ctxt,
fold_regions_and_ty(
cx, typ,
|r| match r {
re_bound(br_self) => substs.self_r.get(),
re_bound(br_self) => substs.self_r.expect(
#fmt("ty::subst: \
Reference to self region when given substs with no \
self region, ty = %s", ty_to_str(cx, typ))),
_ => r
},
|t| do_subst(cx, substs, t),
@ -3910,9 +3913,11 @@ fn normalize_ty(cx: ctxt, t: t) -> t {
ty_enum(did, r) =>
match r.self_r {
Some(_) =>
// This enum has a self region. Get rid of it
// Use re_static since trans doesn't care about regions
mk_enum(cx, did,
{self_r: None, self_ty: None, tps: r.tps}),
{self_r: Some(ty::re_static),
self_ty: None,
tps: r.tps}),
None =>
t
},
@ -3921,7 +3926,8 @@ fn normalize_ty(cx: ctxt, t: t) -> t {
match r.self_r {
Some(_) =>
// Ditto.
mk_class(cx, did, {self_r: None, self_ty: None, tps: r.tps}),
mk_class(cx, did, {self_r: Some(ty::re_static), self_ty: None,
tps: r.tps}),
None =>
t
},

View File

@ -1,4 +1,3 @@
// xfail-test
struct list<T> {
element: &self/T,
mut next: Option<@list<T>>
@ -11,13 +10,13 @@ impl<T> list<T>{
next: option::None
};
self.next = Some(@newList);
self.next = Some(@(move newList));
}
}
fn main() {
let s = @"str";
let ls: list<@str> = list {
let ls = list {
element: &s,
next: option::None
};