rust-lang / rust-analyzer

A Rust compiler front-end for IDEs
https://rust-analyzer.github.io/
Apache License 2.0
13.62k stars 1.49k forks source link

Fix: Lifetime's Bound Var Debrujin Index in Dyn Traits #17190

Closed dfireBird closed 1 week ago

dfireBird commented 1 week ago

Surely fixes #17182

I have tried running the analysis-stats in some of the repos mentioned in #17080. No panic in almost all of them.

lnicola commented 1 week ago

Almost? 😄

On my phone, but I don't think I understand this. r? @Veykril, might be good to have in the next release.

dfireBird commented 1 week ago

Almost? 😄

cargo check is failing on the one of the repos due to missing file, so almost :sweat_smile: .

Veykril commented 1 week ago

Judging by the zulip discussion this seems to be right so @bors r+

bors commented 1 week ago

:pushpin: Commit 8cbeb04ba2184bf834e11e7deabae269eeca644d has been approved by Veykril

It is now in the queue for this repository.

bors commented 1 week ago

:hourglass: Testing commit 8cbeb04ba2184bf834e11e7deabae269eeca644d with merge c4618fe14d39992fbbb85c2d6cad028a232c13d2...

bors commented 1 week ago

:sunny: Test successful - checks-actions Approved by: Veykril Pushing c4618fe14d39992fbbb85c2d6cad028a232c13d2 to master...