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

Issue 1359243003: Infer setter parameter types in strong mode (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

Infer setter parameter types in strong mode This change also decouples setters and getters, which matches what DDC currently does. The benefit is they're inferred as independent methods, which allows code reuse between method/getter/setters. It also prevents ordering issues that might occur otherwise (i.e. if getter inference considers setter's type, and setter inference considers the getter's type). More context: https://github.com/dart-lang/dev_compiler/issues/342#issuecomment-142713453 R=brianwilkerson@google.com Committed: https://github.com/dart-lang/sdk/commit/83a2449062a56623919f19f57743fc9b841e7fad

Patch Set 1 #

Patch Set 2 : format #

Total comments: 2

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+139 lines, -52 lines) Patch
M pkg/analyzer/lib/src/task/strong_mode.dart View 1 2 4 chunks +51 lines, -51 lines 0 comments Download
M pkg/analyzer/test/src/task/strong_mode_test.dart View 2 chunks +88 lines, -1 line 0 comments Download

Messages

Total messages: 5 (1 generated)
Jennifer Messerly
Note, there's a related issue with generics, https://github.com/dart-lang/dev_compiler/issues/342#issuecomment-142748853, I'm working on that in a separate ...
5 years, 2 months ago (2015-09-24 17:15:36 UTC) #2
Brian Wilkerson
LGTM https://codereview.chromium.org/1359243003/diff/20001/pkg/analyzer/lib/src/task/strong_mode.dart File pkg/analyzer/lib/src/task/strong_mode.dart (right): https://codereview.chromium.org/1359243003/diff/20001/pkg/analyzer/lib/src/task/strong_mode.dart#newcode320 pkg/analyzer/lib/src/task/strong_mode.dart:320: void _inferMethod(ExecutableElement element) { nit: consider renaming to ...
5 years, 2 months ago (2015-09-24 19:31:51 UTC) #3
Jennifer Messerly
https://codereview.chromium.org/1359243003/diff/20001/pkg/analyzer/lib/src/task/strong_mode.dart File pkg/analyzer/lib/src/task/strong_mode.dart (right): https://codereview.chromium.org/1359243003/diff/20001/pkg/analyzer/lib/src/task/strong_mode.dart#newcode320 pkg/analyzer/lib/src/task/strong_mode.dart:320: void _inferMethod(ExecutableElement element) { On 2015/09/24 19:31:51, Brian Wilkerson ...
5 years, 2 months ago (2015-09-24 20:20:44 UTC) #4
Jennifer Messerly
5 years, 2 months ago (2015-09-24 20:21:09 UTC) #5
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
83a2449062a56623919f19f57743fc9b841e7fad (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698