Skip to content

Commit 7afa4a5

Browse files
Replace Dart Code Metrics with Solid Lints
1 parent 4024950 commit 7afa4a5

File tree

5 files changed

+18
-51
lines changed

5 files changed

+18
-51
lines changed

.github/workflows/code_check_for_flutter_vlc_player.yaml

Lines changed: 2 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -20,30 +20,10 @@ jobs:
2020
working-directory: ./flutter_vlc_player
2121
run: flutter pub get
2222

23-
- name: Setup Dart Code Metrics
23+
- name: Run Solid Lints
2424
working-directory: ./flutter_vlc_player
25-
run: dart pub get dart_code_metrics
25+
run: dart analyze
2626

27-
- name: Dart Code Metrics
28-
working-directory: ./flutter_vlc_player
29-
run: |
30-
dirs_to_analyze=""
31-
if [ -d lib ]; then dirs_to_analyze+=" lib"; fi
32-
if [ -d test ]; then dirs_to_analyze+=" test"; fi
33-
if [ -d example ]; then dirs_to_analyze+=" example"; fi
34-
if [ dirs_to_analyze != "" ]
35-
then
36-
dart run dart_code_metrics:metrics \
37-
analyze \
38-
$dirs_to_analyze \
39-
--fatal-warnings \
40-
--fatal-performance \
41-
--fatal-style
42-
dart run dart_code_metrics:metrics \
43-
check-unused-files \
44-
$dirs_to_analyze \
45-
--fatal-unused
46-
fi
4727
- name: Check formatting
4828
run: dart format . --set-exit-if-changed
4929

.github/workflows/code_check_for_flutter_vlc_player_platform_interface.yaml

Lines changed: 2 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -21,30 +21,10 @@ jobs:
2121
working-directory: ./flutter_vlc_player_platform_interface
2222
run: flutter pub get
2323

24-
- name: Setup Dart Code Metrics
24+
- name: Run Solid Lints
2525
working-directory: ./flutter_vlc_player_platform_interface
26-
run: dart pub get dart_code_metrics
26+
run: dart analyze
2727

28-
- name: Dart Code Metrics
29-
working-directory: ./flutter_vlc_player_platform_interface
30-
run: |
31-
dirs_to_analyze=""
32-
if [ -d lib ]; then dirs_to_analyze+=" lib"; fi
33-
if [ -d test ]; then dirs_to_analyze+=" test"; fi
34-
if [ -d example ]; then dirs_to_analyze+=" example"; fi
35-
if [ dirs_to_analyze != "" ]
36-
then
37-
dart run dart_code_metrics:metrics \
38-
analyze \
39-
$dirs_to_analyze \
40-
--fatal-warnings \
41-
--fatal-performance \
42-
--fatal-style
43-
dart run dart_code_metrics:metrics \
44-
check-unused-files \
45-
$dirs_to_analyze \
46-
--fatal-unused
47-
fi
4828
- name: Check formatting
4929
working-directory: ./flutter_vlc_player_platform_interface
5030
run: dart format . --set-exit-if-changed

flutter_vlc_player/example/lib/single_tab.dart

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,7 @@ class _SingleTabState extends State<SingleTab> {
219219
await Future<void>.delayed(const Duration(seconds: 1));
220220
final tempVideo = await _loadVideoToFs();
221221
await Future<void>.delayed(const Duration(seconds: 1));
222-
if (!mounted) break;
222+
if (!context.mounted) break;
223223
ScaffoldMessenger.of(context).showSnackBar(
224224
const SnackBar(
225225
content: Text('Now trying to play...'),
@@ -229,7 +229,7 @@ class _SingleTabState extends State<SingleTab> {
229229
if (await tempVideo.exists()) {
230230
await _controller.setMediaFromFile(tempVideo);
231231
} else {
232-
if (!mounted) break;
232+
if (!context.mounted) break;
233233
ScaffoldMessenger.of(context).showSnackBar(
234234
const SnackBar(
235235
content: Text('File load error.'),
@@ -264,3 +264,9 @@ class _SingleTabState extends State<SingleTab> {
264264
await _controller.dispose();
265265
}
266266
}
267+
268+
class MyClass {
269+
final int _myField = 0;
270+
271+
int get myField => _myField;
272+
}

flutter_vlc_player/lib/src/vlc_player_controller.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -572,7 +572,8 @@ class VlcPlayerController extends ValueNotifier<VlcPlayerValue> {
572572
_throwIfNotInitialized('getVolume');
573573
final volume = await vlcPlayerPlatform.getVolume(_viewId);
574574
value = value.copyWith(
575-
volume: volume != null ? volume.clamp(0, _maxVolume) : null);
575+
volume: volume?.clamp(0, _maxVolume),
576+
);
576577

577578
return volume;
578579
}

flutter_vlc_player/pigeons/messages.dart

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ class SnapshotMessage {
7171

7272
class SpuTracksMessage {
7373
int? viewId;
74-
Map? subtitles;
74+
Map<Object?, Object?>? subtitles;
7575
}
7676

7777
class SpuTrackMessage {
@@ -88,7 +88,7 @@ class AddSubtitleMessage {
8888

8989
class AudioTracksMessage {
9090
int? viewId;
91-
Map? audios;
91+
Map<Object?, Object?>? audios;
9292
}
9393

9494
class AudioTrackMessage {
@@ -105,7 +105,7 @@ class AddAudioMessage {
105105

106106
class VideoTracksMessage {
107107
int? viewId;
108-
Map? videos;
108+
Map<Object?, Object?>? videos;
109109
}
110110

111111
class VideoTrackMessage {
@@ -135,7 +135,7 @@ class RendererScanningMessage {
135135

136136
class RendererDevicesMessage {
137137
int? viewId;
138-
Map? rendererDevices;
138+
Map<Object?, Object?>? rendererDevices;
139139
}
140140

141141
class RenderDeviceMessage {

0 commit comments

Comments
 (0)