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

Issue 1700203002: Serialize and resynthesize unresolved identifiers. (Closed)

Created:
4 years, 10 months ago by scheglov
Modified:
4 years, 10 months ago
Reviewers:
Paul Berry
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Serialize and resynthesize unresolved identifiers. This is surprising, but SDK (dart2js part) has unresolved constant expressions. So, we need to support them at least to some extent in order to pass shared tests. R=paulberry@google.com BUG= Committed: https://github.com/dart-lang/sdk/commit/665b82d04a7c9f6ef2bccc012c7de272de00cde8

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+152 lines, -18 lines) Patch
M pkg/analyzer/lib/src/summary/resynthesize.dart View 2 chunks +4 lines, -8 lines 0 comments Download
M pkg/analyzer/lib/src/summary/summarize_elements.dart View 2 chunks +30 lines, -3 lines 0 comments Download
M pkg/analyzer/test/src/summary/resynthesize_test.dart View 3 chunks +39 lines, -2 lines 0 comments Download
M pkg/analyzer/test/src/summary/summary_common.dart View 7 chunks +79 lines, -5 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
scheglov
4 years, 10 months ago (2016-02-16 17:47:27 UTC) #1
Paul Berry
lgtm
4 years, 10 months ago (2016-02-16 17:51:56 UTC) #2
scheglov
4 years, 10 months ago (2016-02-16 17:56:00 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
665b82d04a7c9f6ef2bccc012c7de272de00cde8 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698