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

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

Issue 2526633002: Support generic methods. Fix #556. (Closed)
Patch Set: Merge branch 'master' into generic-methods Created 4 years, 1 month 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 | « pubspec.lock ('k') | test/splitting/type_arguments.stmt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 40 columns |
2 >>> all fit on one line
3 method<A,B,C,D>();
4 <<<
5 method<A, B, C, D>();
6 >>> prefer to split between args even when they all fit on next line
7 lengthyMethodName<First, Second, Third>();
8 <<<
9 lengthyMethodName<First, Second,
10 Third>();
11 >>> split before first if needed
12 lengthyMethodName<FirstTypeArgumentIsLong, Second>();
13 <<<
14 lengthyMethodName<
15 FirstTypeArgumentIsLong, Second>();
16 >>> split in middle if fit in two lines
17 lengthyMethodName<First, Second, Third, Fourth, Fifth, Sixth, Seventh>();
18 <<<
19 lengthyMethodName<First, Second, Third,
20 Fourth, Fifth, Sixth, Seventh>();
21 >>> split one per line if they don't fit in two lines
22 lengthyMethodName<First, Second, Third, Fourth, Fifth, Sixth, Seventh, Eighth>() ;
23 <<<
24 lengthyMethodName<
25 First,
26 Second,
27 Third,
28 Fourth,
29 Fifth,
30 Sixth,
31 Seventh,
32 Eighth>();
33 >>> prefers to not split at type arguments
34 lengthyMethodName<
35 TypeArgument>(valueArgument);
36 <<<
37 lengthyMethodName<TypeArgument>(
38 valueArgument);
39 >>> split both type and value arguments
40 lengthyMethodName<First, Second, Third, Fourth, Fifth, Sixth, Seventh, Eighth>(f irst, second, third, fourth, fifth, sixth, seventh, eighth);
41 <<<
42 lengthyMethodName<
43 First,
44 Second,
45 Third,
46 Fourth,
47 Fifth,
48 Sixth,
49 Seventh,
50 Eighth>(
51 first,
52 second,
53 third,
54 fourth,
55 fifth,
56 sixth,
57 seventh,
58 eighth);
59 >>> in method chain
60 receiver.method<First, Second, Third, Fourth, Fifth>
61 (first, second, third, fourth, fifth)
62 .method<First, Second, Third, Fourth, Fifth>
63 (first, second, third, fourth, fifth);
64 <<<
65 receiver
66 .method<First, Second, Third,
67 Fourth, Fifth>(first,
68 second, third, fourth, fifth)
69 .method<First, Second, Third,
70 Fourth, Fifth>(first,
71 second, third, fourth, fifth);
OLDNEW
« no previous file with comments | « pubspec.lock ('k') | test/splitting/type_arguments.stmt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698