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

Issue 2747683003: update fasta scanner to always generate INDEX or INDEX_EQ tokens (Closed)

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

Description

update fasta scanner to always generate INDEX or INDEX_EQ tokens * updated scanner to always generate INDEX and INDEX_EQ for "[]" and "[]=" * updated parser to generate ErrorKind.EmptyOptionalParameterList when "[]" detected * updated unparser to prevent INDEX token from being generated as [][] Take 2 of https://codereview.chromium.org/2744463003/ Fix https://github.com/dart-lang/sdk/issues/28665 Committed: https://github.com/dart-lang/sdk/commit/deb6eb8f8261e03b7191bad4c6f74b9654805c25

Patch Set 1 #

Patch Set 2 : merge #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -22 lines) Patch
M pkg/compiler/lib/src/tree/unparser.dart View 1 chunk +5 lines, -1 line 1 comment Download
M pkg/front_end/lib/src/fasta/parser/parser.dart View 1 chunk +5 lines, -0 lines 0 comments Download
M pkg/front_end/lib/src/fasta/scanner/abstract_scanner.dart View 1 chunk +1 line, -5 lines 0 comments Download
M pkg/front_end/test/scanner_fasta_test.dart View 1 chunk +0 lines, -16 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
danrubel
Same as prior CL except * fixed comment * tweaked code in https://codereview.chromium.org/2747683003/diff/1/pkg/front_end/lib/src/fasta/parser/parser.dart
3 years, 9 months ago (2017-03-13 14:58:30 UTC) #2
danrubel
Committed patchset #2 (id:20001) manually as deb6eb8f8261e03b7191bad4c6f74b9654805c25 (presubmit successful).
3 years, 9 months ago (2017-03-13 14:59:49 UTC) #4
ahe
3 years, 6 months ago (2017-06-01 14:45:01 UTC) #5
Message was sent while issue was closed.
lgtm

https://codereview.chromium.org/2747683003/diff/20001/pkg/compiler/lib/src/tr...
File pkg/compiler/lib/src/tree/unparser.dart (right):

https://codereview.chromium.org/2747683003/diff/20001/pkg/compiler/lib/src/tr...
pkg/compiler/lib/src/tree/unparser.dart:389: // if the NodeList is a single "[]"
token
if -> If

Powered by Google App Engine
This is Rietveld 408576698