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

Unified Diff: pkg/front_end/lib/src/incremental/kernel_driver.dart

Issue 2982093003: Unifying compiler context (Closed)
Patch Set: Created 3 years, 5 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/front_end/lib/src/incremental/kernel_driver.dart
diff --git a/pkg/front_end/lib/src/incremental/kernel_driver.dart b/pkg/front_end/lib/src/incremental/kernel_driver.dart
index 4b55b6987023ac4387eab996bc4bee563e4aa7db..205e861edb391df6bb38c02f2623d7c051e185f2 100644
--- a/pkg/front_end/lib/src/incremental/kernel_driver.dart
+++ b/pkg/front_end/lib/src/incremental/kernel_driver.dart
@@ -5,9 +5,10 @@
import 'dart:async';
import 'package:front_end/file_system.dart';
+import 'package:front_end/compiler_options.dart';
import 'package:front_end/src/base/api_signature.dart';
+import 'package:front_end/src/base/processed_options.dart';
import 'package:front_end/src/base/performace_logger.dart';
-import 'package:front_end/src/fasta/compiler_command_line.dart';
import 'package:front_end/src/fasta/compiler_context.dart';
import 'package:front_end/src/fasta/dill/dill_library_builder.dart';
import 'package:front_end/src/fasta/dill/dill_target.dart';
@@ -149,13 +150,11 @@ class KernelDriver {
}
Future<T> runWithFrontEndContext<T>(String msg, Future<T> f()) async {
- return await CompilerCommandLine.withGlobalOptions("", [""],
- (CompilerContext context) {
- context.options.options["--target"] = _target;
- context.options.options["report"] = (message, severity) {};
- context.options.options["reportWithoutLocation"] = (message, severity) {};
- return _logger.runAsync(msg, f);
- });
+ var options = new CompilerOptions()
+ ..target = _target
+ ..reportMessages = false; // ignore errors.
ahe 2017/07/18 16:54:37 Seems like we need a TODO or an explanation why we
Siggi Cherem (dart-lang) 2017/07/18 22:50:44 Done.
+ return await CompilerContext.runWithOptions(
+ new ProcessedOptions(options), (_) => _logger.runAsync(msg, f));
}
/// The file with the given [uri] might have changed - updated, added, or

Powered by Google App Engine
This is Rietveld 408576698