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

Issue 2866973005: align fasta.Token and analyzer.Token (Closed)

Created:
3 years, 7 months ago by danrubel
Modified:
3 years, 6 months ago
Reviewers:
Paul Berry, scheglov, ahe
CC:
reviews_dartlang.org, dart-fe-team+reviews_google.com
Target Ref:
refs/heads/master
Visibility:
Public.

Description

align fasta.Token and analyzer.Token * move kind to analyzer.Token * move charOffset, charLength, charEnd to analyzer.Token * move isEOF from fasta.Token into analyzer.Token * move stringValue into TokenType R=paulberry@google.com, scheglov@google.com Committed: https://github.com/dart-lang/sdk/commit/a14bda01a94c8553a72d08fd7d75a195d786cb83

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+105 lines, -56 lines) Patch
M pkg/front_end/lib/src/fasta/scanner/error_token.dart View 1 chunk +0 lines, -2 lines 0 comments Download
M pkg/front_end/lib/src/fasta/scanner/token.dart View 7 chunks +7 lines, -38 lines 0 comments Download
M pkg/front_end/lib/src/scanner/token.dart View 10 chunks +98 lines, -16 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
danrubel
3 years, 7 months ago (2017-05-08 16:49:01 UTC) #2
scheglov
LGTM
3 years, 7 months ago (2017-05-08 16:54:03 UTC) #3
Paul Berry
lgtm
3 years, 7 months ago (2017-05-08 17:02:14 UTC) #4
danrubel
Committed patchset #1 (id:1) manually as a14bda01a94c8553a72d08fd7d75a195d786cb83 (presubmit successful).
3 years, 7 months ago (2017-05-08 19:44:44 UTC) #6
ahe
3 years, 6 months ago (2017-05-31 13:20:24 UTC) #8
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698