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

Issue 2923113002: translate remaining fasta scanner error codes (Closed)

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

Description

translate remaining fasta scanner error codes This updates the fasta scanner to translate the remaining previously untranslated error codes so that they will be seen by the analyzer when the fasta scanner is used by analyzer. R=ahe@google.com Committed: https://github.com/dart-lang/sdk/commit/c36936f37f68bfd8eb82dca53eb2fd1a43a5d779

Patch Set 1 #

Total comments: 4

Patch Set 2 : address comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+188 lines, -95 lines) Patch
M pkg/front_end/lib/src/fasta/scanner/recover.dart View 1 chunk +1 line, -2 lines 0 comments Download
M pkg/front_end/lib/src/scanner/errors.dart View 3 chunks +25 lines, -4 lines 0 comments Download
M pkg/front_end/test/scanner_fasta_test.dart View 1 15 chunks +39 lines, -39 lines 0 comments Download
M pkg/front_end/test/scanner_replacement_test.dart View 1 7 chunks +30 lines, -14 lines 0 comments Download
M pkg/front_end/test/scanner_test.dart View 1 18 chunks +93 lines, -36 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
danrubel
3 years, 6 months ago (2017-06-06 02:08:59 UTC) #2
ahe
lgtm Subjective (ignorable) nits below. https://codereview.chromium.org/2923113002/diff/1/pkg/front_end/test/scanner_fasta_test.dart File pkg/front_end/test/scanner_fasta_test.dart (right): https://codereview.chromium.org/2923113002/diff/1/pkg/front_end/test/scanner_fasta_test.dart#newcode214 pkg/front_end/test/scanner_fasta_test.dart:214: scanWithListener(r'"${({(}}"', listener).next as BeginToken; ...
3 years, 6 months ago (2017-06-06 10:17:17 UTC) #3
danrubel
https://codereview.chromium.org/2923113002/diff/1/pkg/front_end/test/scanner_fasta_test.dart File pkg/front_end/test/scanner_fasta_test.dart (right): https://codereview.chromium.org/2923113002/diff/1/pkg/front_end/test/scanner_fasta_test.dart#newcode214 pkg/front_end/test/scanner_fasta_test.dart:214: scanWithListener(r'"${({(}}"', listener).next as BeginToken; On 2017/06/06 10:17:16, ahe wrote: ...
3 years, 6 months ago (2017-06-06 11:50:41 UTC) #4
danrubel
3 years, 6 months ago (2017-06-06 13:08:19 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
c36936f37f68bfd8eb82dca53eb2fd1a43a5d779 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698