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

Unified Diff: pkg/fasta/lib/src/source/source_loader.dart

Issue 2664593002: Port parser and scanner fixes from rasta branch. (Closed)
Patch Set: Update status files. Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: pkg/fasta/lib/src/source/source_loader.dart
diff --git a/pkg/fasta/lib/src/source/source_loader.dart b/pkg/fasta/lib/src/source/source_loader.dart
index f4aa0e9ac54b872c07b5bc501ce408afdacf3717..53a0aabae306a6b2f1f3aec3b44cb0afdeb8e3c8 100644
--- a/pkg/fasta/lib/src/source/source_loader.dart
+++ b/pkg/fasta/lib/src/source/source_loader.dart
@@ -13,10 +13,9 @@ import 'dart:io' show
import 'package:dart_scanner/io.dart' show
readBytesFromFile;
-import 'package:dart_scanner/src/token.dart' show
- Token;
-
import 'package:dart_scanner/dart_scanner.dart' show
+ ErrorToken,
+ Token,
scan;
import 'package:dart_parser/src/class_member_parser.dart' show
@@ -32,6 +31,7 @@ import 'package:kernel/core_types.dart' show
CoreTypes;
import '../errors.dart' show
+ InputError,
inputError;
import '../export.dart' show
@@ -77,7 +77,8 @@ class SourceLoader<L> extends Loader<L> {
SourceLoader(TargetImplementation target)
: super(target);
- Future<Token> tokenize(SourceLibraryBuilder library) async {
+ Future<Token> tokenize(SourceLibraryBuilder library,
+ {bool suppressLexicalErrors: false}) async {
Uri uri = library.uri;
if (uri.scheme != "file") {
uri = target.translateUri(uri);
@@ -90,7 +91,17 @@ class SourceLoader<L> extends Loader<L> {
try {
List<int> bytes = await readBytesFromFile(uri);
byteCount += bytes.length - 1;
- return scan(bytes).tokens;
+ Token token = scan(bytes).tokens;
+ while (token is ErrorToken) {
+ if (!suppressLexicalErrors) {
+ ErrorToken error = token;
+ String message = new InputError(
+ uri, token.charOffset, error.assertionMessage).format();
+ print(message);
+ }
+ token = token.next;
+ }
+ return token;
} on FileSystemException catch (e) {
String message = e.message;
String osMessage = e.osError?.message;
@@ -110,7 +121,7 @@ class SourceLoader<L> extends Loader<L> {
Future<Null> buildBody(LibraryBuilder library, AstKind astKind) async {
if (library is SourceLibraryBuilder) {
- Token tokens = await tokenize(library);
+ Token tokens = await tokenize(library, suppressLexicalErrors: true);
Johnni Winther 2017/01/30 09:04:39 Why suppress errors here?
ahe 2017/01/30 13:26:22 Added: // We tokenize source files twice to
if (tokens == null) return;
DietListener listener = new DietListener(
library, elementStore, hierarchy, coreTypes, astKind);

Powered by Google App Engine
This is Rietveld 408576698