Skip to content

Conversation

ada4a
Copy link
Contributor

@ada4a ada4a commented Sep 24, 2025

While looking into #15569, I stumbled upon clippy_utils::ptr.

This module was created in #2117, to share the logic from ptr_arg with needless_pass_by_value. But then later, #8409 removed the use of the logic from ptr_arg, which left needless_pass_by_value as the only user of this module.

Still, I wanted to try to add docs to the module, but the two functions looked all too specific, so I thought it'd be better to just move them to needless_pass_by_value, in the hopes that whoever is looking at that lint will hopefully have enough context to understand what those functions are doing.

changelog: none

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 24, 2025
@rustbot
Copy link
Collaborator

rustbot commented Sep 24, 2025

r? @llogiq

rustbot has assigned @llogiq.
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

Copy link
Member

@samueltardieu samueltardieu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r? samueltardieu @rustbot author

View changes since this review

@rustbot rustbot assigned samueltardieu and unassigned llogiq Sep 25, 2025
@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Sep 25, 2025
@rustbot
Copy link
Collaborator

rustbot commented Sep 25, 2025

Reminder, once the PR becomes ready for a review, use @rustbot ready.

..and make it less generic, thanks to all the callers being known
@ada4a
Copy link
Contributor Author

ada4a commented Sep 25, 2025

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties and removed S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) labels Sep 25, 2025
@samueltardieu samueltardieu added this pull request to the merge queue Sep 25, 2025
Merged via the queue into rust-lang:master with commit 24b0282 Sep 25, 2025
11 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 25, 2025
@ada4a ada4a deleted the utils-ptr branch September 25, 2025 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants