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

Unified Diff: pkg/front_end/lib/src/fasta/dill/dill_target.dart

Issue 2872903005: Rework DillLoader to allow adding multiple dills. (Closed)
Patch Set: Fixes for review changes. Created 3 years, 7 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/fasta/dill/dill_target.dart
diff --git a/pkg/front_end/lib/src/fasta/dill/dill_target.dart b/pkg/front_end/lib/src/fasta/dill/dill_target.dart
index 4e53c689b4a8f9b47f3d53314330587d58b78401..4dc805a04e79ceae053a27df5b24a9e9b752aec5 100644
--- a/pkg/front_end/lib/src/fasta/dill/dill_target.dart
+++ b/pkg/front_end/lib/src/fasta/dill/dill_target.dart
@@ -10,7 +10,7 @@ import 'package:kernel/ast.dart' show Class;
import 'dill_loader.dart' show DillLoader;
-import '../errors.dart' show inputError, internalError;
+import '../errors.dart' show internalError;
import '../target_implementation.dart' show TargetImplementation;
@@ -37,11 +37,7 @@ class DillTarget extends TargetImplementation {
}
void read(Uri uri) {
- if (loader.input == null) {
- loader.input = uri;
- } else {
- inputError(uri, -1, "Can only read one dill file.");
- }
+ internalError("Unsupported operation.");
}
Future<Null> writeProgram(Uri uri) {
@@ -49,7 +45,7 @@ class DillTarget extends TargetImplementation {
}
Future<Null> writeOutline(Uri uri) async {
- if (loader.input == null) return null;
+ if (loader.program == null) return null;
await loader.buildOutlines();
isLoaded = true;
return null;

Powered by Google App Engine
This is Rietveld 408576698