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
HintCode.MISSING_RETURN should include factory constructors as well. #17121
Comments
Yes, it does seem reasonable. Added this to the 1.3 milestone. |
This is for the hint, not warning. Bumping priority to later. HintCode.MISSING_RETURN should be created on factory constructors as well as on methods and functions. Source source = addSource(createSource(// cc @bwilkerson. |
Changed the title to: "[Hint] HintCode.MISSING_RETURN should include factory constructors as well.". |
Added Analyzer-Hint label. |
Changed the title to: "HintCode.MISSING_RETURN should include factory constructors as well.". |
Removed this from the Later milestone. |
Removed Oldschool-Milestone-Later label. |
BUG=#17121 R=brianwilkerson@google.com, scheglov@google.com Review URL: https://codereview.chromium.org/1923593002 .
Done in 7cb775f. |
Create an empty method with a return type defined, but no return statement: you get a warning.
Create an empty factory ctor with no return statement: nothing.
Seems reasonable.
r32954
The text was updated successfully, but these errors were encountered: