Skip to content

Conversation

@tgross35
Copy link
Contributor

Rustfmt will format () or [] macro invocations that contain valid Rust syntax, but not {} invocations. Switch to (), and update the syntax to be valid Rust so rustfmt cleans them up.

Rustfmt will format `()` or `[]` macro invocations that contain valid
Rust syntax, but not `{}` invocations. Switch to `()`, and update the
syntax to be valid Rust so rustfmt cleans them up.
Rustfmt was skipping this entire block because it doesn't support
comments in the middle of chained `|`. So, add a `=> true` after
`IPV6_FLOWINFO_PRIORITY` and the whole closure starts to get formatted
again.
@tgross35 tgross35 added this pull request to the merge queue Oct 23, 2025
Merged via the queue into rust-lang:main with commit 4e4d9c2 Oct 23, 2025
52 checks passed
@tgross35 tgross35 deleted the headers-macro branch October 23, 2025 10:10
@tgross35 tgross35 added the stable-nominated This PR should be considered for cherry-pick to libc's stable release branch label Nov 3, 2025
tgross35 added a commit to tgross35/rust-libc that referenced this pull request Nov 3, 2025
Rustfmt will format `()` or `[]` macro invocations that contain valid
Rust syntax, but not `{}` invocations. Switch to `()`, and update the
syntax to be valid Rust so rustfmt cleans them up.

(backport <rust-lang#4765>)
(cherry picked from commit a8b1532)
tgross35 added a commit to tgross35/rust-libc that referenced this pull request Nov 3, 2025
Rustfmt was skipping this entire block because it doesn't support
comments in the middle of chained `|`. So, add a `=> true` after
`IPV6_FLOWINFO_PRIORITY` and the whole closure starts to get formatted
again.

(backport <rust-lang#4765>)
(cherry picked from commit 4e4d9c2)
tgross35 added a commit to tgross35/rust-libc that referenced this pull request Nov 3, 2025
Rustfmt will format `()` or `[]` macro invocations that contain valid
Rust syntax, but not `{}` invocations. Switch to `()`, and update the
syntax to be valid Rust so rustfmt cleans them up.

(backport <rust-lang#4765>)
(cherry picked from commit a8b1532)
tgross35 added a commit to tgross35/rust-libc that referenced this pull request Nov 3, 2025
Rustfmt was skipping this entire block because it doesn't support
comments in the middle of chained `|`. So, add a `=> true` after
`IPV6_FLOWINFO_PRIORITY` and the whole closure starts to get formatted
again.

(backport <rust-lang#4765>)
(cherry picked from commit 4e4d9c2)
@tgross35 tgross35 mentioned this pull request Nov 3, 2025
github-merge-queue bot pushed a commit that referenced this pull request Nov 4, 2025
Rustfmt will format `()` or `[]` macro invocations that contain valid
Rust syntax, but not `{}` invocations. Switch to `()`, and update the
syntax to be valid Rust so rustfmt cleans them up.

(backport <#4765>)
(cherry picked from commit a8b1532)
github-merge-queue bot pushed a commit that referenced this pull request Nov 4, 2025
Rustfmt was skipping this entire block because it doesn't support
comments in the middle of chained `|`. So, add a `=> true` after
`IPV6_FLOWINFO_PRIORITY` and the whole closure starts to get formatted
again.

(backport <#4765>)
(cherry picked from commit 4e4d9c2)
@tgross35 tgross35 added stable-applied This PR has been cherry-picked to libc's stable release branch and removed stable-nominated This PR should be considered for cherry-pick to libc's stable release branch labels Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stable-applied This PR has been cherry-picked to libc's stable release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants