From 381209889a348349f97e414485d5fcf3c8e8931d Mon Sep 17 00:00:00 2001 From: Adriaan Moors Date: Thu, 6 Aug 2009 15:36:00 +0000 Subject: fixed #2208 --- src/compiler/scala/tools/nsc/symtab/Types.scala | 2 ++ src/compiler/scala/tools/nsc/typechecker/Typers.scala | 8 ++++++-- 2 files changed, 8 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/compiler/scala/tools/nsc/symtab/Types.scala b/src/compiler/scala/tools/nsc/symtab/Types.scala index 5375f77569..28ca47e6a3 100644 --- a/src/compiler/scala/tools/nsc/symtab/Types.scala +++ b/src/compiler/scala/tools/nsc/symtab/Types.scala @@ -1511,6 +1511,8 @@ A type's typeSymbol should never be inspected directly. PolyType(typeParams, typeRef(pre, sym.initialize, higherKindedArgs)) } else if (sym.isRefinementClass) { sym.info.normalize // @MO to AM: OK? + //@M I think this is okay, but changeset 12414 (which fixed #1241) re-introduced another bug (#2208) + // see typedTypeConstructor in Typers } else { super.normalize } diff --git a/src/compiler/scala/tools/nsc/typechecker/Typers.scala b/src/compiler/scala/tools/nsc/typechecker/Typers.scala index 8056c2794e..f4f73cc1b3 100644 --- a/src/compiler/scala/tools/nsc/typechecker/Typers.scala +++ b/src/compiler/scala/tools/nsc/typechecker/Typers.scala @@ -3857,11 +3857,15 @@ trait Typers { self: Analyzer => /** Types a type constructor tree used in a new or supertype */ def typedTypeConstructor(tree: Tree, mode: Int): Tree = { val result = typed(tree, typeMode(mode) | FUNmode, WildcardType) - val restpe = result.tpe.normalize + + val restpe = result.tpe.normalize // normalize to get rid of type aliases for the following check (#1241) if (!phase.erasedTypes && restpe.isInstanceOf[TypeRef] && !restpe.prefix.isStable) { error(tree.pos, restpe.prefix+" is not a legal prefix for a constructor") } - result setType restpe // @M: normalization is done during erasure + + // @M: during uncurry, all types are normalized (after refchecks and before genicode) + result // must not normalize before refchecks, and thus must not do `result setType(restpe)` + // the original type must be ref-checked first, so that bounds of type args of type aliases are checked (see #2208) } def typedTypeConstructor(tree: Tree): Tree = typedTypeConstructor(tree, NOmode) -- cgit v1.2.3