rustfmt eats attributes while formatting functions with variadic parameters

[mejrs]

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

[ytmimi]

Thanks for the report. confirming I can reproduce the issue with rustfmt 1.8.0-nightly (fd0ea74 2025-04-12).

I believe the issue is that we're forgetting to take the attribute into account when rewriting the variadic parameter:

rustfmt/src/items.rs

Line 2345 in bf5f0ea

self.ty.rewrite_result(context, shape)

I think this could be fixed with the following change, thought some additional test cases would be needed to verify that.

-            self.ty.rewrite_result(context, shape)
+            combine_strs_with_missing_comments(
+                context,
+                &param_attrs_result,
+                &self.ty.rewrite_result(context, shape)?,
+                span,
+                shape,
+                !has_multiple_attr_lines && !has_doc_comments,
+            )