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

Analyzer should support metadata annotation syntax #4056

Closed
gbracha opened this issue Jul 11, 2012 · 8 comments
Closed

Analyzer should support metadata annotation syntax #4056

gbracha opened this issue Jul 11, 2012 · 8 comments
Assignees
Milestone

Comments

@gbracha
Copy link
Contributor

gbracha commented Jul 11, 2012

Per issue #3214.

@bwilkerson
Copy link
Member

Added Waiting label.

@bwilkerson
Copy link
Member

Added Triaged label.

@bwilkerson
Copy link
Member

Set owner to @bwilkerson.

@bwilkerson
Copy link
Member

http://codereview.chromium.org/10860012/


Added Fixed label.

@kasperl
Copy link

kasperl commented Sep 14, 2012

Can you guys verify that you allow metadata annotations in all the right places? Someone mentioned in passing that the implementation you have isn't complete (sorry that I do not have more details).

@bwilkerson
Copy link
Member

Yes, unless the grammar has changed since we implemented this feature, we are accepting metadata in all of the places specified.

Until earlier this week our metadata support was incomplete in that we were not recording the parsed information in the element model and hence not using it to ensure correctness, but that has been fixed for the two kinds of metadata defined in the "meta" package: @­override and @­deprecated.

@gbracha
Copy link
Contributor Author

gbracha commented Sep 14, 2012

I'm not sure about timelines, but metadata on type variables is one thing
that might have come in a tad later, so please check that.

@bwilkerson
Copy link
Member

Confirmed that metadata on type variables was indeed added after we implemented the feature (unless we just missed it the first time around). It has now been added.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants