Uh oh!
There was an error while loading. Please reload this page.
Allow ~const bounds on trait assoc functions - #88418
Merged
bors merged 2 commits intoAug 31, 2021
Merged
Conversation
fee1-deadforce-pushed
the
trait-assoc-tilde-const
branch
from
August 28, 2021 08:03
3a18b5f to
523490eCompareoli-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); |
oli-obk
commented
Aug 28, 2021
Contributor
@bors r+ |
bors
commented
Aug 28, 2021
Collaborator
📌 Commit ee02c8e has been approved by |
bors
commented
Aug 30, 2021
Collaborator
⌛ Testing commit ee02c8e with merge 98c17e16ab9f27c1948e0c8d9aa262bfa0800128... |
rust-log-analyzer
commented
Aug 30, 2021
Collaborator
The job Click to see the possible cause of the failure (guessed by this bot) |
bors
commented
Aug 30, 2021
Collaborator
💔 Test failed - checks-actions |
oli-obk
commented
Aug 30, 2021
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
nalply
commented
Sep 15, 2021
I got here by TWIR and I never have seen |
oli-obk
commented
Sep 15, 2021
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 freeto 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