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

Issue 1565643002: Eliminate constructor return types from summaries. (Closed)

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

Description

Eliminate constructor return types from summaries. These can be easily inferred at the time the summary is resynthesized into an element model. This should make the summary size marginally smaller, and make it easier to create summaries straight from the AST. R=scheglov@google.com Committed: https://github.com/dart-lang/sdk/commit/5d470281376e64c369607540352683d2f5b4de00

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -31 lines) Patch
M pkg/analyzer/lib/src/summary/format.dart View 2 chunks +4 lines, -4 lines 0 comments Download
M pkg/analyzer/lib/src/summary/resynthesize.dart View 3 chunks +10 lines, -3 lines 0 comments Download
M pkg/analyzer/lib/src/summary/summarize_elements.dart View 1 chunk +2 lines, -1 line 0 comments Download
M pkg/analyzer/test/src/summary/summary_test.dart View 4 chunks +6 lines, -21 lines 1 comment Download
M pkg/analyzer/tool/summary/idl.dart View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
Paul Berry
https://codereview.chromium.org/1565643002/diff/1/pkg/analyzer/test/src/summary/summary_test.dart File pkg/analyzer/test/src/summary/summary_test.dart (right): https://codereview.chromium.org/1565643002/diff/1/pkg/analyzer/test/src/summary/summary_test.dart#newcode2191 pkg/analyzer/test/src/summary/summary_test.dart:2191: checkTypeRef(unlinkedUnits[0].variables.single.type, null, null, 'C', Note that this test was ...
4 years, 11 months ago (2016-01-06 02:18:13 UTC) #2
scheglov
LGTM
4 years, 11 months ago (2016-01-06 02:26:21 UTC) #3
Paul Berry
4 years, 11 months ago (2016-01-06 14:48:03 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
5d470281376e64c369607540352683d2f5b4de00 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698