Skip to content
This repository was archived by the owner on Aug 28, 2024. It is now read-only.

Commit 4160915

Browse files
authored
Replace uses of forEach with for-in loops (#232)
Generally, for-loops should be preferred over forEach() unless calling an existing unary function. See: https://www.dartlang.org/guides/language/effective-dart/usage#avoid-using-iterableforeach-with-a-function-literal This is a minor usage cleanup; no changes to functionality.
1 parent 63d875b commit 4160915

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

bin/format_coverage.dart

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,11 +77,15 @@ Future<Null> main(List<String> arguments) async {
7777
if (env.verbose) {
7878
if (resolver.failed.length > 0) {
7979
print('Failed to resolve:');
80-
resolver.failed.toSet().forEach((e) => print(' $e'));
80+
for (String error in resolver.failed.toSet()) {
81+
print(' $error');
82+
}
8183
}
8284
if (loader.failed.length > 0) {
8385
print('Failed to load:');
84-
loader.failed.toSet().forEach((e) => print(' $e'));
86+
for (String error in loader.failed.toSet()) {
87+
print(' $error');
88+
}
8589
}
8690
}
8791
await env.output.close();

lib/src/formatter.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,9 @@ class LcovFormatter implements Formatter {
5151

5252
buf.write('SF:$source\n');
5353
final lines = v.keys.toList()..sort();
54-
lines.forEach((int k) {
54+
for (int k in lines) {
5555
buf.write('DA:$k,${v[k]}\n');
56-
});
56+
}
5757
buf.write('LF:${lines.length}\n');
5858
buf.write('LH:${lines.where((k) => v[k] > 0).length}\n');
5959
buf.write('end_of_record\n');

0 commit comments

Comments
 (0)