-
Notifications
You must be signed in to change notification settings - Fork 13.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Revert #49719 #51417
Revert #49719 #51417
Conversation
@bors r+ |
📌 Commit 1df7817 has been approved by |
@bors p=1 (needs to be backported into beta) |
…tsakis Revert rust-lang#49719 This also needs to be backported into beta. Fixes rust-lang#51416. r? @nikomatsakis
Rollup of 9 pull requests Successful merges: - #51186 (Remove two redundant .nll.stderr files) - #51283 (Deny #[cfg] and #[cfg_attr] on generic parameters.) - #51368 (Fix the use of closures within #[panic_implementation]) - #51380 (Remove dependency on fmt_macros from typeck) - #51389 (rustdoc: Fix missing stability and src links for inlined external macros) - #51399 (NLL performance boost) - #51407 (Update RLS and Rustfmt) - #51417 (Revert #49719) - #51420 (Tries to address the recent network issues) Failed merges:
Marking this as beta-accepted since the compiler team decided at the last meeting to rollback the change, which is already in beta. |
Argh, I'm late and this is merged already. |
Remove `?` macro separator compatibility note from 1.27 release notes The implementation has been reverted in #51417, so this no longer applies to 1.27.0. r? @Mark-Simulacrum
This also needs to be backported into beta.
Fixes #51416.
r? @nikomatsakis