-
Notifications
You must be signed in to change notification settings - Fork 13.5k
New const traits syntax #139858
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
base: master
Are you sure you want to change the base?
New const traits syntax #139858
Conversation
Some changes occurred in src/tools/rustfmt cc @rust-lang/rustfmt Some changes occurred in src/tools/clippy cc @rust-lang/clippy |
f6be056
to
2003458
Compare
This comment has been minimized.
This comment has been minimized.
2003458
to
d6da247
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment has been minimized.
This comment has been minimized.
d6da247
to
e16c40b
Compare
e16c40b
to
234ba91
Compare
This comment has been minimized.
This comment has been minimized.
234ba91
to
ef1acde
Compare
ef1acde
to
5fa25d4
Compare
This comment has been minimized.
This comment has been minimized.
5fa25d4
to
7762468
Compare
☔ The latest upstream changes (presumably #140695) made this pull request unmergeable. Please resolve the merge conflicts. |
Have we actually settled on a syntax here? Sorry for punting it back over, but would like to get an update here if it's actually ready to review :> ping me when it's ready, sorry for the delays! @rustbot author |
Reminder, once the PR becomes ready for a review, use |
No. Discussion is ongoing. |
7762468
to
d292dc6
Compare
rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead. cc @rust-lang/rust-analyzer This PR modifies cc @jieyouxu Some changes occurred to the CTFE machinery Some changes occurred to the intrinsics. Make sure the CTFE / Miri interpreter cc @rust-lang/miri, @RalfJung, @oli-obk, @lcnr Some changes occurred to constck cc @fee1-dead HIR ty lowering was modified cc @fmease |
This comment has been minimized.
This comment has been minimized.
d292dc6
to
ae4dc9e
Compare
This comment has been minimized.
This comment has been minimized.
ae4dc9e
to
2e751d2
Compare
This comment has been minimized.
This comment has been minimized.
2e751d2
to
aca6032
Compare
This PR only affects the AST and doesn't actually change anything semantically.
All occurrences of
~const
outside of libcore have been replaced by[const]
. Within libcore we have to wait for rustfmt to be bumped in the bootstrap compiler.Caveat discovered during impl: there is no legacy bare trait object recovery for
[const] Trait
as that snippet in type position goes down the slice /array parsing code and will errorr? @fee1-dead
cc @nikomatsakis @traviscross @compiler-errors