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

Issue 2900963009: enable analyzer scanner adapter generic method support (Closed)

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

Description

enable analyzer scanner adapter generic method support This turns on generic method support in the scanner adapter layer and updates the token.offset when the analyzer scanner is being used to parse generic method comments. R=scheglov@google.com Committed: https://github.com/dart-lang/sdk/commit/6e981d0cbf2fa0d41765bb78b10c2021f6939b6f

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+38 lines, -29 lines) Patch
M pkg/analyzer/lib/src/dart/scanner/scanner.dart View 6 chunks +27 lines, -10 lines 0 comments Download
M pkg/analyzer/lib/src/fasta/token_utils.dart View 1 chunk +10 lines, -2 lines 0 comments Download
M pkg/front_end/lib/src/scanner/reader.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/front_end/test/scanner_roundtrip_test.dart View 1 chunk +0 lines, -16 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
danrubel
3 years, 7 months ago (2017-05-24 17:07:31 UTC) #2
scheglov
LGTM
3 years, 7 months ago (2017-05-24 17:24:07 UTC) #3
danrubel
3 years, 7 months ago (2017-05-24 17:37:58 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
6e981d0cbf2fa0d41765bb78b10c2021f6939b6f (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698