-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
More descriptive error when qself path doesnt have a trait on the RHS of as
#109788
More descriptive error when qself path doesnt have a trait on the RHS of as
#109788
Conversation
This PR would close #61729 |
41ae86c
to
b23b7a4
Compare
Had a bit of trouble with @rustbot ready |
b23b7a4
to
d5b1ef1
Compare
@bors r+ |
⌛ Testing commit d5b1ef1 with merge f9ce453b2d4920b5bb93587793c0aaf78e4107c2... |
💔 Test failed - checks-actions |
The job Click to see the possible cause of the failure (guessed by this bot)
|
Spurious? |
…n-trait, r=petrochenkov More descriptive error when qself path doesnt have a trait on the RHS of `as` `<Ty as Enum>::Assoc` should report that `Enum` is a trait. Main question is whether to eagerly report the error, or raise it with `return Err(..)` -- i'll note that in an inline comment though. cc `@GuillaumeGomez` who said this came up at a Paris Rust meetup. r? `@petrochenkov`
…n-trait, r=petrochenkov More descriptive error when qself path doesnt have a trait on the RHS of `as` `<Ty as Enum>::Assoc` should report that `Enum` is a trait. Main question is whether to eagerly report the error, or raise it with `return Err(..)` -- i'll note that in an inline comment though. cc ``@GuillaumeGomez`` who said this came up at a Paris Rust meetup. r? ``@petrochenkov``
☀️ Test successful - checks-actions |
Finished benchmarking commit (23ee2af): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)ResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesThis benchmark run did not return any relevant results for this metric. |
<Ty as Enum>::Assoc
should report thatEnum
is a trait. Main question is whether to eagerly report the error, or raise it withreturn Err(..)
-- i'll note that in an inline comment though.cc @GuillaumeGomez who said this came up at a Paris Rust meetup.
r? @petrochenkov