Skip to content

[unnecessary_lazy_evaluations] Do not suggest switching to early evaluation when type has custom Drop#9551

Merged
bors merged 1 commit intorust-lang:masterfrom
kraktus:unnecessary_lazy_evaluations
Sep 29, 2022
Merged

[unnecessary_lazy_evaluations] Do not suggest switching to early evaluation when type has custom Drop#9551
bors merged 1 commit intorust-lang:masterfrom
kraktus:unnecessary_lazy_evaluations

Conversation

@kraktus
Copy link
Copy Markdown
Contributor

@kraktus kraktus commented Sep 28, 2022

fix #9427

changelog: [unnecessary_lazy_evaluations] Do not suggest switching to early evaluation when type has custom Drop

@rust-highfive
Copy link
Copy Markdown

r? @xFrednet

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 28, 2022
@kraktus kraktus force-pushed the unnecessary_lazy_evaluations branch 2 times, most recently from 49661ce to 314d57a Compare September 28, 2022 20:45
@xFrednet
Copy link
Copy Markdown
Contributor

Looks good to me, thank you for the fix 🙃

@bors r+

@bors
Copy link
Copy Markdown
Contributor

bors commented Sep 29, 2022

📌 Commit 314d57a has been approved by xFrednet

It is now in the queue for this repository.

@bors
Copy link
Copy Markdown
Contributor

bors commented Sep 29, 2022

⌛ Testing commit 314d57a with merge 5e8a4b6...

@bors
Copy link
Copy Markdown
Contributor

bors commented Sep 29, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: xFrednet
Pushing 5e8a4b6 to master...

@Jules-Bertholet
Copy link
Copy Markdown
Contributor

As mentioned in #9427 (comment), this PR did not fully fix the issue.

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.

unnecessary_lazy_evaluations: Shouldn't warn about values with significant Drop impl

5 participants