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

Issue 2982323003: Implement AstBuilder integration for parenthesized expressions. (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

Implement AstBuilder integration for parenthesized expressions. The type inference engine doesn't even see parenthesized expressions as a separate entity, since parentheses don't have any semantics. So we just copy the static type from the enclosed expression. R=scheglov@google.com Committed: https://github.com/dart-lang/sdk/commit/cc97126aed1716c7aac1b95421b72b970bffa54a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -2 lines) Patch
M pkg/analyzer/lib/src/fasta/resolution_applier.dart View 1 chunk +6 lines, -0 lines 0 comments Download
M pkg/front_end/testcases/ast_builder.status View 2 chunks +0 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
Paul Berry
3 years, 5 months ago (2017-07-20 20:19:07 UTC) #2
scheglov
LGTM
3 years, 5 months ago (2017-07-20 20:23:17 UTC) #3
Paul Berry
3 years, 5 months ago (2017-07-20 20:49:25 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
cc97126aed1716c7aac1b95421b72b970bffa54a (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698