rust-lang/rustfmt

rustfmt eats attributes while formatting functions with variadic parameters

Open

#6.561 aberto em 11 de mai. de 2025

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)Rust (760 forks)batch import
C-buggood first issue

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

If you format...

#[allow()]
unsafe extern "C" {
    #[allow()]
    pub fn foo(#[allow()] arg: *mut u8, #[allow()]...);
}

Then it will format into

#[allow()]
unsafe extern "C" {
    #[allow()]
    pub fn foo(#[allow()] arg: *mut u8, ...);
}

(The choice of attribute seems irrelevant, I've used allow because it compiles in any position).

Note that this position is explicitly documented at https://doc.rust-lang.org/nightly/reference/attributes.html#r-attributes.allowed-position

Guia do colaborador

Direção de pesquisa
Investigue a lógica de formatação para parâmetros variádicos no rustfmt, provavelmente no arquivo `src/formatting/items.rs` ou similar, onde os atributos são analisados e anexados. O problema mostra que os atributos no parâmetro variádico (`...`) são descartados. Verifique o código que lida com atributos de parâmetros de função, particularmente no contexto de argumentos variádicos. A documentação em https://doc.rust lang.org/nightly/reference/attributes.html#r attributes.allowed position confirma que atributos são permitidos em parâmetros variádicos. Procure testes no diretório `tests/` que cubram funções variádicas para entender o comportamento esperado.
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.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Basic Rust knowledgeUnderstanding of rustfmt internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35