Skip to content

move crosspointer_transmute from complexity to suspicious #15403

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

Merged
merged 1 commit into from
Aug 8, 2025

Conversation

ada4a
Copy link
Contributor

@ada4a ada4a commented Aug 3, 2025

changelog: [crosspointer_transmute]: move from complexity to suspicious

this looks to me like an obvious miscategorisation -- and there was already a proposal to move the lint

@rustbot
Copy link
Collaborator

rustbot commented Aug 3, 2025

r? @samueltardieu

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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 3, 2025
@samueltardieu
Copy link
Member

@ada4a
Copy link
Contributor Author

ada4a commented Aug 3, 2025

oops, forgot about that. thank you:)

@samueltardieu
Copy link
Member

samueltardieu commented Aug 3, 2025

oops, forgot about that. thank you:)

No worries, this allowed me to give my view on this move as well. I'll probably merge this PR on Friday if no team members object in the meantime.

@ada4a ada4a force-pushed the crosspointer-transmute branch from 558f9b8 to 7aa7563 Compare August 7, 2025 21:54
@samueltardieu samueltardieu added this pull request to the merge queue Aug 8, 2025
Merged via the queue into rust-lang:master with commit 2208884 Aug 8, 2025
11 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 8, 2025
@ada4a ada4a deleted the crosspointer-transmute branch August 8, 2025 13:12
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.

3 participants