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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: tests/language/return_type_test.dart

Issue 2718513002: Void is not required to be `null` anymore. (Closed)
Patch Set: Fix more places in the VM. Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« runtime/vm/object.cc ('K') | « sdk/lib/_internal/js_runtime/lib/js_helper.dart ('k') | tests/language/void_check_test.dart » ('j') | tests/language/void_check_test.dart » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/language/return_type_test.dart
diff --git a/tests/language/return_type_test.dart b/tests/language/return_type_test.dart
index a2137bb0ca402efab9181559f62d2daffbf26d2b..1346e415cea5940c172d06b140385c24cd72d709 100644
--- a/tests/language/return_type_test.dart
+++ b/tests/language/return_type_test.dart
@@ -21,7 +21,7 @@ void returnString2() => 's';
main() {
if (isCheckedMode()) {
Expect.throws(returnString1, (e) => e is TypeError);
- Expect.throws(returnString2, (e) => e is TypeError);
+ returnString2();
returnNull();
} else {
returnString1();
« runtime/vm/object.cc ('K') | « sdk/lib/_internal/js_runtime/lib/js_helper.dart ('k') | tests/language/void_check_test.dart » ('j') | tests/language/void_check_test.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698