rust-clippy icon indicating copy to clipboard operation
rust-clippy copied to clipboard

`redundant_closure` fixes

Open Jarcho opened this issue 2 years ago • 6 comments

fixes #8548

A good chunk of the code is fixing false negatives. The old code banned any non late-bound regions from appearing in the callee's signature. The new version checks when the late-bound region is actually required.

changelog: Better track when a early-bound region appears when a late-bound region is required in redundant_closure. changelog: Don't lint redundant_closure when the closure gives explicit types.

Jarcho avatar Apr 11 '22 16:04 Jarcho

r? @Manishearth

(rust-highfive has picked a reviewer for you, use r? to override)

rust-highfive avatar Apr 11 '22 16:04 rust-highfive

may not have time to get to this quickly, bouncing it over, feel free to bounce to someone else

Manishearth avatar Apr 11 '22 20:04 Manishearth

:umbrella: The latest upstream changes (presumably #8676) made this pull request unmergeable. Please resolve the merge conflicts.

bors avatar Apr 14 '22 05:04 bors

r? @flip1995

m-ou-se avatar Aug 09 '22 16:08 m-ou-se

I will be on $day_job vacation starting Saturday and then finally have time to go through my whole review queue over the weekend and next week :tada:

flip1995 avatar Aug 10 '22 12:08 flip1995

:umbrella: The latest upstream changes (presumably #9447) made this pull request unmergeable. Please resolve the merge conflicts.

bors avatar Sep 08 '22 19:09 bors

ping @flip1995. Just a reminder this still exists.

Jarcho avatar Feb 25 '23 02:02 Jarcho

:umbrella: The latest upstream changes (presumably #10401) made this pull request unmergeable. Please resolve the merge conflicts.

bors avatar Feb 26 '23 00:02 bors

:umbrella: The latest upstream changes (presumably #10473) made this pull request unmergeable. Please resolve the merge conflicts.

bors avatar Mar 10 '23 09:03 bors

:umbrella: The latest upstream changes (presumably #10520) made this pull request unmergeable. Please resolve the merge conflicts.

bors avatar Mar 21 '23 19:03 bors

Since @flip1995 doesn't really have time.

r? @llogiq

Jarcho avatar Jun 09 '23 23:06 Jarcho

:umbrella: The latest upstream changes (presumably #11048) made this pull request unmergeable. Please resolve the merge conflicts.

bors avatar Jun 30 '23 15:06 bors

Took me a while to read through, but looks good to me. r=me with the conflicts fixed.

@bors delegate

llogiq avatar Jul 23 '23 10:07 llogiq

Oops, missed a plus sign.

@bors delegate+

llogiq avatar Jul 23 '23 11:07 llogiq

:v: @Jarcho, you can now approve this pull request!

If @llogiq told you to "r=me" after making some further change, please make that change, then do @bors r=@llogiq

bors avatar Jul 23 '23 11:07 bors

The terrible diff probably didn't help much with that.

@bors r=@llogiq

Jarcho avatar Jul 30 '23 05:07 Jarcho

:pushpin: Commit 7423c2760b0dbfd4379d2c3eac309d660b859f41 has been approved by llogiq

It is now in the queue for this repository.

bors avatar Jul 30 '23 05:07 bors

:hourglass: Testing commit 7423c2760b0dbfd4379d2c3eac309d660b859f41 with merge 4c2f460dccf78b9f94df0a235de6ea10b0d495d3...

bors avatar Jul 30 '23 05:07 bors

:sunny: Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test Approved by: llogiq Pushing 4c2f460dccf78b9f94df0a235de6ea10b0d495d3 to master...

bors avatar Jul 30 '23 05:07 bors