diff --git a/core/src/main/java/com/google/errorprone/bugpatterns/StringConcatToTextBlock.java b/core/src/main/java/com/google/errorprone/bugpatterns/StringConcatToTextBlock.java index 308ca4899d8..4ef37d134b0 100644 --- a/core/src/main/java/com/google/errorprone/bugpatterns/StringConcatToTextBlock.java +++ b/core/src/main/java/com/google/errorprone/bugpatterns/StringConcatToTextBlock.java @@ -98,7 +98,7 @@ && isSameType(getType(parent), state.getSymtab().stringType, state)) { .filter(t -> t.kind().equals(Tokens.TokenKind.STRINGLITERAL)) .map(t -> t.stringVal()) .collect(toImmutableList()); - boolean trailingNewline = getLast(strings).endsWith("\n"); + boolean trailingNewline = !strings.isEmpty() && getLast(strings) != null && getLast(strings).endsWith("\n"); // Only migrate if there are enough lines to make it worthwhile. Escaping the trailing newline // slightly reduces the readability benefit of migrating, so require an extra line to make it // worth it.