-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Strong mode should infer parameter types from default value expressions #25794
Comments
Nice, yeah, we probably just forgot about this AST node. |
Doh, I just hit this while implementing/testing #25573 (no-implicit-dynamic). I may roll in a fix or look at it concurrently. |
update, fixed it in resolver & element summaries, still haven't got it working in AST-based summaries yet. |
https://codereview.chromium.org/2092333002/ -- but I might be way off on that approach :) |
Revert CL up, because this regressed cyclic_default_values https://codereview.chromium.org/2109353004/ It seems like this inference will need to be implemented as part of the top-level field inference, if we want to handle top-level/static functions correctly. |
I'm guessing we do not plan to change the language inference here at this point. closing |
Currently, strong mode does not infer the type of the parameter
x
in the code below:It seems like it would be reasonable to do so, since it would be similar to the way it infers the type of variables from their initializer expressions.
The text was updated successfully, but these errors were encountered: