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

Side by Side Diff: test/splitting/mixed.stmt

Issue 1258203006: Handle function arguments inside function calls. (Closed) Base URL: https://github.com/dart-lang/dart_style.git@master
Patch Set: Reformat self. Created 5 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/splitting/function_arguments.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 >>> keeps map on one line if possible 2 >>> keeps map on one line if possible
3 sendPort.send({'type': 'error', 'error': 'oops'}); 3 sendPort.send({'type': 'error', 'error': 'oops'});
4 <<< 4 <<<
5 sendPort.send( 5 sendPort.send(
6 {'type': 'error', 'error': 'oops'}); 6 {'type': 'error', 'error': 'oops'});
7 >>> prefers to wrap before "." 7 >>> prefers to wrap before "."
8 new Future.sync(() => callback('msg')).then( 8 new Future.sync(() => callback('msg')).then(
9 (result) => replyTo.send()).catchError((error) {}); 9 (result) => replyTo.send()).catchError((error) {});
10 <<< 10 <<<
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 argument, 64 argument,
65 obj 65 obj
66 .method(argument) 66 .method(argument)
67 .method(argument) 67 .method(argument)
68 .method(() { 68 .method(() {
69 body; 69 body;
70 }).another().another()); 70 }).another().another());
71 >>> unnested function inside nested expression 71 >>> unnested function inside nested expression
72 function(argument, function(() {;})); 72 function(argument, function(() {;}));
73 <<< 73 <<<
74 function( 74 function(argument, function(() {
75 argument, 75 ;
76 function(() { 76 }));
77 ;
78 }));
79 >>> nested function inside nested expression 77 >>> nested function inside nested expression
80 function(argument, function(() {;}, argument, () {;})); 78 function(argument, function(() {;}, argument, () {;}));
81 <<< 79 <<<
82 function( 80 function(
83 argument, 81 argument,
84 function( 82 function(
85 () { 83 () {
86 ; 84 ;
87 }, 85 },
88 argument, 86 argument,
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
189 veryLongIdentifier - 187 veryLongIdentifier -
190 veryLongIdentifier; 188 veryLongIdentifier;
191 >>> choose extra nesting if it leads to better solution 189 >>> choose extra nesting if it leads to better solution
192 longIdentifier + 190 longIdentifier +
193 (longIdentifier ? 0 : 191 (longIdentifier ? 0 :
194 1) == identifier; 192 1) == identifier;
195 <<< 193 <<<
196 longIdentifier + 194 longIdentifier +
197 (longIdentifier ? 0 : 1) == 195 (longIdentifier ? 0 : 1) ==
198 identifier; 196 identifier;
OLDNEW
« no previous file with comments | « test/splitting/function_arguments.stmt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698