Skip to content

Commit da38d17

Browse files
committed
Indent formal parameter declarations broken after annotations
------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=120391980
1 parent c6b20c8 commit da38d17

File tree

4 files changed

+90
-6
lines changed

4 files changed

+90
-6
lines changed

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

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3044,7 +3044,8 @@ void declareOne(
30443044
}
30453045
}
30463046

3047-
builder.open(ZERO);
3047+
boolean isParam = node.getParent().getNodeType() == ASTNode.METHOD_DECLARATION;
3048+
builder.open(isParam && hasAnnotations(modifiers) ? plusFour : ZERO);
30483049
{
30493050
visitAndBreakModifiers(modifiers, annotationsDirection, Optional.of(verticalAnnotationBreak));
30503051
builder.open(plusFour);
@@ -3123,6 +3124,15 @@ void declareOne(
31233124
}
31243125
}
31253126

3127+
private boolean hasAnnotations(List<IExtendedModifier> modifiers) {
3128+
for (IExtendedModifier modifier : modifiers) {
3129+
if (modifier.isAnnotation()) {
3130+
return true;
3131+
}
3132+
}
3133+
return false;
3134+
}
3135+
31263136
/**
31273137
* Declare multiple variables or variable-like things.
31283138
* @param annotationsDirection {@link Direction#VERTICAL} or {@link Direction#HORIZONTAL}
Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
class B23804934{
2+
3+
@Zzzzzz
4+
B23804934(
5+
ZzzzzzzzZzzzzzZzzzzZzzzzzzz zzzzzzzzZzzzzzZzzzzZzzzzzzz,
6+
ZzzzzzzZzzzzzzz<ZzzzzzZzzzZzzz> zzzzzzZzzzZzzzZzzzzzzz,
7+
@ZzzzZzzzzZzzzzzz Zzzzzz zzzzzZzzzzzz,
8+
@ZzzzzzzzzzzZzzzzzz(Zz.ZZZZZZZ_ZZZZZ_ZZZZ_ZZZZZZ_ZZZZ_ZZZZZZZZZZ)
9+
Zzzzzzzz<Zzzzzzz> zzzzzZzzzZzzzzzZzzzZzzzzzzzzz) {
10+
zzzz.zzzzzzzzZzzzzzZzzzzZzzzzzzz = zzzzzzzzZzzzzzZzzzzZzzzzzzz;
11+
zzzz.zzzzzzZzzzZzzzZzzzzzzz = zzzzzzZzzzZzzzZzzzzzzz;
12+
zzzz.zzzzzZzzzzzz = zzzzzZzzzzzz;
13+
zzzz.zzzzzZzzzZzzzzzZzzzZzzzzzzzzz = zzzzzZzzzZzzzzzZzzzZzzzzzzzzz;
14+
}
15+
16+
zzzz z(
17+
zzz z,
18+
@Zzzzzzzz @Zzzzzzzzzz
19+
ZzzzzzzzzZzzz<Zzzzzz>
20+
zzzzZzzzZzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz) {}
21+
22+
@ZzzZzzzZzzzzzzzzzz
23+
@Zzzzzzzz(zzzzzzzzz = Zzzzzzzzz.ZZ)
24+
ZzzzzzzzzzZzzzzz<ZzzZzzzzzzzzzzZzzzzzzzz> zzzzzzzZzzzzzzzzzz(
25+
@Zzz @Zzz zzzz zzzzZzzzZz,
26+
@Zzz @ZzzzzzzzzzZzzzz(zzzz = ZzzzzZzzzzzZzzzz.ZZZZZZ_ZZZ_ZZZ_ZZZZZZZ)
27+
zzzzzzz zzzzzzZzzZzzZzzzzzz,
28+
@Zzzzz @ZzzzzzzzzzZzzzz(zzzz = ZzzzzZzzzzzZzzzz.ZZZZZ_ZZZ_ZZZZZZZZZZZ)
29+
zzzzzzz zzzzzZzzZzzzzzzzzzz,
30+
@Zzzzz ZzzzzZzzz zzzzzZzzz) {}
31+
32+
void fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff(
33+
final zzzzzzzzzzzzzz zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz) {}
34+
}
Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
class B23804934 {
2+
3+
@Zzzzzz
4+
B23804934(
5+
ZzzzzzzzZzzzzzZzzzzZzzzzzzz zzzzzzzzZzzzzzZzzzzZzzzzzzz,
6+
ZzzzzzzZzzzzzzz<ZzzzzzZzzzZzzz> zzzzzzZzzzZzzzZzzzzzzz,
7+
@ZzzzZzzzzZzzzzzz Zzzzzz zzzzzZzzzzzz,
8+
@ZzzzzzzzzzzZzzzzzz(Zz.ZZZZZZZ_ZZZZZ_ZZZZ_ZZZZZZ_ZZZZ_ZZZZZZZZZZ)
9+
Zzzzzzzz<Zzzzzzz> zzzzzZzzzZzzzzzZzzzZzzzzzzzzz) {
10+
zzzz.zzzzzzzzZzzzzzZzzzzZzzzzzzz = zzzzzzzzZzzzzzZzzzzZzzzzzzz;
11+
zzzz.zzzzzzZzzzZzzzZzzzzzzz = zzzzzzZzzzZzzzZzzzzzzz;
12+
zzzz.zzzzzZzzzzzz = zzzzzZzzzzzz;
13+
zzzz.zzzzzZzzzZzzzzzZzzzZzzzzzzzzz = zzzzzZzzzZzzzzzZzzzZzzzzzzzzz;
14+
}
15+
16+
zzzz z(
17+
zzz z,
18+
@Zzzzzzzz @Zzzzzzzzzz
19+
ZzzzzzzzzZzzz<Zzzzzz>
20+
zzzzZzzzZzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz) {}
21+
22+
@ZzzZzzzZzzzzzzzzzz
23+
@Zzzzzzzz(zzzzzzzzz = Zzzzzzzzz.ZZ)
24+
ZzzzzzzzzzZzzzzz<ZzzZzzzzzzzzzzZzzzzzzzz> zzzzzzzZzzzzzzzzzz(
25+
@Zzz @Zzz zzzz zzzzZzzzZz,
26+
@Zzz @ZzzzzzzzzzZzzzz(zzzz = ZzzzzZzzzzzZzzzz.ZZZZZZ_ZZZ_ZZZ_ZZZZZZZ)
27+
zzzzzzz zzzzzzZzzZzzZzzzzzz,
28+
@Zzzzz @ZzzzzzzzzzZzzzz(zzzz = ZzzzzZzzzzzZzzzz.ZZZZZ_ZZZ_ZZZZZZZZZZZ)
29+
zzzzzzz zzzzzZzzZzzzzzzzzzz,
30+
@Zzzzz ZzzzzZzzz zzzzzZzzz) {}
31+
32+
void fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff(
33+
final zzzzzzzzzzzzzz
34+
zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz) {}
35+
}

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

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,16 +40,21 @@ class B24702438 {
4040
void f(
4141
int a,
4242
@Nullable @Deprecated
43-
ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
43+
ImmutableList<String>
44+
veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
4445
@Nullable @Deprecated
45-
ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
46+
ImmutableList<String>
47+
veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
4648
@Nullable @Deprecated
47-
ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
49+
ImmutableList<String>
50+
veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
4851
int c) {}
4952

5053
void g(
5154
@Nullable @Deprecated
52-
ImmutableList<String> veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
55+
ImmutableList<String>
56+
veryVeryLooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong,
5357
@Nullable @Deprecated
54-
ImmutableList<String> veryVeryLoooooooooooooooooooooooooooooooooooooooooooooooooooooooong) {}
58+
ImmutableList<String>
59+
veryVeryLoooooooooooooooooooooooooooooooooooooooooooooooooooooooong) {}
5560
}

0 commit comments

Comments
 (0)