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

Issue 2980053002: Further integration of front end type inference into analyzer. (Closed)

Created:
3 years, 5 months ago by Paul Berry
Modified:
3 years, 5 months ago
Reviewers:
scheglov
CC:
reviews_dartlang.org, dart-fe-team+reviews_google.com
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Further integration of front end type inference into analyzer. This CL propagates types for simple variable declarations into analyzer ASTs. Note that for a variable declaration like `int x` we need to store two pieces of information: the element pointed to by `int`, and the type of `x`. This CL handles the latter. The former will wait until we have enough of the element model built that we have an element to point to. R=scheglov@google.com Committed: https://github.com/dart-lang/sdk/commit/df91c790783266d5fe55676c9f9563fc338bed1c

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+36 lines, -23 lines) Patch
M pkg/analyzer/lib/src/fasta/resolution_applier.dart View 2 chunks +22 lines, -2 lines 2 comments Download
M pkg/analyzer/lib/src/fasta/resolution_storer.dart View 2 chunks +14 lines, -5 lines 0 comments Download
M pkg/front_end/testcases/ast_builder.status View 11 chunks +0 lines, -16 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Paul Berry
3 years, 5 months ago (2017-07-13 21:06:28 UTC) #2
scheglov
LGTM https://codereview.chromium.org/2980053002/diff/1/pkg/analyzer/lib/src/fasta/resolution_applier.dart File pkg/analyzer/lib/src/fasta/resolution_applier.dart (right): https://codereview.chromium.org/2980053002/diff/1/pkg/analyzer/lib/src/fasta/resolution_applier.dart#newcode56 pkg/analyzer/lib/src/fasta/resolution_applier.dart:56: void _applyTypeAnnotation(DartType type, TypeAnnotation typeAnnotation) { I think ...
3 years, 5 months ago (2017-07-13 21:15:18 UTC) #3
Paul Berry
https://codereview.chromium.org/2980053002/diff/1/pkg/analyzer/lib/src/fasta/resolution_applier.dart File pkg/analyzer/lib/src/fasta/resolution_applier.dart (right): https://codereview.chromium.org/2980053002/diff/1/pkg/analyzer/lib/src/fasta/resolution_applier.dart#newcode56 pkg/analyzer/lib/src/fasta/resolution_applier.dart:56: void _applyTypeAnnotation(DartType type, TypeAnnotation typeAnnotation) { On 2017/07/13 21:15:17, ...
3 years, 5 months ago (2017-07-13 21:17:05 UTC) #4
Paul Berry
3 years, 5 months ago (2017-07-13 21:42:11 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
df91c790783266d5fe55676c9f9563fc338bed1c (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698