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

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

Issue 846133002: Put conditional operators on the left. (Closed) Base URL: https://github.com/dart-lang/dart_style.git@master
Patch Set: Update version. 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/80.unit ('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 >>> space-separated adjacent strings are not split 2 >>> space-separated adjacent strings are not split
3 var name = new Symbol("the first string" "the second string"); 3 var name = new Symbol("the first string" "the second string");
4 <<< 4 <<<
5 var name = new Symbol( 5 var name = new Symbol(
6 "the first string" "the second string"); 6 "the first string" "the second string");
7 >>> preserve one newline between adjacent strings 7 >>> preserve one newline between adjacent strings
8 var name = "the first string" 8 var name = "the first string"
9 "the second string" 9 "the second string"
10 10
(...skipping 15 matching lines...) Expand all
26 >>> conditions, different operators 26 >>> conditions, different operators
27 if (identifier && identifier || identifier 27 if (identifier && identifier || identifier
28 && identifier) { 28 && identifier) {
29 } 29 }
30 <<< 30 <<<
31 if (identifier && identifier || 31 if (identifier && identifier ||
32 identifier && identifier) {} 32 identifier && identifier) {}
33 >>> split conditional because then doesn't fit 33 >>> split conditional because then doesn't fit
34 var kind = element != null ? longArgument : arg; 34 var kind = element != null ? longArgument : arg;
35 <<< 35 <<<
36 var kind = element != null ? 36 var kind = element != null
37 longArgument : 37 ? longArgument
38 arg; 38 : arg;
39 >>> split conditional because else doesn't fit 39 >>> split conditional because else doesn't fit
40 var kind = element != null ? argument : secondArgumentThatIsReallyLong; 40 var kind = element != null ? argument : secondArgumentThatIsReallyLong;
41 <<< 41 <<<
42 var kind = element != null ? 42 var kind = element != null
43 argument : 43 ? argument
44 secondArgumentThatIsReallyLong; 44 : secondArgumentThatIsReallyLong;
45 >>> unsplit operator chains before and after block 45 >>> unsplit operator chains before and after block
46 first + second + () {body;} + third + fourth; 46 first + second + () {body;} + third + fourth;
47 <<< 47 <<<
48 first + second + () { 48 first + second + () {
49 body; 49 body;
50 } + third + fourth; 50 } + third + fourth;
51 >>> split operator chain before block 51 >>> split operator chain before block
52 first + second + third + fourth + fifth + () {body;} + sixth + seventh; 52 first + second + third + fourth + fifth + () {body;} + sixth + seventh;
53 <<< 53 <<<
54 first + 54 first +
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
90 >>> nested parenthesized are indented more 90 >>> nested parenthesized are indented more
91 (identifier * (verylongIdentifier * verylongIdentifier) * identifier); 91 (identifier * (verylongIdentifier * verylongIdentifier) * identifier);
92 <<< 92 <<<
93 (identifier * 93 (identifier *
94 (verylongIdentifier * 94 (verylongIdentifier *
95 verylongIdentifier) * 95 verylongIdentifier) *
96 identifier); 96 identifier);
97 >>> conditional operands are nested 97 >>> conditional operands are nested
98 identifier ? identifier ? someParticularlyLongOperand : someParticularlyLongOper and : identifier ? someParticularlyLongOperand : someParticularlyLongOperand; 98 identifier ? identifier ? someParticularlyLongOperand : someParticularlyLongOper and : identifier ? someParticularlyLongOperand : someParticularlyLongOperand;
99 <<< 99 <<<
100 identifier ? 100 identifier
101 identifier ? 101 ? identifier
102 someParticularlyLongOperand : 102 ? someParticularlyLongOperand
103 someParticularlyLongOperand : 103 : someParticularlyLongOperand
104 identifier ? 104 : identifier
105 someParticularlyLongOperand : 105 ? someParticularlyLongOperand
106 someParticularlyLongOperand; 106 : someParticularlyLongOperand;
OLDNEW
« no previous file with comments | « test/regression/80.unit ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698