rust-lang/rustfmt

rustfmt eats attributes while formatting functions with variadic parameters

Open

#6 561 ouverte le 11 mai 2025

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Rust (760 forks)batch import
C-buggood first issue

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

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

Guide contributeur

Direction de recherche
Étudiez la logique de formatage pour les paramètres variadiques dans rustfmt, probablement dans le fichier `src/formatting/items.rs` ou similaire, où les attributs sont analysés et attachés. Le problème montre que les attributs sur le paramètre variadique (`...`) sont supprimés. Vérifiez le code qui gère les attributs des paramètres de fonction, en particulier dans le contexte des arguments variadiques. La documentation à l'adresse https://doc.rust lang.org/nightly/reference/attributes.html#r attributes.allowed position confirme que les attributs sont autorisés sur les paramètres variadiques. Recherchez des tests dans le répertoire `tests/` qui couvrent les fonctions variadiques pour comprendre le comportement attendu.
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.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Basic Rust knowledgeUnderstanding of rustfmt internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35