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

Issue 1380963004: fixes strong mode assignability (Closed)

Created:
5 years, 2 months ago by Jennifer Messerly
Modified:
5 years, 2 months ago
Reviewers:
Leaf, Brian Wilkerson
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

fixes strong mode assignability this fixes https://github.com/dart-lang/dev_compiler/issues/356 The issue here is that StrongTypeSystemImpl has toType and fromType backwards from how they are defined on the base class. The way it works is to read it as "is the first type assignable to the second type"? (I can see how this may have happened: the base class uses "leftType" and "rightType", which refers to the parameter order, not the order they would appear in an assignment expression.) R=leafp@google.com Committed: https://github.com/dart-lang/sdk/commit/a8570d9e7cfe2427a2bd99348b9ae8199a5903ff

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M pkg/analyzer/lib/src/generated/resolver.dart View 1 chunk +1 line, -1 line 1 comment Download
M pkg/analyzer/test/generated/type_system_test.dart View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (1 generated)
Jennifer Messerly
Here's an example of how ErrorVerifier._checkForInvalidAssignment(lhs, rhs) uses this method, to illustrate the problem: if ...
5 years, 2 months ago (2015-10-01 22:24:15 UTC) #2
Leaf
Nasty one. Without warnings, our assignability is *almost* symmetric! LGTM
5 years, 2 months ago (2015-10-01 22:38:55 UTC) #3
Jennifer Messerly
Committed patchset #1 (id:1) manually as a8570d9e7cfe2427a2bd99348b9ae8199a5903ff (presubmit successful).
5 years, 2 months ago (2015-10-01 22:40:54 UTC) #4
Brian Wilkerson
5 years, 2 months ago (2015-10-01 23:50:57 UTC) #5
Message was sent while issue was closed.
LGTM

Powered by Google App Engine
This is Rietveld 408576698