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

Issue 13375005: Implement subtype for type variables. (Closed)

Created:
7 years, 8 months ago by Johnni Winther
Modified:
7 years, 8 months ago
Reviewers:
karlklose, ahe
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Implement subtype for type variables. This implementation handles cyclicity in subtyping on type variables. Committed: https://code.google.com/p/dart/source/detail?r=21056

Patch Set 1 #

Patch Set 2 : Rebased #

Patch Set 3 : Rebased #

Patch Set 4 : Rebased #

Unified diffs Side-by-side diffs Delta from patch set Stats (+242 lines, -2 lines) Patch
M sdk/lib/_internal/compiler/implementation/dart_types.dart View 1 2 3 1 chunk +24 lines, -2 lines 0 comments Download
M tests/compiler/dart2js/subtype_test.dart View 2 chunks +218 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Johnni Winther
7 years, 8 months ago (2013-03-31 18:45:52 UTC) #1
Johnni Winther
7 years, 8 months ago (2013-03-31 18:45:52 UTC) #2
ahe
See my comments in https://codereview.chromium.org/13133006/
7 years, 8 months ago (2013-04-02 08:29:06 UTC) #3
Johnni Winther
On 2013/04/02 08:29:06, ahe wrote: > See my comments in https://codereview.chromium.org/13133006/ Fixed. PTAL
7 years, 8 months ago (2013-04-02 09:03:15 UTC) #4
ahe
LGTM!
7 years, 8 months ago (2013-04-02 09:11:38 UTC) #5
Johnni Winther
7 years, 8 months ago (2013-04-08 10:30:43 UTC) #6
Message was sent while issue was closed.
Committed patchset #4 manually as r21056 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698