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

Side by Side Diff: test/whitespace/for.stmt

Issue 815423003: Don't put spaces in empty for loop clauses. (Closed) Base URL: https://github.com/dart-lang/dart_style.git@master
Patch Set: Created 5 years, 11 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 unified diff | Download patch
« no previous file with comments | « test/regression/132.stmt ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 40 columns | 1 40 columns |
2 >>> DO place spaces around in, and after each ; in a loop. 2 >>> DO place spaces around in, and after each ; in a loop.
3 for (var i = 0;i<100;i++) { 3 for (var i = 0;i<100;i++) {
4 print(i); 4 print(i);
5 } 5 }
6 <<< 6 <<<
7 for (var i = 0; i < 100; i++) { 7 for (var i = 0; i < 100; i++) {
8 print(i); 8 print(i);
9 } 9 }
10 >>> empty clauses TODO(rnystrom): Is this what we want? 10 >>> empty clauses
11 for(;;); 11 for(;;);
12 <<< 12 <<<
13 for ( ; ; ); 13 for (;;);
14 >>> empty initializer clause
15 for ( ; foo; bar) {}
16 <<<
17 for (; foo; bar) {}
14 >>> whitespace 18 >>> whitespace
15 for (var file in files ) { 19 for (var file in files ) {
16 print(file); 20 print(file);
17 } 21 }
18 <<< 22 <<<
19 for (var file in files) { 23 for (var file in files) {
20 print(file); 24 print(file);
21 } 25 }
22 >>> for-in with type 26 >>> for-in with type
23 for (Foo foo in bar.foos) { 27 for (Foo foo in bar.foos) {
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
62 print(a); 66 print(a);
63 } 67 }
64 >>> 68 >>>
65 for (x in xs) { 69 for (x in xs) {
66 print(x); 70 print(x);
67 } 71 }
68 <<< 72 <<<
69 for (x in xs) { 73 for (x in xs) {
70 print(x); 74 print(x);
71 } 75 }
OLDNEW
« no previous file with comments | « test/regression/132.stmt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698