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
[web] Cleanup assertionsEnabled #41829
Conversation
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.
// TODO: @yjbanov what's the purpose of this? It's always true. | ||
assert(refCount == debugReferrers.length); |
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.
@yjbanov, a question for you :)
During construction, refCount
is initialized to 1
and debugReferrers
is initialized to an empty set then one item is added.
2655629
to
f759f87
Compare
f759f87
to
8f16fcf
Compare
For consistency with the rest of the flutter code, this PR explores the possibility of removing our proprietary
assertionsEnabled
.