Crash: infinite recursion to cycle bounds

[DartBot]

This issue was originally filed by kiris60...@gmail.com

---

What steps will reproduce the problem?
Dartc compile this code.

---

class A<T> {}
class C<T extends T> extends A<T> {}
main() {}

---

Compiled results:

java.lang.StackOverflowError
    at com.google.dart.compiler.type.TypeVariableImplementation.getKind(TypeVariableImplementation.java:72)
    at com.google.dart.compiler.type.Types.asInstanceOf(Types.java:249)
    at com.google.dart.compiler.type.Types.asInstanceOf(Types.java:287)
    at com.google.dart.compiler.type.Types.asInstanceOf(Types.java:287)
    at com.google.dart.compiler.type.Types.asInstanceOf(Types.java:287)
    at com.google.dart.compiler.type.Types.asInstanceOf(Types.java:287)
    at com.google.dart.compiler.type.Types.asInstanceOf(Types.java:287)
    at com.google.dart.compiler.type.Types.asInstanceOf(Types.java:287)
    at com.google.dart.compiler.type.Types.asInstanceOf(Types.java:287)
    at com.google.dart.compiler.type.Types.asInstanceOf(Types.java:287)
    at com.google.dart.compiler.type.Types.asInstanceOf(Types.java:287)
        ...

[DartBot]

This comment was originally written by ki...@gmail.com

---

other case:

---

class A<T> {}
class C<T1 extends T2, T2 extends T1> extends A<T1> {}
main() {}

---

[DartBot]

This comment was originally written by drfibonacci@google.com

---

Added Area-Compiler, Triaged labels.

[DartBot]

This comment was originally written by mmendez@google.com

---

Set owner to mmendez@google.com.

[DartBot]

This comment was originally written by mmendez@google.com

---

Added Started label.

[DartBot]

This comment was originally written by mmendez@google.com

---

class C<T extends T> extends A<T> {} and class C<T1 extends T2, T2 extends T1> extends A<T1> {} have cyclic upper bounds on the type parameters and should probably result in a type error.

I don't recall seeing this restriction on the spec though although I may have missed it.

---

cc @gbracha.

[gbracha]

Yes, this is missing and should be a static type warning.

[DartBot]

This comment was originally written by mmendez@google.com

---

Removed Priority-Medium label.
Added Priority-High label.

[DartBot]

This comment was originally written by zundel@google.com

---

Set owner to zundel@google.com.

[DartBot]

This comment was originally written by zundel@google.com

---

http://codereview.chromium.org/9186017/

[DartBot]

This comment was originally written by zundel@google.com

---

r3751

---

Added Fixed label.