Rename splat to avoid stable name collisions#159817
Conversation
fastmod --fixed-strings '#[splat]' '#[arg_splat]' fastmod --fixed-strings '#![feature(splat' '#![feature(arg_splat' fastmod --fixed-strings 'multiple `splat` attributes' 'multiple `arg_splat` attributes' fastmod --fixed-strings '$splat$' '$arg_splat$' Revert some changes that aren't in rustc_demangle yet fastmod --fixed-strings 'main::Type<fn(#[arg_splat]' 'main::Type<fn(#[splat]' fastmod --fixed-strings 'main::Type<*const fn(#[arg_splat]' 'main::Type<*const fn(#[splat]'
|
The reflection data structures are tied exactly to the implementation cc @oli-obk This PR changes rustc_public cc @oli-obk, @celinval, @ouz-a, @makai410 Some changes occurred in compiler/rustc_hir/src/attrs cc @jdonszelmann, @JonathanBrouwer
cc @rust-lang/rustfmt Some changes occurred in compiler/rustc_attr_parsing |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
No P-high for me! @rustbot label +beta-nominated +A-attributes +A-macros +A-resolve +C-bug +F-splat +T-compiler |
|
FWIW we only assign |
|
I will "de-nominate" from backport as now it's a bit too early. Feel free to nominate again when the PR is reviewed and approved :) @rustbot label -beta-nominated |
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
☔ The latest upstream changes (presumably #159814) made this pull request unmergeable. Please resolve the merge conflicts by rebasing. |
There was a problem hiding this comment.
Please restore src/tools/rustfmt/tests/source/reorder_modules/ZYXW/mod.rs, src/tools/rustfmt/tests/source/reorder_modules/ZYXW/mod.rs, src/tools/rustfmt/tests/target/reorder_modules/ABCD/mod.rs, src/tools/rustfmt/tests/target/reorder_modules/ZYXW/mod.rs
Removing these files is causing rustfmt test failures:
thread '<unnamed>' (54612) panicked at src/tools/rustfmt/src/test/mod.rs:842:40:
called `Result::unwrap()` on an `Err` value: ModuleResolutionError(ModuleResolutionError { module: "ZYXW", kind: NotFound { file: "tests/target/reorder_modules/ZYXW.rs" }
|
Reminder, once the PR becomes ready for a review, use |
TODO
Tracking issue: #153629
This PR renames
#[splat]to#[arg_splat], in a mostly automated way (usingfastmod).The rename commands I used were:
Part of #159428
Backport Advice
Cherry-pick the first commit to the beta branch, then run the fastmod commands above.
I've tested this locally, and it works, see my
splat-rename-betabranch.Remaining Work
rustc_demangle needs an update to the attribute name, a release, and then a version bump in rust-lang/rust. I'll do that separately. It doesn't need to be backported, because the last version bump was after beta branched.
There's also some external code that needs updates, the full list is in the ticket: #159428 (comment)
Can someone with write access also update the description for the F-splat GitHub label 🙂
@rustbot label +beta-nominated +A-attributes +A-macros +A-resolve +C-bug +F-splat +P-high +T-compiler