rust-lang/rustfmt

PreferSameLine brace_style with trailing_comma Vertical produces weird results

Open

#2.321 aberto em 30 de dez. de 2017

Ver no GitHub
 (3 comments) (0 reactions) (1 assignee)Rust (760 forks)batch import
I-poor-formattinggood first issueonly-with-option

Métricas do repositório

Stars
 (4.893 stars)
Métricas de merge de PR
 (Mesclagem média 10d 21h) (8 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Investigue a interação entre `brace style = "PreferSameLine"` e `trailing comma = "Vertical"` no rustfmt. O bug mostra que quando a chave de abertura está na mesma linha da declaração do trait, uma vírgula final é adicionada incorretamente. Examine o código fonte para formatação de traits e funções, provavelmente em arquivos relacionados à colocação de chaves e à lógica de vírgula final. Verifique se o responsável (gentoid) tem algum progresso ou comentários sobre o problema.
Pilha de tecnologia
rust
Domain
toolingdeveloper experience
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Bloqueado
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rust programmingunderstanding of rustfmt configuration options
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40