[go: nahoru, domu]
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.

Sign up for GitHub

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

Jump to bottom

Analyzer warns about missing return value in sync*/async* functions #22411

Closed
sigurdm opened this issue Feb 13, 2015 · 3 comments
Closed

Analyzer warns about missing return value in sync*/async* functions #22411

sigurdm opened this issue Feb 13, 2015 · 3 comments
Labels
area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. closed-invalid Closed as we don't believe the reported issue is generally actionable P1 A high priority bug; for example, a single project is unusable or has many test failures
Milestone
1.9

Comments

@sigurdm
Copy link
Contributor
sigurdm commented Feb 13, 2015

python tools/test.py -mrelease -cdartanalyzer -rnone -t240 language/asyncstar_yield_test

Minimal reproduction:

Iterable main() sync* {
  return;
}
@bwilkerson
Copy link
Member

Added this to the 1.9 milestone.
Removed Priority-Unassigned label.
Added Priority-High label.

@stereotype441
Copy link
Member

Is it possible that you forgot to build the Java-based analyzer code? AFAICT this test passes, and marking it as "StaticWarning" in the status file is contributing to an analyzer bot failure (see http://build.chromium.org/p/client.dart/builders/new_analyzer-linux-release-be/builds/11465/steps/tests%20/logs/stdio)

@stereotype441
Copy link
Member

Added Invalid label.

@sigurdm sigurdm added Type-Defect P1 A high priority bug; for example, a single project is unusable or has many test failures area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. closed-invalid Closed as we don't believe the reported issue is generally actionable labels Feb 13, 2015
@sigurdm sigurdm added this to the 1.9 milestone Feb 13, 2015
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. closed-invalid Closed as we don't believe the reported issue is generally actionable P1 A high priority bug; for example, a single project is unusable or has many test failures
Projects
None yet
Milestone
1.9
Development

No branches or pull requests
3 participants
@stereotype441 @bwilkerson @sigurdm