Unclosed string literal error on regex string

Offen

#1,253 erstellt am 22.4.2025 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 42/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: java
Bereich: tooling

Rechercherichtung

Beginne mit dem vereinfachten Java-Reproduzierer im Issue und führe google-java-format 1.26.0 über den dokumentierten Befehl java -jar aus. Verfolge, wie der Formatter String-Literale mit Regex-Backslashes parst, und ermittle, warum der gültige Quelltext als nicht geschlossenes String-Literal gemeldet wird. Erledigt ist es, wenn der Reproduzierer ohne diesen Fehler formatiert wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Hello, I found this issue when updating the formatter version in googleapis/java-spanner

package com.example;

final class Test {
  // Error: unclosed string literal
  String breaks = "'\\d{2}):(\\d{2}):(\\d{2})(\\.\\d{1,9})?)([Zz]|([+-])(\\d{2}):(\\d{2})))|(READ_TIMESTAMP)[\\t";
  // Works
  String fix =    "'\\d{2}):(\\d{2}):(\\d{2})(\\.\\d{1,9})?)([Zz]|([+-])(\\d{2}):(\\d{2})))|(READ_TIMESTAMP)[\u005Ct";
}

Produces the following error:

hi on] diegomarquezp:java-spanner$ java -jar ~/.library_generation/google-java-format.jar --version 
google-java-format: Version 1.26.0
[hi on] diegomarquezp:java-spanner$ java -jar ~/.library_generation/google-java-format.jar test.java 
test.java:7:13: error: unclosed string literal
  String fix = "'((STRONG)|(MIN_READ_TIMESTAMP)[\\t ]+((\\d{4})-(\\d{2})-(\\d{2})([Tt](\\d{2}):(\\d{2}):(\\d{2})(\\.\\d{1,9})?)([Zz]|([+-])(\\d{2}):(\\d{2})))|(READ_TIMESTAMP)[\u005Ct ]+((\\d{4})-(\\d{2})-(\\d{2})([Tt](\\d{2}):(\\d{2}):(    \\d{2})(\\.\\d{1,9})?)([Zz]|([+-])(\\d{2}):(\\d{2})))|(MAX_STALENESS)[\u005Ct ]+((\\d{1,19})(s|ms|us|ns))|(EXACT_STALENESS)[\\t ]+((\\d{1,19})(s|ms|us|ns)))'";
            ^
[hi on] diegomarquezp:java-spanner$

Note that String fix doesn't have the error after replacing these two specific double backslashes \\ with the hexcode version \u0005C.

update: simplified reproducer code.

Vorherrschende Sprache: Java
Sterne: 6.2k
Forks: 936
Ø Merge: 6 Min.
Gemergte PRs (30 T.): 3

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus google/google-java-format

Misleading message for --dry-run with --replace Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100

google/google-java-format#1094 · 1 Kommentar · Erstellt am 07.4.2024
Complementary Teamups Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 15/100

google/google-java-format#1450 · 1 Reaktion · Erstellt am 06.9.2026
IndexOutOfBoundsException - Wrong formatted content when dealing with latex String Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100

google/google-java-format#1439 · 1 Kommentar · Erstellt am 06.8.2026
unused import removal leaves extra blank line between package declaration and class Javadoc / declaration Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 62/100

google/google-java-format#1436 · 2 Kommentare · Erstellt am 04.8.2026
Possible issue with Eclipse plugin Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100

google/google-java-format#1428 · 4 Kommentare · Erstellt am 30.7.2026

Alle Issues in google/google-java-format