From 0ffdad5a1759d01b899e09b9f406beb078dbfb26 Mon Sep 17 00:00:00 2001
From: Stefano Ponzi <iznopste@gmail.com>
Date: Wed, 26 Nov 2025 17:44:25 +0100
Subject: [PATCH] Added check for empty iterables

---
 .../google/errorprone/bugpatterns/StringConcatToTextBlock.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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.