rust-lang/rustfmt

PreferSameLine brace_style with trailing_comma Vertical produces weird results

Open

#2.321 geöffnet am 30. Dez. 2017

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Rust (760 Forks)batch import
I-poor-formattinggood first issueonly-with-option

Repository-Metriken

Stars
 (4.893 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 10T 21h) (8 gemergte PRs in 30 T)

Beschreibung

Upgrading rustfmt-nightly from 0.2.15 to 0.3.4 created this change:

@@ -104,8 +122,7 @@ impl FetchCommandMessages {
 
 pub trait FetchBuilderMessages
 where
-    Self: Sized,
-{
+    Self: Sized, {
     fn prepare(self) -> FetchCommandMessages;
 
     fn num(self, num: u32) -> FetchCommandMessages {

This seems like a bug, since trailing_comma = "Vertical" should not trigger if the opening brace is on the same line.

Contributor Guide

Research-Richtung
Untersuche die Interaktion zwischen `brace style = "PreferSameLine"` und `trailing comma = "Vertical"` in rustfmt. Der Fehler zeigt, dass wenn die öffnende geschweifte Klammer in derselben Zeile wie die Trait Deklaration steht, fälschlicherweise ein abschließendes Komma hinzugefügt wird. Untersuche den Quellcode für die Formatierung von Traits und Funktionen, wahrscheinlich in Dateien, die sich mit der Platzierung geschweifter Klammern und der Logik für abschließende Kommas befassen. Prüfe, ob der Beauftragte (gentoid) Fortschritte oder Kommentare zu dem Issue hat.
Tech Stack
rust
Domain
toolingdeveloper experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Blockiert
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Rust programmingunderstanding of rustfmt configuration options
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40