rust-lang/rustfmt

PreferSameLine brace_style with trailing_comma Vertical produces weird results

Open

#2321 aperta il 30 dic 2017

Vedi su GitHub
 (3 commenti) (0 reazioni) (1 assegnatario)Rust (760 fork)batch import
I-poor-formattinggood first issueonly-with-option

Metriche repository

Star
 (4893 star)
Metriche merge PR
 (Merge medio 10g 21h) (8 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Indaga l'interazione tra `brace style = "PreferSameLine"` e `trailing comma = "Vertical"` in rustfmt. Il bug mostra che quando la parentesi graffa di apertura è sulla stessa linea della dichiarazione del trait, viene aggiunta erroneamente una virgola finale. Esamina il codice sorgente per la formattazione di trait e funzioni, probabilmente nei file relativi alla posizionamento delle parentesi graffe e alla logica delle virgole finali. Controlla se l'assegnatario (gentoid) ha fatto progressi o ha commenti sull'issue.
Tech stack
rust
Dominio
toolingdeveloper experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Bloccata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust programmingunderstanding of rustfmt configuration options
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40