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

Unified Diff: pkg/kernel/test/class_hierarchy_tester.dart

Issue 2561723003: Merge kernel closure conversion into the Dart SDK (Closed)
Patch Set: Remove path constraint Created 4 years 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
« no previous file with comments | « pkg/kernel/test/class_hierarchy_membench.dart ('k') | pkg/kernel/test/closures/kernel_chain.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/kernel/test/class_hierarchy_tester.dart
diff --git a/pkg/kernel/test/class_hierarchy_tester.dart b/pkg/kernel/test/class_hierarchy_tester.dart
index d4e04ad95f84a0d460d1a347e6ba0ee8b5e3b2ab..895006df2e2037de436659690d8efc41150e4258 100644
--- a/pkg/kernel/test/class_hierarchy_tester.dart
+++ b/pkg/kernel/test/class_hierarchy_tester.dart
@@ -115,12 +115,14 @@ void testClassHierarchyOnProgram(Program program, {bool verbose: false}) {
String eq = setter ? '=' : '';
return '$member$eq overrides $superMember$eq';
}
+
Set<String> expectedOverrides = new Set<String>();
basic.forEachOverridePair(classNode, (member, superMember, setter) {
expectedOverrides.add(getHash(member, superMember, setter));
});
Set<String> actualOverrides = new Set<String>();
- classHierarchy.forEachOverridePair(classNode, (member, superMember, setter) {
+ classHierarchy.forEachOverridePair(classNode,
+ (member, superMember, setter) {
actualOverrides.add(getHash(member, superMember, setter));
});
for (var actual in actualOverrides) {
@@ -142,7 +144,7 @@ void testClassHierarchyOnProgram(Program program, {bool verbose: false}) {
var random = new Random(12345);
List/*<T>*/ pickRandom/*<T>*/(List/*<T>*/ items, int n) {
- var result = /*<T>*/[];
+ var result = /*<T>*/ [];
for (int i = 0; i < n; ++i) {
result.add(items[random.nextInt(items.length)]);
}
« no previous file with comments | « pkg/kernel/test/class_hierarchy_membench.dart ('k') | pkg/kernel/test/closures/kernel_chain.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698