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

Side by Side Diff: test/comments/expressions.stmt

Issue 1183633004: Harden named arguments if a hard split occurs in the positional ones. (Closed) Base URL: https://github.com/dart-lang/dart_style.git@master
Patch Set: Update changelog. Created 5 years, 6 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 | « lib/src/chunk_builder.dart ('k') | test/regression/189.stmt » ('j') | 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 >>> trailing line comment after split 2 >>> trailing line comment after split
3 someMethod(argument1, argument2, // comment 3 someMethod(argument1, argument2, // comment
4 argument3); 4 argument3);
5 <<< 5 <<<
6 someMethod( 6 someMethod(
7 argument1, 7 argument1,
8 argument2, // comment 8 argument2, // comment
9 argument3); 9 argument3);
10 >>> trailing line comment after non-split 10 >>> trailing line comment after non-split
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
69 .clamp(); 69 .clamp();
70 >>> line comment before binary operator 70 >>> line comment before binary operator
71 { 71 {
72 // comment 72 // comment
73 1 + 2; 73 1 + 2;
74 } 74 }
75 <<< 75 <<<
76 { 76 {
77 // comment 77 // comment
78 1 + 2; 78 1 + 2;
79 } 79 }
80 >>> force named args to split on line comment in positional
81 function(argument, //
82 argument, named: argument, another: argument);
83 <<<
84 function(
85 argument, //
86 argument,
87 named: argument,
88 another: argument);
OLDNEW
« no previous file with comments | « lib/src/chunk_builder.dart ('k') | test/regression/189.stmt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698