Skip to content
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

Fix new_return_no_self with recursive bounds #10086

Merged
merged 1 commit into from
Dec 16, 2022
Merged

Conversation

nbdd0121
Copy link
Contributor

Fix #10041

This uses a hash set, as described in #10068 (comment)

changelog: [new_return_no_self]: fix stack overflow when the return type is impl Trait and contains recursive bounds

@rustbot
Copy link
Collaborator

rustbot commented Dec 15, 2022

r? @llogiq

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Dec 15, 2022
@nbdd0121
Copy link
Contributor Author

r? @matthiaskrgr

cc @Nashenas88 @dswij

@rustbot rustbot assigned matthiaskrgr and unassigned llogiq Dec 15, 2022
Copy link
Member

@matthiaskrgr matthiaskrgr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!
I ran patched / unpatched clippy through lintcheck in order to see if there were any other side effects in lint output but couldn't find anything (hoping I didn't use it wrongly somehow, it has been a while... ^^' )

@matthiaskrgr
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 16, 2022

📌 Commit 7574c98 has been approved by matthiaskrgr

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Dec 16, 2022

⌛ Testing commit 7574c98 with merge 02f3959...

@bors
Copy link
Collaborator

bors commented Dec 16, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: matthiaskrgr
Pushing 02f3959 to master...

@bors bors merged commit 02f3959 into rust-lang:master Dec 16, 2022
@dswij dswij added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Dec 16, 2022
@dswij
Copy link
Member

dswij commented Dec 16, 2022

@flip1995 IIRC, we agreed this should be backported in Zulip. I've added the label, but let me know if this is not the case.

@flip1995
Copy link
Member

flip1995 commented Dec 16, 2022

Thanks for taking care of this! I'll will do the backport in the next few days.

Good thing, I didn't have time to do the sync yesterday, so that gets in earlier 😅

bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 31, 2022
…ulacrum

[beta] backport clippy stack overflow fix

Backports rust-lang/rust-clippy#10086 to the beta branch where the bug is still present.

I just cherry-picked the commit.

cc `@rust-lang/clippy`
@flip1995 flip1995 added beta-accepted Accepted for backporting to the compiler in the beta channel. and removed beta-nominated Nominated for backporting to the compiler in the beta channel. labels Jan 11, 2023
@xFrednet xFrednet removed the beta-accepted Accepted for backporting to the compiler in the beta channel. label Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clippy crashes on simple method returning impl PartialOrd
8 participants