Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Call super.dispose() at the end of dispose() in stanim snippet #3965

Merged
merged 1 commit into from
May 17, 2022

Conversation

nilsreichardt
Copy link
Contributor

From the Flutter docs of the dispose method:

Implementations of this method should end with a call to the inherited method, as in super.dispose().

Therefore, we should call super.dispose() at the end.

@DanTup
Copy link
Member

DanTup commented May 17, 2022

Thanks!

With newer versions of the extension/SDKs, the snippets come from the server so I've applied the same fix there at https://dart-review.googlesource.com/c/sdk/+/245003/.

@DanTup DanTup added this to the v3.42.0 milestone May 17, 2022
@DanTup DanTup added is bug fix in flutter Relates to running Flutter apps in editor Relates to code editing or language features labels May 17, 2022
@DanTup DanTup merged commit 3ee1cc4 into Dart-Code:master May 17, 2022
@nilsreichardt
Copy link
Contributor Author

Great, thanks!

copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request May 17, 2022
…pose method

Equiv change for extension-shipped snippet in Dart-Code/Dart-Code#3965.

Change-Id: Ief61a7d11f19edac6f4f4a14f311d7789add8d80
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/245003
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in editor Relates to code editing or language features in flutter Relates to running Flutter apps is bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants