rustfmt stopped formatting `const` items in macros

[CAD97]

trafficstars

Test:

> echo "m!(const N: usize = 0;);" | rustup run beta rustfmt
m!(
    const N: usize = 0;
);

> echo "m!(const N: usize = 0;);" | rustup run nightly rustfmt
m!(const N: usize = 0;);

> rustup run beta rustfmt -Vv
rustfmt 1.7.0-beta (f732c37b417 2024-01-12)
> rustup run nightly rustfmt -Vv
rustfmt 1.7.0-nightly (88189a71 2024-01-19)

[ytmimi]

We haven't done a subtree sync since 2023-10-22 https://github.com/rust-lang/rust/pull/117066

Running rustfmt built from source :rustfmt 1.7.0-nightly (bf967319 2024-01-20)

cargo run --bin rustfmt <<< "m!(const N: usize = 0;);"
#output
m!(const N: usize = 0;);

Running rustfmt nightly-2023-10-22 (rustfmt 1.6.0-nightly (1c05d50c 2023-10-21))

rustfmt +nightly-2023-10-22 <<< "m!(const N: usize = 0;);"
m!(
    const N: usize = 0;
);

Running rustfmt nightly-2023-10-23 (rustfmt 1.6.0-nightly (54b0434c 2023-10-22))

rustfmt +nightly-2023-10-23 <<< "m!(const N: usize = 0;);"          
m!(
    const N: usize = 0;
);

Running rustfmt nightly-2023-10-24 (rustfmt 1.7.0-nightly (cd674d61 2023-10-24)):

rustfmt +nightly-2023-10-24 <<< "m!(const N: usize = 0;);"
m!(
    const N: usize = 0;
);

Running rustfmt nightly-2023-10-25 (rustfmt 1.7.0-nightly (df871fbf 2023-10-24)):

rustfmt +nightly-2023-10-25 <<< "m!(const N: usize = 0;);" 
m!(
    const N: usize = 0;
);

We've done 2 subtree pushes since the last release: https://github.com/rust-lang/rustfmt/pull/5980 and https://github.com/rust-lang/rustfmt/pull/5994. The last of which brought in changes from nightly-2023-12-28

Not really sure what the issue is right now, but given that bors merged the last subtree sync on 2023-10-23 I'd assume that the error would have presented itself in the nightly-2023-10-24 release. Will try to look into this more a little later to figure out which nightly version started demonstrating this different macro formatting behavior.

[ytmimi]

Tracked the issue down to the nightly-2023-12-28 release.

rustfmt +nightly-2023-12-27 <<< "m!(const N: usize = 0;);"
m!(
    const N: usize = 0;
);

rustfmt +nightly-2023-12-28 <<< "m!(const N: usize = 0;);"
m!(const N: usize = 0;);

[ytmimi]

bisected the formatting changes back to https://github.com/rust-lang/rust/pull/119099

[compiler-errors]

This has to do with the weirdness of the way that parse_macro_arg works. Unlike parsing nonterminal args in a macro-by-example, it eagerly tries, for example, to parse a type without checking that the beginning token may begin a type:

https://github.com/rust-lang/rustfmt/blob/bf967319e258acb9b1648a952bba52665eceaf52/src/parse/macros/mod.rs#L54

Contrast this to the nonterminal parsing code, which first checks that the nonterminal may begin with a given token:

https://github.com/rust-lang/rust/blob/ef71f1047e04438181d7cb925a833e2ada6ab390/compiler/rustc_parse/src/parser/nonterminal.rs#L47

In rust-lang/rust#119099, @fmease implemented a change so that const Tr would be parsed as dyn const Tr (a trait object to a const trait) in edition 2015.

This is okay for the purposes of macros, because he explicitly made sure that const did not get added to the list of tokens that may begin a :ty nonterminal kind: https://github.com/rust-lang/rust/pull/119099#discussion_r1436996007

However, since rustfmt is not so careful about eagerly parsing macro args before checking that they're legal in macro position, this changed the way that the string of tokens is being parsed into macro args.

I put up a fix against rust-lang/rust (since that should fix the nightly rustfmt): rust-lang/rust#120218.

[ytmimi]

@compiler-errors thank you for the detailed explanation and for jumping into action so quickly on this one 🙏🏼

[ytmimi]

Just checked and nightly-2024-01-23 restored the formatting.

rustfmt +nightly-2024-01-23 <<< "m!(const N: usize = 0;);"
m!(
    const N: usize = 0;
);

will hold off on closing this until the next subree push so that we incorporate the fix from rust-lang/rust back into rustfmt