Wrong indentation on if-then with comments

Offen

#1,009 erstellt am 12.12.2023 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 45/100

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

Rechercherichtung

Beginne damit, den Formatter-Quellcode um Zeile 331 zu untersuchen, und reproduziere das gemeldete Snippet mit einer aufgeteilten if-Bedingung und einem abschließenden Kommentar. Verfolge, wie die Einrückung für den then-Block berechnet wird, füge anschließend eine Abdeckung für diesen Fall hinzu oder aktualisiere sie und überprüfe, dass die formatierte Ausgabe prd korrekt einrückt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Comments at the end of this if makes the then block to not get indented.

                            if (prd.source()
                                .isPresent()) // exclude generated productions like casts
                            prd = null;

Line 331. IntelliJ reports this as well.
I think the problem comes from the if condition that got split into two lines. If I move the comment to a separate line, then indentation works as expected.

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