Handle RTN projections in assoc type restriction diagnostics#154932
Handle RTN projections in assoc type restriction diagnostics#154932TaKO8Ki wants to merge 3 commits intorust-lang:mainfrom
Conversation
|
r? @jackh726 rustbot has assigned @jackh726. Use Why was this reviewer chosen?The reviewer was selected based on:
|
There was a problem hiding this comment.
As I said before, could you please further reduce any potential reproducers if they're not minimal enough yet (e.g., by removing unnecessary elements and unnecessary user errors)?
This would be tremendously helpful for other contributors to reason about it. They might want to extend this test file with more test cases years down the line or they see this test failing after modifying the compiler in which case it's very beneficial if the test is easy to digest instead of being a giant blob you first have to wade through. The extra fluff certainly won't increase test coverage by much.
E.g., the code below can be shrunk to the following (maybe even further) while still reproducing the ICE:
#![feature(return_type_notation)]
pub trait Trait { async fn func(); }
impl<T: Trait<func(..): Send>> Trait for T {}
fn check(_: impl Trait) {}
fn main() { check(()); }There was a problem hiding this comment.
Oh. Sorry, I completely missed your earlier comment. I’ll reduce the reproducers further.
Fixes #152887