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

Unified Diff: tests/language/switch_case_warn_test.dart

Issue 2774783002: Re-land "Format all multitests" (Closed)
Patch Set: Created 3 years, 9 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
« no previous file with comments | « tests/language/switch_case_test.dart ('k') | tests/language/symbol_literal_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/language/switch_case_warn_test.dart
diff --git a/tests/language/switch_case_warn_test.dart b/tests/language/switch_case_warn_test.dart
index eb68bb01eb1a1a3481b04016b635a5865da69ebe..fe6d71016fdf7e921e648399e54d677189c6a83b 100644
--- a/tests/language/switch_case_warn_test.dart
+++ b/tests/language/switch_case_warn_test.dart
@@ -11,22 +11,24 @@
void testSwitch(int x) {
// Catch all control flow leaving the switch.
// Run switch in catch clause to check rethrow.
- TRY: try {
+ TRY:
+ try {
throw x;
} catch (x) {
// Add loop as break/continue target.
- LOOP: do {
+ LOOP:
+ do {
switch (x) {
case 0:
case 1:
nop(x);
- break; // Break switch.
+ break; // Break switch.
case 2:
nop(x);
break LOOP;
case 3:
nop(x);
- continue; // Continue loop.
+ continue; // Continue loop.
case 4:
nop(x);
continue LOOP;
@@ -47,26 +49,31 @@ void testSwitch(int x) {
nop(x);
rethrow;
case 10:
- case 11: {
- nop(x);
- break; // Break switch.
- }
- case 12: {
- nop(x);
- break LOOP;
- }
- case 13: {
- nop(x);
- continue; // Continue loop.
- }
- case 14: {
- nop(x);
- continue LOOP;
- }
- case 15: {
- nop(x);
- continue LAST;
- }
+ case 11:
+ {
+ nop(x);
+ break; // Break switch.
+ }
+ case 12:
+ {
+ nop(x);
+ break LOOP;
+ }
+ case 13:
+ {
+ nop(x);
+ continue; // Continue loop.
+ }
+ case 14:
+ {
+ nop(x);
+ continue LOOP;
+ }
+ case 15:
+ {
+ nop(x);
+ continue LAST;
+ }
case 16: { // //# retnon: continued
nop(x); // //# retnon: continued
return; // //# retnon: continued
@@ -75,19 +82,22 @@ void testSwitch(int x) {
nop(x); // //# retval: continued
return x; // //# retval: continued
} // //# retval: continued
- case 18: {
- nop(x);
- throw x;
- }
- case 19: {
- nop(x);
- rethrow;
- }
+ case 18:
+ {
+ nop(x);
+ throw x;
+ }
+ case 19:
+ {
+ nop(x);
+ rethrow;
+ }
LAST:
- case 20: {
- nop(x);
- // Fallthrough allowed on last statements.
- }
+ case 20:
+ {
+ nop(x);
+ // Fallthrough allowed on last statements.
+ }
}
} while (false);
} finally {
@@ -101,11 +111,13 @@ void testSwitch(int x) {
// All these switch cases should cause warnings.
void testSwitchWarn(x) {
// Catch all control flow from the switch and ignore it.
- TRY: try {
+ TRY:
+ try {
throw 0;
} catch (e) {
// Wrap in loop as target for continue/break.
- LOOP: do {
+ LOOP:
+ do {
switch (x) {
case 0: // //# 01: static type warning
case 1: { // //# 01: continued
@@ -180,7 +192,7 @@ void testSwitchWarn(x) {
L: break; // //# 17: continued
LAST:
case 99:
- // Last case can't cause static warning.
+ // Last case can't cause static warning.
}
} while (false);
} finally {
@@ -194,7 +206,7 @@ void testSwitchWarn(x) {
main() {
// Ensure that all the cases compile and run (even if they might throw).
for (int i = 0; i <= 20; i++) {
- testSwitch(i); // Just make sure it runs.
+ testSwitch(i); // Just make sure it runs.
}
for (int i = 0; i <= 18; i++) {
testSwitchWarn(i);
« no previous file with comments | « tests/language/switch_case_test.dart ('k') | tests/language/symbol_literal_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698