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_undescore_lock false-positive when binding without locking #8990

Merged
merged 1 commit into from
Jun 27, 2022

Conversation

tsoutsman
Copy link
Contributor

Fixes #8486.

changelog: Fix let_undescore_lock false-positive when binding without locking.

@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 Jun 13, 2022
@llogiq
Copy link
Contributor

llogiq commented Jun 25, 2022

Thank you! @bors r+

@bors
Copy link
Collaborator

bors commented Jun 25, 2022

📌 Commit 7b8adb2 has been approved by llogiq

bors added a commit that referenced this pull request Jun 25, 2022
Fix `let_undescore_lock` false-positive when binding without locking

Fixes #8486.

changelog: Fix `let_undescore_lock` false-positive when binding without locking.
@bors
Copy link
Collaborator

bors commented Jun 25, 2022

⌛ Testing commit 7b8adb2 with merge 8cd9256...

@bors
Copy link
Collaborator

bors commented Jun 25, 2022

💔 Test failed - checks-action_remark_test

@llogiq
Copy link
Contributor

llogiq commented Jun 26, 2022

Rebasing might help as #9055 was just merged.

Signed-off-by: Klim Tsoutsman <klimusha@gmail.com>
@tsoutsman
Copy link
Contributor Author

Thanks for the hint, @llogiq; it should be good now.

@llogiq
Copy link
Contributor

llogiq commented Jun 27, 2022

@bors r+

@bors
Copy link
Collaborator

bors commented Jun 27, 2022

📌 Commit 65f700f has been approved by llogiq

@bors
Copy link
Collaborator

bors commented Jun 27, 2022

⌛ Testing commit 65f700f with merge 889b361...

@bors
Copy link
Collaborator

bors commented Jun 27, 2022

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

@bors bors merged commit 889b361 into rust-lang:master Jun 27, 2022
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::let_underscore_lock false-positive when binding mutex without locking
4 participants