useless_asref: do not lint .as_ref().map(Arc::clone)

[samueltardieu]

This applies to Arc, Rc, and their weak variants. Using .clone() would be less idiomatic.

This follows the discussion in #12528 (comment).

changelog: [useless_asref]: do not lint .as_ref().map(Arc::clone) and similar

[rustbot]

r? @Manishearth

rustbot has assigned @Manishearth.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

[samueltardieu]

r? @y21

[y21]

Makes sense, good idea to move it to utils

r=me with (or without if it doesn't make sense) doc nit fixed

[samueltardieu]

@bors r=y21

[bors]

@samueltardieu: 🔑 Insufficient privileges: Not in reviewers

[samueltardieu]

@bors r=y21

I don't have the needed privileges, maybe you could have used delegate+

[y21]

oops, my bad 😅

Thanks, looks good!

@bors r+

[bors]

📌 Commit fed2f28 has been approved by y21

It is now in the queue for this repository.

[bors]

⌛ Testing commit fed2f28 with merge 12f7c17...

[bors]

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: y21
Pushing 12f7c17 to master...