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 2422333002: Remove _functionTypesToFix from ElementBuilder. (Closed)

Created:
4 years, 2 months ago by scheglov
Modified:
4 years, 2 months ago
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Remove _functionTypesToFix from ElementBuilder. We might have needed it in the past, but now 'new FunctionTypeImpl(element)' does not use the enclosing element, at least until we start calling its methods, which happens after we finish building the element model. R=paulberry@google.com, brianwilkerson@google.com BUG= Committed: https://github.com/dart-lang/sdk/commit/fabb9dbf18042c1ba7ada20a7e2b0ca0bab35a2f

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -20 lines) Patch
M pkg/analyzer/lib/src/dart/element/builder.dart View 4 chunks +1 line, -20 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
scheglov
4 years, 2 months ago (2016-10-17 20:29:04 UTC) #1
Paul Berry
lgtm
4 years, 2 months ago (2016-10-17 20:31:39 UTC) #2
scheglov
Committed patchset #1 (id:1) manually as fabb9dbf18042c1ba7ada20a7e2b0ca0bab35a2f (presubmit successful).
4 years, 2 months ago (2016-10-17 21:26:46 UTC) #4
Brian Wilkerson
4 years, 2 months ago (2016-10-17 21:30:30 UTC) #5
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698