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 let_underscore_drop FP #6682

Merged
merged 2 commits into from
Feb 7, 2021
Merged

Conversation

camsteffen
Copy link
Contributor

changelog: Fix let_underscore_drop false positives and negatives

Fixes #6633

This fixes false positives and false negatives.
@rust-highfive
Copy link

r? @llogiq

(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 Feb 5, 2021
@llogiq
Copy link
Contributor

llogiq commented Feb 7, 2021

Thank you! @bors r+

@bors
Copy link
Contributor

bors commented Feb 7, 2021

📌 Commit 40ce056 has been approved by llogiq

@bors
Copy link
Contributor

bors commented Feb 7, 2021

⌛ Testing commit 40ce056 with merge d792210...

@bors
Copy link
Contributor

bors commented Feb 7, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: llogiq
Pushing d792210 to master...

@bors bors merged commit d792210 into rust-lang:master Feb 7, 2021
@camsteffen camsteffen deleted the let-underscore-ref branch July 8, 2021 22:01
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.

let_underscore_drop false positive for references to types that implement Drop
4 participants