rust-lang/rustfmt

rustfmt eats attributes while formatting functions with variadic parameters

Open

#6.561 geöffnet am 11. Mai 2025

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (760 Forks)batch import
C-buggood first issue

Repository-Metriken

Stars
 (4.893 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 10T 21h) (8 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die Formatierungslogik für variadische Parameter in rustfmt, wahrscheinlich in der Datei `src/formatting/items.rs` oder ähnlich, wo Attribute geparst und angehängt werden. Das Problem zeigt, dass Attribute auf dem variadischen Parameter (`...`) entfernt werden. Überprüfe den Code, der Funktionsparameterattribute behandelt, insbesondere im Kontext variadischer Argumente. Die Dokumentation unter https://doc.rust lang.org/nightly/reference/attributes.html#r attributes.allowed position bestätigt, dass Attribute auf variadischen Parametern erlaubt sind. Suche nach Tests im Verzeichnis `tests/`, die variadische Funktionen abdecken, um das erwartete Verhalten zu verstehen.
Tech Stack
rust
Domain
toolingdeveloper experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Basic Rust knowledgeUnderstanding of rustfmt internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
35