Skip to content

Conversation

ada4a
Copy link
Contributor

@ada4a ada4a commented Oct 13, 2025

Resolves #3440

changelog: [cast_ptr_alignment]: also lint to-types with unknown alignment

Copy link

github-actions bot commented Oct 13, 2025

Lintcheck changes for 3d18841

Lint Added Removed Changed
clippy::cast_ptr_alignment 30 0 1

This comment will be updated if you push new changes

@ada4a ada4a force-pushed the cast_alignment branch 2 times, most recently from 3cc75c8 to 0f51832 Compare October 14, 2025 10:18
Copy link
Member

@Centri3 Centri3 left a comment

Choose a reason for hiding this comment

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

@Centri3
Copy link
Member

Centri3 commented Oct 17, 2025

Considering this is a draft rn, what is still WIP?

@ada4a
Copy link
Contributor Author

ada4a commented Oct 17, 2025

Considering this is a draft rn, what is still WIP?

The implementation should be complete, but I wasn't sure whether this is a desirable change in the first place, which is why I added S-needs-discussion to the original issue. I guess we could start an FCP

@Centri3 Centri3 added the I-nominated Issue: Nominated to be discussed at the next Clippy meeting label Oct 20, 2025
@Centri3
Copy link
Member

Centri3 commented Oct 20, 2025

Instead of an FCP, I'm gonna nominate this for the next meeting. Feel free to make a Zulip thread instead tho if you want!

@Centri3 Centri3 removed the I-nominated Issue: Nominated to be discussed at the next Clippy meeting label Oct 20, 2025
@Centri3
Copy link
Member

Centri3 commented Oct 20, 2025

Oh cool, checked Zulip and you already did it!

@ada4a
Copy link
Contributor Author

ada4a commented Oct 20, 2025

Yes, sorry, forgot to post the link here 😅 There wasn't too much activity there yet though, so it could be discussed a bit more thoroughly in the meeting:)

@Centri3 Centri3 added the I-nominated Issue: Nominated to be discussed at the next Clippy meeting label Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

I-nominated Issue: Nominated to be discussed at the next Clippy meeting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cast_ptr_alignment false negative in trait method default impl

2 participants