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

Don't lint ptr_arg on &mut Cow<_> #8552

Merged
merged 1 commit into from
Mar 17, 2022
Merged

Conversation

Jarcho
Copy link
Contributor

@Jarcho Jarcho commented Mar 16, 2022

fixes: #8495

changelog: Don't lint ptr_arg on &mut Cow<_>

@rust-highfive
Copy link

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 Mar 16, 2022
xFrednet
xFrednet approved these changes Mar 17, 2022
@xFrednet
Copy link
Member

LGTM, thank you for the fix 🙃

@bors r+

@bors
Copy link
Collaborator

bors commented Mar 17, 2022

📌 Commit 851e715 has been approved by xFrednet

@bors
Copy link
Collaborator

bors commented Mar 17, 2022

⌛ Testing commit 851e715 with merge 8d5c0ea...

@bors
Copy link
Collaborator

bors commented Mar 17, 2022

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

@bors bors merged commit 8d5c0ea into rust-lang:master Mar 17, 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.

ptr_arg suggests not using &mut Cow in ways that are invalid or would add clones
4 participants