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

Issue 2536253002: Consolidate AST element/type interfaces. (Closed)

Created:
4 years ago by Paul Berry
Modified:
4 years ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Consolidate AST element/type interfaces. The AST getters/setters for `bestElement`, `propagatedElement`, and `staticElement` having type `MethodElement` are moved to a common interface class, `MethodReferenceExpression`. The AST getters/setters for `staticElement` having type `ConstructorElement` are moved to a common interface class, `ConstructorReferenceNode`. The overrides for `propagatedInvokeType` and `staticInvokeType` are removed from `FunctionExpressionInvocation` and `MethodInvocation`, since these overrides are identical to the corresponding declarations in the base class, `InvocationExpression`. R=brianwilkerson@google.com Committed: https://github.com/dart-lang/sdk/commit/be11197a6ca272d30b3f8ecac83cc065fccdbd35

Patch Set 1 #

Patch Set 2 : Add a missing comment #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+82 lines, -316 lines) Patch
M pkg/analyzer/lib/dart/ast/ast.dart View 1 31 chunks +82 lines, -316 lines 4 comments Download

Messages

Total messages: 6 (2 generated)
Paul Berry
4 years ago (2016-11-29 17:42:56 UTC) #2
Brian Wilkerson
lgtm https://codereview.chromium.org/2536253002/diff/20001/pkg/analyzer/lib/dart/ast/ast.dart File pkg/analyzer/lib/dart/ast/ast.dart (right): https://codereview.chromium.org/2536253002/diff/20001/pkg/analyzer/lib/dart/ast/ast.dart#newcode2404 pkg/analyzer/lib/dart/ast/ast.dart:2404: * An AST node which makes reference to ...
4 years ago (2016-11-29 18:07:28 UTC) #3
Paul Berry
https://codereview.chromium.org/2536253002/diff/20001/pkg/analyzer/lib/dart/ast/ast.dart File pkg/analyzer/lib/dart/ast/ast.dart (right): https://codereview.chromium.org/2536253002/diff/20001/pkg/analyzer/lib/dart/ast/ast.dart#newcode2404 pkg/analyzer/lib/dart/ast/ast.dart:2404: * An AST node which makes reference to a ...
4 years ago (2016-11-29 18:11:03 UTC) #4
Paul Berry
4 years ago (2016-11-29 18:11:42 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
be11197a6ca272d30b3f8ecac83cc065fccdbd35 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698