Skip to content

Analyzer should implement const variables #3550

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

Closed
gbracha opened this issue Jun 12, 2012 · 8 comments
Closed

Analyzer should implement const variables #3550

gbracha opened this issue Jun 12, 2012 · 8 comments
Assignees
Labels
legacy-area-analyzer Use area-devexp instead.
Milestone

Comments

@gbracha
Copy link
Contributor

gbracha commented Jun 12, 2012

Per spec section 5. See issue #3549. This does not include treating finals statics as non-constants - that would be a separate bug.

@gbracha
Copy link
Contributor Author

gbracha commented Jun 12, 2012

Added this to the M1 milestone.

@scheglov
Copy link
Contributor

Set owner to @scheglov.
Added Accepted label.

@anders-sandholm
Copy link
Contributor

Issue #3323 has been merged into this issue.

@scheglov
Copy link
Contributor

@scheglov
Copy link
Contributor

@johnniwinther
Copy link
Member

The test tests/language/const_var_test.dart is not handled.


Added Triaged label.

@scheglov
Copy link
Contributor

It looks as parser problem.

We need to support:
const v = 1;

and also somehow distinguish it from:
const A();


Set owner to @bwilkerson.

@bwilkerson
Copy link
Member

http://codereview.chromium.org/10857041/


Added Fixed label.

@gbracha gbracha added Type-Defect legacy-area-analyzer Use area-devexp instead. labels Aug 16, 2012
@gbracha gbracha added this to the M1 milestone Aug 16, 2012
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
legacy-area-analyzer Use area-devexp instead.
Projects
None yet
Development

No branches or pull requests

5 participants