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

Issue 1595243003: Move AST implementation out of generated (Closed)

Created:
4 years, 11 months ago by Brian Wilkerson
Modified:
4 years, 11 months ago
Reviewers:
Paul Berry, scheglov
CC:
reviews_dartlang.org
Base URL:
https://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 (+82 lines, -10868 lines) Patch
M pkg/analyzer/lib/dart/ast/ast.dart View 1 chunk +2 lines, -2 lines 0 comments Download
A + pkg/analyzer/lib/src/dart/ast/ast.dart View 2 chunks +1 line, -5 lines 0 comments Download
M pkg/analyzer/lib/src/dart/ast/utilities.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer/lib/src/generated/ast.dart View 1 chunk +10 lines, -10800 lines 2 comments Download
M pkg/analyzer/lib/src/generated/element.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer/lib/src/generated/element_resolver.dart View 5 chunks +56 lines, -53 lines 0 comments Download
M pkg/analyzer/lib/src/generated/parser.dart View 3 chunks +9 lines, -5 lines 0 comments Download
M pkg/analyzer/lib/src/generated/testing/ast_factory.dart View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 7 (2 generated)
Brian Wilkerson
Import clean-up to follow.
4 years, 11 months ago (2016-01-18 19:18:56 UTC) #2
scheglov
LGTM https://codereview.chromium.org/1595243003/diff/1/pkg/analyzer/lib/src/generated/ast.dart File pkg/analyzer/lib/src/generated/ast.dart (right): https://codereview.chromium.org/1595243003/diff/1/pkg/analyzer/lib/src/generated/ast.dart#newcode15 pkg/analyzer/lib/src/generated/ast.dart:15: library analyzer.src.generated.ast; Maybe @deprecated also? Or this will ...
4 years, 11 months ago (2016-01-18 19:52:31 UTC) #3
Brian Wilkerson
https://codereview.chromium.org/1595243003/diff/1/pkg/analyzer/lib/src/generated/ast.dart File pkg/analyzer/lib/src/generated/ast.dart (right): https://codereview.chromium.org/1595243003/diff/1/pkg/analyzer/lib/src/generated/ast.dart#newcode15 pkg/analyzer/lib/src/generated/ast.dart:15: library analyzer.src.generated.ast; > Or this will cause hints and ...
4 years, 11 months ago (2016-01-18 21:00:50 UTC) #4
Brian Wilkerson
Committed patchset #1 (id:1) manually as 539f0dff4ab9523c48b0ac51c63304c68c33eaab (presubmit successful).
4 years, 11 months ago (2016-01-18 21:02:09 UTC) #6
skybrian
4 years, 11 months ago (2016-01-20 00:57:14 UTC) #7
Message was sent while issue was closed.
On 2016/01/18 21:02:09, Brian Wilkerson wrote:
> Committed patchset #1 (id:1) manually as
> 539f0dff4ab9523c48b0ac51c63304c68c33eaab (presubmit successful).

This is great, but ast.dart is still a 10k+ line file. Any thoughts on splitting
it up?

Or maybe remove most of the comments on the fields? Some of them look redundant.

  /**
   * The left parenthesis.
   */
  Token leftParenthesis;

Powered by Google App Engine
This is Rietveld 408576698