Skip to content

Don't suggest eta-reducible replacements in UNNECESSARY_FOLD lint #2371

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
theotherphil opened this issue Jan 17, 2018 · 0 comments · Fixed by #12210
Closed

Don't suggest eta-reducible replacements in UNNECESSARY_FOLD lint #2371

theotherphil opened this issue Jan 17, 2018 · 0 comments · Fixed by #12210
Labels
C-enhancement Category: Enhancement of lints, like adding more cases or adding help messages

Comments

@theotherphil
Copy link
Contributor

See TODO in lint_unnecessary_fold.
@theotherphil theotherphil mentioned this issue Jan 17, 2018
Merged
4 tasks
@oli-obk oli-obk added the C-enhancement Category: Enhancement of lints, like adding more cases or adding help messages label Feb 1, 2018
@flip1995 flip1995 mentioned this issue Jan 15, 2020
Closed
GuillaumeGomez added a commit to GuillaumeGomez/rust-clippy that referenced this issue Jan 29, 2024
@GuillaumeGomez
Add regression test for rust-lang#2371
b2f2080
@GuillaumeGomez GuillaumeGomez mentioned this issue Jan 29, 2024
Merged
bors added a commit that referenced this issue Jan 29, 2024
@bors
Auto merge of #12210 - GuillaumeGomez:add-regression-test-2371, r=bly…
c148bce 
…xyas

Add regression ui test for #2371

Fixes #2371.

#2371 seems to already be handled correctly in the lint. This PR adds a ui regression test so we can close it.

r? `@blyxyas`

changelog: Add regression ui test for #2371
@bors bors closed this as completed in 455c07b Jan 29, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
C-enhancement Category: Enhancement of lints, like adding more cases or adding help messages
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add regression ui test for #2371 GuillaumeGomez/rust-clippy
2 participants
@oli-obk @theotherphil