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

Side by Side Diff: packages/dart_style/test/splitting/parameters.unit

Issue 2990843002: Removed fixed dependencies (Closed)
Patch Set: Created 3 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
OLDNEW
1 40 columns | 1 40 columns |
2 >>> long parameters list, this.field 2 >>> long parameters list, this.field
3 class Foo { 3 class Foo {
4 Foo(this.first, this.second, this.third, this.fourth); 4 Foo(this.first, this.second, this.third, this.fourth);
5 } 5 }
6 <<< 6 <<<
7 class Foo { 7 class Foo {
8 Foo(this.first, this.second, 8 Foo(this.first, this.second,
9 this.third, this.fourth); 9 this.third, this.fourth);
10 } 10 }
11 >>> indent parameters more if body is a wrapped => 11 >>> indent parameters more if body is a wrapped =>
12 method(int firstArgument, int argumentTwo) => "very long body that must wrap"; 12 method(int firstArgument, int argumentTwo) => "very long body that must wrap";
13 <<< 13 <<<
14 method(int firstArgument, 14 method(int firstArgument,
15 int argumentTwo) => 15 int argumentTwo) =>
16 "very long body that must wrap"; 16 "very long body that must wrap";
17 >>> split before "covariant" (skip: published version of analyzer doesn't suppor t 'covariant yet')
18 class A {
19 longMethod(covariant parameterNameHere) {}
20 }
21 <<<
22 class A {
23 longMethod(
24 covariant parameterNameHere) {}
25 }
26 >>> split before "covariant" with multiple parameters (skip: published version o f analyzer doesn't support 'covariant yet')
27 class A {
28 longMethod(covariant first, second, covariant int third(parameter), fourth) {}
29 }
30 <<<
31 class A {
32 longMethod(
33 covariant first,
34 second,
35 covariant int third(parameter),
36 fourth) {}
37 }
38 >>> never split after "covariant" (at least for now) (skip: published version of analyzer doesn't support 'covariant yet')
39 class A {
40 longMethod(covariant int veryLongParameterNameWow) {}
41 }
42 <<<
43 class A {
44 longMethod(
45 covariant int veryLongParameterNameWow) {}
46 }
OLDNEW
« no previous file with comments | « packages/dart_style/test/splitting/parameters.stmt ('k') | packages/dart_style/test/splitting/statements.stmt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698