Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(346)

Issue 2870063002: Allowing `e` of any type in `void f(...) => e` (Closed)

Created:
3 years, 7 months ago by eernst
Modified:
3 years, 7 months ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org, floitsch
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : New test #

Patch Set 3 : There was no test for the block case, added that #

Unified diffs Side-by-side diffs Delta from patch set Stats (+33 lines, -2 lines) Patch
M pkg/analyzer/lib/src/generated/error_verifier.dart View 3 chunks +8 lines, -2 lines 0 comments Download
A tests/language/void_arrow_return_test.dart View 1 2 1 chunk +11 lines, -0 lines 0 comments Download
A tests/language/void_block_return_test.dart View 1 2 1 chunk +14 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
eernst
To address SDK issue #28940, this CL allows all types of `e` in an `=>` ...
3 years, 7 months ago (2017-05-09 13:33:08 UTC) #2
eernst
Completed more tests: Test configurations: dart2analyzer_none_release_x64 dart2analyzer_none_release_x64 [01:22 | 100% | +18427 | - 0].
3 years, 7 months ago (2017-05-09 13:51:56 UTC) #3
Brian Wilkerson
lgtm
3 years, 7 months ago (2017-05-09 15:33:11 UTC) #5
eernst
3 years, 7 months ago (2017-05-10 09:37:15 UTC) #7
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
025b83fb892e02bd8465d83bd51c5d1fb3b21235 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698