rust-lang/rustfmt

PreferSameLine brace_style with trailing_comma Vertical produces weird results

Open

#2 321 ouverte le 30 déc. 2017

Voir sur GitHub
 (3 commentaires) (0 réactions) (1 assigné)Rust (760 forks)batch import
I-poor-formattinggood first issueonly-with-option

Métriques du dépôt

Stars
 (4 893 stars)
Métriques de merge PR
 (Merge moyen 10j 21h) (8 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Étudiez l'interaction entre `brace style = "PreferSameLine"` et `trailing comma = "Vertical"` dans rustfmt. Le bug montre que lorsque l'accolade ouvrante est sur la même ligne que la déclaration du trait, une virgule finale est incorrectement ajoutée. Examinez le code source pour le formatage des traits et des fonctions, probablement dans les fichiers liés au placement des accolades et à la logique des virgules finales. Vérifiez si l'assigné (gentoid) a des progrès ou des commentaires sur le problème.
Stack technique
rust
Domaine
toolingdeveloper experience
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Bloquée
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust programmingunderstanding of rustfmt configuration options
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40