Allow ~const bounds on trait assoc functions#88418
Merged
bors merged 2 commits intorust-lang:masterfrom Aug 31, 2021
Merged
Conversation
3a18b5f to
523490e
Compare
oli-obk
reviewed
Aug 28, 2021
|
|
||
| trait Foo { | ||
| fn a(); | ||
| fn b() where Self: ~const Bar; |
Contributor
There was a problem hiding this comment.
A few more tests I think we should have:
where Self: ~const Foo- a const and a nonconst impl for such a trait
- a monomorphic use that actually runs it
oli-obk
reviewed
Aug 28, 2021
Comment on lines
+37
to
+40
| const ANS1: u8 = Const::foo(); | ||
| let ans2 = NonConst::foo(); | ||
|
|
||
| assert_eq!(ANS1 + ans2, 42); |
Contributor
|
@bors r+ |
Collaborator
|
📌 Commit ee02c8e has been approved by |
Collaborator
|
⌛ Testing commit ee02c8e with merge 98c17e16ab9f27c1948e0c8d9aa262bfa0800128... |
Collaborator
|
The job Click to see the possible cause of the failure (guessed by this bot) |
Collaborator
|
💔 Test failed - checks-actions |
Contributor
|
@bors retry |
bors
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Aug 31, 2021
Rollup of 10 pull requests Successful merges: - rust-lang#85017 (Add carrying_add, borrowing_sub, widening_mul, carrying_mul methods to integers) - rust-lang#86362 (Avoid cloning LocalDecls) - rust-lang#88391 (Fix json tuple struct enum variant ) - rust-lang#88399 (Disallow the aapcs CC on Aarch64) - rust-lang#88418 (Allow `~const` bounds on trait assoc functions) - rust-lang#88445 (Clean up the lowering of AST items) - rust-lang#88495 (Add `TcpStream::set_linger` and `TcpStream::linger`) - rust-lang#88501 (Use right span in prelude collision suggestions with macros. ) - rust-lang#88504 (Keep turbofish in prelude collision lint.) - rust-lang#88524 (Remove unnecessary `mut` from udp doctests) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
|
I got here by TWIR and I never have seen |
Contributor
|
it's placeholder syntax for "const if const" trait bounds: #67792 the full updated RFC text can be found at https://internals.rust-lang.org/t/pre-rfc-revamped-const-trait-impl-aka-rfc-2632/15192 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
r? @oli-obk