| OLD | NEW |
|---|
| (Empty) | |
| 1 >>> |
| 2 void main(List argv) { |
| 3 // This is ok (it moves the ? and : to the ends of the previous lines) |
| 4 var cg = outputDart |
| 5 ? new A( |
| 6 outputDir, uri, results.libraries, results.infoMap, results.rules, |
| 7 formatOutput) |
| 8 : new B( |
| 9 outputDir, uri, results.libraries, results.infoMap, results.rules); |
| 10 |
| 11 // This gets rewritten to something much harder to read |
| 12 // var cg = outputDart ? |
| 13 // new ALongerClass(outputDir, uri, results.libraries, results.infoMap, |
| 14 // results.rules, formatOutput) : new BLongerClass( |
| 15 // outputDir, uri, results.libraries, results.infoMap, results.rules); |
| 16 var cg = outputDart |
| 17 ? new ALongerClass( |
| 18 outputDir, uri, results.libraries, results.infoMap, results.rules, |
| 19 formatOutput) |
| 20 : new BLongerClass( |
| 21 outputDir, uri, results.libraries, results.infoMap, results.rules); |
| 22 |
| 23 // OK, left unchanged |
| 24 var cg = outputDart ? |
| 25 new A( |
| 26 outputDir, uri, results.libraries, results.infoMap, results.rules, |
| 27 formatOutput) : |
| 28 new B( |
| 29 outputDir, uri, results.libraries, results.infoMap, results.rules); |
| 30 |
| 31 // Bad, as above |
| 32 var cg = outputDart ? |
| 33 new ALongerClass( |
| 34 outputDir, uri, results.libraries, results.infoMap, results.rules, |
| 35 formatOutput) : |
| 36 new BLongerClass( |
| 37 outputDir, uri, results.libraries, results.infoMap, results.rules); |
| 38 |
| 39 } |
| 40 <<< |
| 41 void main(List argv) { |
| 42 // This is ok (it moves the ? and : to the ends of the previous lines) |
| 43 var cg = outputDart |
| 44 ? new A(outputDir, uri, results.libraries, results.infoMap, results.rules, |
| 45 formatOutput) |
| 46 : new B( |
| 47 outputDir, uri, results.libraries, results.infoMap, results.rules); |
| 48 |
| 49 // This gets rewritten to something much harder to read |
| 50 // var cg = outputDart ? |
| 51 // new ALongerClass(outputDir, uri, results.libraries, results.infoMap, |
| 52 // results.rules, formatOutput) : new BLongerClass( |
| 53 // outputDir, uri, results.libraries, results.infoMap, results.rules); |
| 54 var cg = outputDart |
| 55 ? new ALongerClass(outputDir, uri, results.libraries, results.infoMap, |
| 56 results.rules, formatOutput) |
| 57 : new BLongerClass( |
| 58 outputDir, uri, results.libraries, results.infoMap, results.rules); |
| 59 |
| 60 // OK, left unchanged |
| 61 var cg = outputDart |
| 62 ? new A(outputDir, uri, results.libraries, results.infoMap, results.rules, |
| 63 formatOutput) |
| 64 : new B( |
| 65 outputDir, uri, results.libraries, results.infoMap, results.rules); |
| 66 |
| 67 // Bad, as above |
| 68 var cg = outputDart |
| 69 ? new ALongerClass(outputDir, uri, results.libraries, results.infoMap, |
| 70 results.rules, formatOutput) |
| 71 : new BLongerClass( |
| 72 outputDir, uri, results.libraries, results.infoMap, results.rules); |
| 73 } |
| OLD | NEW |
|---|
Chromium Code Reviews
Issue 1400473008:
Roll Observatory packages and add a roll script (Closed)
Base URL: git@github.com:dart-lang/observatory_pub_packages.git@master