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

Unified Diff: pkg/front_end/lib/src/fasta/kernel/kernel_target.dart

Issue 2731633003: [Fasta] Include line starts (Closed)
Patch Set: Created 3 years, 10 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/kernel/kernel_target.dart
diff --git a/pkg/front_end/lib/src/fasta/kernel/kernel_target.dart b/pkg/front_end/lib/src/fasta/kernel/kernel_target.dart
index e13920775982b0b76f2cc1b6ec6089f3a2169022..4bef35917f563f1215bc19eab98367496202112a 100644
--- a/pkg/front_end/lib/src/fasta/kernel/kernel_target.dart
+++ b/pkg/front_end/lib/src/fasta/kernel/kernel_target.dart
@@ -310,12 +310,8 @@ class KernelTarget extends TargetImplementation {
/// Creates a program by combining [libraries] with the libraries of
/// `dillTarget.loader.program`.
Program link(List<Library> libraries) {
- Map<String, Source> uriToSource = <String, Source>{};
-
- // for (Library library in libraries) {
- // // TODO(ahe): Compute line starts instead.
- // uriToLineStarts[library.fileUri] = <int>[0];
- // }
+ Map<String, Source> uriToSource =
+ new Map<String, Source>.from(this.uriToSource);
final Program binary = dillTarget.loader.program;
if (binary != null) {

Powered by Google App Engine
This is Rietveld 408576698