Skip to content

Commit 8e6fbbb

Browse files
cushonronshapiro
authored andcommitted
Unify breaks between type parameters
------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=183326714
1 parent ffd2494 commit 8e6fbbb

File tree

4 files changed

+16
-3
lines changed

4 files changed

+16
-3
lines changed

core/src/main/java/com/google/googlejavaformat/java/JavaInputAstVisitor.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1530,8 +1530,7 @@ public Void visitParameterizedType(ParameterizedTypeTree node, Void unused) {
15301530
for (Tree typeArgument : node.getTypeArguments()) {
15311531
if (!first) {
15321532
token(",");
1533-
// TODO(cushon): unify breaks
1534-
builder.breakToFill(" ");
1533+
builder.breakOp(" ");
15351534
}
15361535
scan(typeArgument, null);
15371536
first = false;

core/src/test/resources/com/google/googlejavaformat/java/testdata/B20128760.output

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,8 @@ class B20128760 {
8282
public class Foo
8383
extends Xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
8484
implements Xxxxxxxxxxxxx<
85-
Xxxxxxxxxxxxxxxxxxxxxxxxxxxxx, Xxxxxxxxxxxxxxxxxxxxxxxxxxxxx,
85+
Xxxxxxxxxxxxxxxxxxxxxxxxxxxxx,
86+
Xxxxxxxxxxxxxxxxxxxxxxxxxxxxx,
8687
Xxxxxxxxxxxxxxxxxxxxxxxxxxxxx>,
8788
Xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx {}
8889

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
class B72507902 {
2+
private static class Xxxxxxxxxxxxxxxxxxxx
3+
extends Xxxxxxxxxxx<
4+
Pair<xxxxxxxxxxxxxxxxxxx, Xxxxxxxxxx<Xxxxxxxxx>>, Xxxxxxxxxxxxxxxxxxxx,
5+
List<Xxxxxxxxx>> {}
6+
}
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
class B72507902 {
2+
private static class Xxxxxxxxxxxxxxxxxxxx
3+
extends Xxxxxxxxxxx<
4+
Pair<xxxxxxxxxxxxxxxxxxx, Xxxxxxxxxx<Xxxxxxxxx>>,
5+
Xxxxxxxxxxxxxxxxxxxx,
6+
List<Xxxxxxxxx>> {}
7+
}

0 commit comments

Comments
 (0)