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

Editor could warn if a function is annotated with a return type but does not have explicit return #7418

Closed
sethladd opened this issue Dec 14, 2012 · 4 comments
Assignees
Labels
closed-duplicate Closed in favor of an existing report type-enhancement A request for a change that isn't a bug
Milestone

Comments

@sethladd
Copy link
Contributor

It would be extremely helpful to get warned if I do this (which, I've done :(

int computeIt(int foo) {
  foo += 1;
  // no explicit return !
}

So, if I take the time to add a type annotation for the return type of a function or method, but do not include an explicit return statement, I'd really appreciate a warning or a "head's up!"

Thanks!

(see also issue #73)

@clayberg
Copy link

Set owner to @scheglov.
Added this to the M3 milestone.

@danrubel
Copy link

Should be addressed when we move to the new analysis engine


Removed this from the M3 milestone.
Added this to the Later milestone.

@clayberg
Copy link

Added Duplicate label.
Marked as being merged into #4919.

@clayberg
Copy link

Removed this from the Later milestone.
Added this to the M3 milestone.

@sethladd sethladd added Type-Enhancement closed-duplicate Closed in favor of an existing report labels Jan 14, 2013
@sethladd sethladd added this to the M3 milestone Jan 14, 2013
@clayberg clayberg mentioned this issue Jun 3, 2015
@kevmoo kevmoo added type-enhancement A request for a change that isn't a bug and removed type-enhancement labels Mar 1, 2016
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-duplicate Closed in favor of an existing report type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

5 participants