-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Add error explanations for E0072, E0073, E0121, E0178, E0371, E0372. #25501
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
One way to fix this is by wrapping `ByteList` in a `Box`, like so: | ||
|
||
``` | ||
struct ByteList { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The name changed from ListNode to ByteList here, was that intentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh, oops! Fixed.
You're a machine @nham! Thanks so much for these! |
r=me with a rebase |
rebased |
Hm, github still says it can't be merged? |
Doh! 😳 I squashed instead of rebasing. Didnt even see there was a merge conflict. Should be rebased now. |
Ahh hold on, it's still borked. |
Alright, my brain broke for a bit there, but it looks good now. |
cc #24407