Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(256)

Issue 247313004: Fix for 14221/ 15553/ 16673- X isSubtypeOf E now returns false when X is some type and E is a type … (Closed)

Created:
6 years, 8 months ago by jwren
Modified:
6 years, 8 months ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org, ricow1
Visibility:
Public.

Description

Fix for 14221/ 15553/ 16673- X isSubtypeOf E now returns false when X is some type and E is a type parameter. This fixes a few issues, but also breaks some assertions (language tests that now have valid warnings from the analyzer). Additional work will need to be done to fix the language tests, and to investigate the case of X isSubtypeOf E where X is of type Type. R=brianwilkerson@google.com Committed: https://code.google.com/p/dart/source/detail?r=35281

Patch Set 1 #

Patch Set 2 : Typo fix #

Patch Set 3 : #

Total comments: 2

Patch Set 4 : #

Messages

Total messages: 4 (0 generated)
jwren
6 years, 8 months ago (2014-04-22 17:40:52 UTC) #1
Brian Wilkerson
LGTM https://codereview.chromium.org/247313004/diff/40001/editor/tools/plugins/com.google.dart.engine_test/src/com/google/dart/engine/resolver/StaticTypeWarningCodeTest.java File editor/tools/plugins/com.google.dart.engine_test/src/com/google/dart/engine/resolver/StaticTypeWarningCodeTest.java (right): https://codereview.chromium.org/247313004/diff/40001/editor/tools/plugins/com.google.dart.engine_test/src/com/google/dart/engine/resolver/StaticTypeWarningCodeTest.java#newcode275 editor/tools/plugins/com.google.dart.engine_test/src/com/google/dart/engine/resolver/StaticTypeWarningCodeTest.java:275: "class B<T> {", Do the lines above this ...
6 years, 8 months ago (2014-04-22 19:36:17 UTC) #2
jwren
https://codereview.chromium.org/247313004/diff/40001/editor/tools/plugins/com.google.dart.engine_test/src/com/google/dart/engine/resolver/StaticTypeWarningCodeTest.java File editor/tools/plugins/com.google.dart.engine_test/src/com/google/dart/engine/resolver/StaticTypeWarningCodeTest.java (right): https://codereview.chromium.org/247313004/diff/40001/editor/tools/plugins/com.google.dart.engine_test/src/com/google/dart/engine/resolver/StaticTypeWarningCodeTest.java#newcode275 editor/tools/plugins/com.google.dart.engine_test/src/com/google/dart/engine/resolver/StaticTypeWarningCodeTest.java:275: "class B<T> {", On 2014/04/22 19:36:17, Brian Wilkerson wrote: ...
6 years, 8 months ago (2014-04-22 20:03:55 UTC) #3
jwren
6 years, 8 months ago (2014-04-22 20:08:12 UTC) #4
Message was sent while issue was closed.
Committed patchset #4 manually as r35281 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698