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

Issue 1660663002: Resynthesize generic class instance creations. (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

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+160 lines, -22 lines) Patch
M pkg/analyzer/lib/src/dart/element/type.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer/lib/src/summary/resynthesize.dart View 8 chunks +64 lines, -18 lines 2 comments Download
M pkg/analyzer/test/src/summary/resynthesize_test.dart View 5 chunks +95 lines, -3 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
scheglov
4 years, 10 months ago (2016-02-02 21:27:00 UTC) #1
Paul Berry
lgtm https://codereview.chromium.org/1660663002/diff/1/pkg/analyzer/lib/src/summary/resynthesize.dart File pkg/analyzer/lib/src/summary/resynthesize.dart (right): https://codereview.chromium.org/1660663002/diff/1/pkg/analyzer/lib/src/summary/resynthesize.dart#newcode512 pkg/analyzer/lib/src/summary/resynthesize.dart:512: constructorElement = _createConstructorElement(ref, info); Nit: since the logic ...
4 years, 10 months ago (2016-02-02 22:01:16 UTC) #2
scheglov
Committed patchset #1 (id:1) manually as 678048132b66783a40bb39d96fa3b649ddb7fe76 (presubmit successful).
4 years, 10 months ago (2016-02-02 22:27:36 UTC) #4
scheglov
4 years, 10 months ago (2016-02-02 22:28:04 UTC) #5
Message was sent while issue was closed.
https://codereview.chromium.org/1660663002/diff/1/pkg/analyzer/lib/src/summar...
File pkg/analyzer/lib/src/summary/resynthesize.dart (right):

https://codereview.chromium.org/1660663002/diff/1/pkg/analyzer/lib/src/summar...
pkg/analyzer/lib/src/summary/resynthesize.dart:512: constructorElement =
_createConstructorElement(ref, info);
On 2016/02/02 22:01:16, Paul Berry wrote:
> Nit: since the logic is the same for both constructor elements and class
> elements, this line and line 515 can be lifted out of the conditional and
merged
> with line 509.

Ah, yes.
This code used to be much more complex and different.
But now it is the same.
Thanks!

Powered by Google App Engine
This is Rietveld 408576698