Skip to content

Commit ffeeac3

Browse files
authored
Clean up some optional Dart lints (flutter#34557)
1 parent 1b3d273 commit ffeeac3

File tree

7 files changed

+27
-24
lines changed

7 files changed

+27
-24
lines changed

testing/dart/assets_test.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44

55
import 'dart:async';
66
import 'dart:convert';
7-
import 'dart:io';
87
import 'dart:typed_data';
98
import 'dart:ui';
109

testing/dart/codec_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ void main() {
8787
});
8888

8989
test('disposed decoded image', () async {
90-
Uint8List data = await _getSkiaResource('flutter_logo.jpg').readAsBytes();
90+
final Uint8List data = await _getSkiaResource('flutter_logo.jpg').readAsBytes();
9191
final ui.Codec codec = await ui.instantiateImageCodec(data);
9292
final ui.FrameInfo frameInfo = await codec.getNextFrame();
9393
expect(frameInfo.image, isNotNull);

testing/dart/compositing_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
import 'dart:typed_data' show Float64List, Uint32List, ByteData;
5+
import 'dart:typed_data' show Float64List, ByteData;
66
import 'dart:ui';
77

88
import 'package:litetest/litetest.dart';

testing/dart/image_shader_test.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
import 'dart:async';
65
import 'dart:typed_data';
76
import 'dart:ui';
87

testing/dart/paragraph_test.dart

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -70,36 +70,36 @@ void main() {
7070
));
7171
builder.addText('Test Ahem');
7272
final Paragraph paragraph = builder.build();
73-
paragraph.layout(ParagraphConstraints(width: fontSize * 5.0));
73+
paragraph.layout(const ParagraphConstraints(width: fontSize * 5.0));
7474

7575
// Wraps to two lines.
7676
expect(paragraph.height, closeTo(fontSize * 2.0, 0.001));
7777

78-
final TextPosition wrapPositionDown = TextPosition(
78+
const TextPosition wrapPositionDown = TextPosition(
7979
offset: 5,
8080
affinity: TextAffinity.downstream,
8181
);
8282
TextRange line = paragraph.getLineBoundary(wrapPositionDown);
8383
expect(line.start, 5);
8484
expect(line.end, 9);
8585

86-
final TextPosition wrapPositionUp = TextPosition(
86+
const TextPosition wrapPositionUp = TextPosition(
8787
offset: 5,
8888
affinity: TextAffinity.upstream,
8989
);
9090
line = paragraph.getLineBoundary(wrapPositionUp);
9191
expect(line.start, 0);
9292
expect(line.end, 5);
9393

94-
final TextPosition wrapPositionStart = TextPosition(
94+
const TextPosition wrapPositionStart = TextPosition(
9595
offset: 0,
9696
affinity: TextAffinity.downstream,
9797
);
9898
line = paragraph.getLineBoundary(wrapPositionStart);
9999
expect(line.start, 0);
100100
expect(line.end, 5);
101101

102-
final TextPosition wrapPositionEnd = TextPosition(
102+
const TextPosition wrapPositionEnd = TextPosition(
103103
offset: 9,
104104
affinity: TextAffinity.downstream,
105105
);
@@ -119,36 +119,36 @@ void main() {
119119
));
120120
builder.addText('القاهرةالقاهرة');
121121
final Paragraph paragraph = builder.build();
122-
paragraph.layout(ParagraphConstraints(width: fontSize * 5.0));
122+
paragraph.layout(const ParagraphConstraints(width: fontSize * 5.0));
123123

124124
// Wraps to three lines.
125125
expect(paragraph.height, closeTo(fontSize * 3.0, 0.001));
126126

127-
final TextPosition wrapPositionDown = TextPosition(
127+
const TextPosition wrapPositionDown = TextPosition(
128128
offset: 5,
129129
affinity: TextAffinity.downstream,
130130
);
131131
TextRange line = paragraph.getLineBoundary(wrapPositionDown);
132132
expect(line.start, 5);
133133
expect(line.end, 10);
134134

135-
final TextPosition wrapPositionUp = TextPosition(
135+
const TextPosition wrapPositionUp = TextPosition(
136136
offset: 5,
137137
affinity: TextAffinity.upstream,
138138
);
139139
line = paragraph.getLineBoundary(wrapPositionUp);
140140
expect(line.start, 0);
141141
expect(line.end, 5);
142142

143-
final TextPosition wrapPositionStart = TextPosition(
143+
const TextPosition wrapPositionStart = TextPosition(
144144
offset: 0,
145145
affinity: TextAffinity.downstream,
146146
);
147147
line = paragraph.getLineBoundary(wrapPositionStart);
148148
expect(line.start, 0);
149149
expect(line.end, 5);
150150

151-
final TextPosition wrapPositionEnd = TextPosition(
151+
const TextPosition wrapPositionEnd = TextPosition(
152152
offset: 9,
153153
affinity: TextAffinity.downstream,
154154
);
@@ -168,12 +168,12 @@ void main() {
168168
));
169169
builder.addText('Test\n\nAhem');
170170
final Paragraph paragraph = builder.build();
171-
paragraph.layout(ParagraphConstraints(width: fontSize * 5.0));
171+
paragraph.layout(const ParagraphConstraints(width: fontSize * 5.0));
172172

173173
// Three lines due to line breaks, with the middle line being empty.
174174
expect(paragraph.height, closeTo(fontSize * 3.0, 0.001));
175175

176-
final TextPosition emptyLinePosition = TextPosition(
176+
const TextPosition emptyLinePosition = TextPosition(
177177
offset: 5,
178178
affinity: TextAffinity.downstream,
179179
);
@@ -182,23 +182,23 @@ void main() {
182182
expect(line.end, 5);
183183

184184
// Since these are hard newlines, TextAffinity has no effect here.
185-
final TextPosition emptyLinePositionUpstream = TextPosition(
185+
const TextPosition emptyLinePositionUpstream = TextPosition(
186186
offset: 5,
187187
affinity: TextAffinity.upstream,
188188
);
189189
line = paragraph.getLineBoundary(emptyLinePositionUpstream);
190190
expect(line.start, 5);
191191
expect(line.end, 5);
192192

193-
final TextPosition endOfFirstLinePosition = TextPosition(
193+
const TextPosition endOfFirstLinePosition = TextPosition(
194194
offset: 4,
195195
affinity: TextAffinity.downstream,
196196
);
197197
line = paragraph.getLineBoundary(endOfFirstLinePosition);
198198
expect(line.start, 0);
199199
expect(line.end, 4);
200200

201-
final TextPosition startOfLastLinePosition = TextPosition(
201+
const TextPosition startOfLastLinePosition = TextPosition(
202202
offset: 6,
203203
affinity: TextAffinity.downstream,
204204
);

testing/dart/serial_gc_test.dart

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,16 @@
66

77
import 'package:litetest/litetest.dart';
88

9+
int use(List<int> a) {
10+
return a[0];
11+
}
12+
913
void main() {
1014
test('Serial GC option test ', () async {
11-
bool threw = false;
15+
const bool threw = false;
1216
for (int i = 0; i < 100; i++) {
13-
var a = <int>[100];
17+
final List<int> a = <int>[100];
18+
use(a);
1419
}
1520
expect(threw, false);
1621
});

testing/dart/text_test.dart

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -252,7 +252,7 @@ void testFontVariation() {
252252
));
253253
baseBuilder.addText('Hello');
254254
final Paragraph baseParagraph = baseBuilder.build();
255-
baseParagraph.layout(ParagraphConstraints(width: double.infinity));
255+
baseParagraph.layout(const ParagraphConstraints(width: double.infinity));
256256
final double baseWidth = baseParagraph.minIntrinsicWidth;
257257

258258
final ParagraphBuilder wideBuilder = ParagraphBuilder(ParagraphStyle(
@@ -262,11 +262,11 @@ void testFontVariation() {
262262
wideBuilder.pushStyle(TextStyle(
263263
fontFamily: 'RobotoSerif',
264264
fontSize: 40.0,
265-
fontVariations: <FontVariation>[FontVariation('wght', 900.0)]
265+
fontVariations: <FontVariation>[const FontVariation('wght', 900.0)],
266266
));
267267
wideBuilder.addText('Hello');
268268
final Paragraph wideParagraph = wideBuilder.build();
269-
wideParagraph.layout(ParagraphConstraints(width: double.infinity));
269+
wideParagraph.layout(const ParagraphConstraints(width: double.infinity));
270270
final double wideWidth = wideParagraph.minIntrinsicWidth;
271271

272272
expect(wideWidth, greaterThan(baseWidth));

0 commit comments

Comments
 (0)