-
Notifications
You must be signed in to change notification settings - Fork 23
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
Stop failing on double finalization #224
Conversation
// Normally it should not happen, but sometimes finalizer is called twice. | ||
// To repro, update next line to throw exception and run flutter tests with | ||
// the updated leak_tracker. | ||
if (_gcedGcCount != null) return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a TODO with a bug tracking this. The Dart VM really should not be calling finalizers twice.
Do we have a standalone repro so that @rmacnak-google and others on the Dart team can reproduce?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done: dart-lang/sdk#55330
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exception in flutter tests: