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

Issue 2745163004: Optionally use Fasta with the new analysis driver. (Closed)

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

Description

Optionally use Fasta with the new analysis driver. It's not ready to be enabled, but it is useful for me to enable it temporary, see what is broken, and continue patching the AST builder. R=brianwilkerson@google.com, paulberry@google.com BUG= Committed: https://github.com/dart-lang/sdk/commit/63ef801665a0ab3c9a880a1713795e1bbf360512

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+77 lines, -14 lines) Patch
M pkg/analyzer/lib/src/dart/analysis/file_state.dart View 6 chunks +75 lines, -12 lines 0 comments Download
M pkg/analyzer/test/src/dart/analysis/driver_test.dart View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
scheglov
3 years, 9 months ago (2017-03-14 14:59:36 UTC) #1
Brian Wilkerson
lgtm
3 years, 9 months ago (2017-03-14 15:10:43 UTC) #2
scheglov
Committed patchset #1 (id:1) manually as 63ef801665a0ab3c9a880a1713795e1bbf360512 (presubmit successful).
3 years, 9 months ago (2017-03-14 15:11:25 UTC) #4
Paul Berry
3 years, 9 months ago (2017-03-14 15:19:54 UTC) #5
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698