Skip to content

Generalize assert syntax in analyzer #26865

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
eernstg opened this issue Jul 12, 2016 · 1 comment
Closed

Generalize assert syntax in analyzer #26865

eernstg opened this issue Jul 12, 2016 · 1 comment
Assignees
Labels
legacy-area-analyzer Use area-devexp instead. P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@eernstg
Copy link
Member

eernstg commented Jul 12, 2016

With reference to #17682 for background, the analyzer should be adjusted such that it accepts arguments to assert parsed as expression, rather than requiring the more restrictive conditionalExpression.

The language team discussed this issue and came up with an analysis concluding that we will make this change: https://gist.github.com/eernstg/50a327addd96035d5393a507c0ea2d9a.

@eernstg eernstg added the legacy-area-analyzer Use area-devexp instead. label Jul 12, 2016
@bwilkerson bwilkerson added P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) labels Jul 12, 2016
@bwilkerson bwilkerson self-assigned this Jul 12, 2016
@bwilkerson
Copy link
Member

@munificent munificent changed the title Generalize assert syntax Generalize assert syntax in analyzer Dec 15, 2016
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. P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Development

No branches or pull requests

2 participants