-
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
E0132 update #35264
E0132 update #35264
Conversation
@bors r+ |
📌 Commit 1607d5b has been approved by |
@bors r+ rollup |
💡 This pull request was already approved, no need to approve it again.
|
📌 Commit 1607d5b has been approved by |
} | ||
if let Some(ref mut span) = span { | ||
span.lo = span.lo - BytePos(1); | ||
span.hi = span.hi + BytePos(1); |
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.
Doesn't account for
fn f< T >() {}
^^^
I decided to omit this manual span expansion in my implementation of this method. #35257
…thandturner E0132 update Fixes rust-lang#35258. r? @jonathandturner
…thandturner E0132 update Fixes rust-lang#35258. r? @jonathandturner
…thandturner E0132 update Fixes rust-lang#35258. r? @jonathandturner
…thandturner E0132 update Fixes rust-lang#35258. r? @jonathandturner
Rollup of 30 pull requests - Successful merges: #34319, #35041, #35042, #35076, #35109, #35137, #35175, #35181, #35182, #35189, #35239, #35264, #35266, #35281, #35285, #35289, #35291, #35294, #35296, #35297, #35298, #35299, #35318, #35319, #35324, #35326, #35328, #35333, #35359, #35362 - Failed merges:
Fixes #35258.
r? @jonathandturner