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

Fix public mixins template text in category sidebar #2613

Merged
merged 1 commit into from
Apr 12, 2021

Conversation

srawlins
Copy link
Member

As neither Category nor CategoryTemplateData have a member called library, I'm pretty sure this code is a runtime error waiting to happen.

It must have never been reported yet because no one (including flutter) has a @category with public mixins? I'm honestly not sure.

@google-cla google-cla bot added the cla: yes Google CLA check succeeded. label Apr 12, 2021
@jcollins-g
Copy link
Contributor

Category usage in the wild is pretty rare, so it seems possible that noone has done this, yes. Good catch!

@srawlins srawlins merged commit cf09214 into dart-lang:master Apr 12, 2021
@srawlins srawlins deleted the category-library branch April 12, 2021 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Google CLA check succeeded.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants