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

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

Issue 2709943005: Canonicalize mixin applications by their name (Closed)
Patch Set: Address comment 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_library_builder.dart
diff --git a/pkg/front_end/lib/src/fasta/kernel/kernel_library_builder.dart b/pkg/front_end/lib/src/fasta/kernel/kernel_library_builder.dart
index 82b942e682295b5c79cd1eaefd8d9acdc213fe33..f4d7f4048df8de34d42ac512c828707c07b6f41b 100644
--- a/pkg/front_end/lib/src/fasta/kernel/kernel_library_builder.dart
+++ b/pkg/front_end/lib/src/fasta/kernel/kernel_library_builder.dart
@@ -61,7 +61,8 @@ class KernelLibraryBuilder
extends SourceLibraryBuilder<KernelTypeBuilder, Library> {
final Library library;
- final List<Class> mixinApplicationClasses = <Class>[];
+ final Map<String, SourceClassBuilder> mixinApplicationClasses =
+ <String, SourceClassBuilder>{};
final List<List> argumentsWithMissingDefaultValues = <List>[];
@@ -351,7 +352,7 @@ class KernelLibraryBuilder
super.includePart(part);
nativeMethods.addAll(part.nativeMethods);
boundlessTypeVariables.addAll(part.boundlessTypeVariables);
- mixinApplicationClasses.addAll(part.mixinApplicationClasses);
+ assert(mixinApplicationClasses.isEmpty);
}
}

Powered by Google App Engine
This is Rietveld 408576698