Skip to content

Commit

Permalink
[video_player] Call super.dispose last in README example (flutter#6051
Browse files Browse the repository at this point in the history
)

As per https://api.flutter.dev/flutter/widgets/State/dispose.html

> Implementations of this method should end with a call to the inherited method, as in super.dispose().
  • Loading branch information
tp authored and TecHaxter committed May 22, 2024
1 parent 1ec2e34 commit 37e4d12
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 3 deletions.
4 changes: 4 additions & 0 deletions packages/video_player/video_player/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
## 2.8.5

* Updates example to call `super.dispose()` last.

## 2.8.4

* Removes `_ambiguate` methods from code.
Expand Down
2 changes: 1 addition & 1 deletion packages/video_player/video_player/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -118,8 +118,8 @@ class _VideoAppState extends State<VideoApp> {
@override
void dispose() {
super.dispose();
_controller.dispose();
super.dispose();
}
}
```
Expand Down
2 changes: 1 addition & 1 deletion packages/video_player/video_player/example/lib/basic.dart
Original file line number Diff line number Diff line change
Expand Up @@ -66,8 +66,8 @@ class _VideoAppState extends State<VideoApp> {

@override
void dispose() {
super.dispose();
_controller.dispose();
super.dispose();
}
}
// #enddocregion basic-example
2 changes: 1 addition & 1 deletion packages/video_player/video_player/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ description: Flutter plugin for displaying inline video with other Flutter
widgets on Android, iOS, and web.
repository: https://github.com/flutter/packages/tree/main/packages/video_player/video_player
issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+video_player%22
version: 2.8.4
version: 2.8.5

environment:
sdk: ">=3.1.0 <4.0.0"
Expand Down

0 comments on commit 37e4d12

Please sign in to comment.