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

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

Issue 2186343003: Enforce a blank line after local function declarations. (Closed) Base URL: https://github.com/dart-lang/dart_style.git@master
Patch Set: Revise. Created 4 years, 4 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/0400/0488.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 >>> require at least one newline between statements 2 >>> require at least one newline between statements
3 {a();b();c();} 3 {a();b();c();}
4 <<< 4 <<<
5 { 5 {
6 a(); 6 a();
7 b(); 7 b();
8 c(); 8 c();
9 } 9 }
10 >>> allow an extra newline between statements 10 >>> allow an extra newline between statements
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 } 47 }
48 >>> dartbug.com/16810 48 >>> dartbug.com/16810
49 { 49 {
50 var a;;; 50 var a;;;
51 } 51 }
52 <<< 52 <<<
53 { 53 {
54 var a; 54 var a;
55 ; 55 ;
56 ; 56 ;
57 }
58 >>> force blank line after non-empty local function
59 {
60 a() {;}
61 b();
62
63
64 c() {;}d(){;}
65
66
67 }
68 <<<
69 {
70 a() {
71 ;
72 }
73
74 b();
75
76 c() {
77 ;
78 }
79
80 d() {
81 ;
82 }
83 }
84 >>> do not force blank line after empty local function
85 { a() {} b() {} }
86 <<<
87 {
88 a() {}
89 b() {}
90 }
91 >>> do not force blank line after => body local function
92 { a() => null; b() => null; }
93 <<<
94 {
95 a() => null;
96 b() => null;
57 } 97 }
OLDNEW
« no previous file with comments | « test/regression/0400/0488.stmt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698