Skip to content

Commit ca0c4d9

Browse files
cushonronshapiro
authored andcommitted
Wrap before /*parameter=*/ comments
------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=154055948
1 parent 79a3917 commit ca0c4d9

File tree

3 files changed

+36
-3
lines changed

3 files changed

+36
-3
lines changed

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

Lines changed: 20 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -481,10 +481,11 @@ private static ImmutableList<Token> buildTokens(List<Tok> toks) {
481481
int k = 0;
482482
int kN = toks.size();
483483

484-
while (k < kN) {
485-
// Remaining non-tokens before the token go here.
486-
ImmutableList.Builder<Tok> toksBefore = ImmutableList.builder();
484+
// Remaining non-tokens before the token go here.
485+
ImmutableList.Builder<Tok> toksBefore = ImmutableList.builder();
487486

487+
OUTERMOST:
488+
while (k < kN) {
488489
while (!toks.get(k).isToken()) {
489490
toksBefore.add(toks.get(k++));
490491
}
@@ -519,17 +520,33 @@ private static ImmutableList<Token> buildTokens(List<Tok> toks) {
519520
break;
520521
}
521522
}
523+
if (isParamComment(toks.get(k))) {
524+
tokens.add(new Token(toksBefore.build(), tok, toksAfter.build()));
525+
toksBefore = ImmutableList.<Tok>builder().add(toks.get(k++));
526+
// drop newlines after parameter comments
527+
while (toks.get(k).isNewline()) {
528+
k++;
529+
}
530+
continue OUTERMOST;
531+
}
522532
Tok nonTokenAfter = toks.get(k++);
523533
toksAfter.add(nonTokenAfter);
524534
if (Newlines.containsBreaks(nonTokenAfter.getText())) {
525535
break;
526536
}
527537
}
528538
tokens.add(new Token(toksBefore.build(), tok, toksAfter.build()));
539+
toksBefore = ImmutableList.builder();
529540
}
530541
return tokens.build();
531542
}
532543

544+
private static boolean isParamComment(Tok tok) {
545+
return tok.isSlashStarComment()
546+
&& tok.getText().endsWith("*/")
547+
&& tok.getText().substring(0, tok.getText().length() - "*/".length()).trim().endsWith("=");
548+
}
549+
533550
/**
534551
* Convert from an offset and length flag pair to a token range.
535552
*
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
class B26694550 {
2+
{
3+
fffffffffffffffffffffffffffffff(
4+
ImmutableList.copyOf(keys), /*keepGoing=*/
5+
false,
6+
ggggggggggggggggggggggggggggggggggggggggggg);
7+
}
8+
}
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
class B26694550 {
2+
{
3+
fffffffffffffffffffffffffffffff(
4+
ImmutableList.copyOf(keys),
5+
/*keepGoing=*/ false,
6+
ggggggggggggggggggggggggggggggggggggggggggg);
7+
}
8+
}

0 commit comments

Comments
 (0)