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 useless_transmute on types with erased regions #8564

Merged
merged 2 commits into from
May 31, 2022

Conversation

Jarcho
Copy link
Contributor

@Jarcho Jarcho commented Mar 18, 2022

fixes #6356
fixes #3340
fixes #2906

This should get a proper fix at some point, but this at least gets the lint running on some types.

cc #5343

changelog: Don't lint useless_transmute on types with erased regions

@rust-highfive
Copy link

r? @camsteffen

(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 18, 2022
@bors
Copy link
Collaborator

bors commented Mar 30, 2022

☔ The latest upstream changes (presumably #8610) made this pull request unmergeable. Please resolve the merge conflicts.

@Alexendoo
Copy link
Member

I'll take a look at this one @xFrednet

@xFrednet xFrednet assigned Alexendoo and unassigned camsteffen May 28, 2022
@xFrednet
Copy link
Member

xFrednet commented May 28, 2022

Thank you, @Alexendoo, with this one you can r+ it once you think it looks good 🙃

@bors delegate=Alexendoo

@bors
Copy link
Collaborator

bors commented May 28, 2022

✌️ @Alexendoo can now approve this pull request

Copy link
Member

@Alexendoo Alexendoo left a comment

Choose a reason for hiding this comment

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

Could add the relevant tests from #2906 as well

@Alexendoo
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented May 31, 2022

📌 Commit 0c6ebf1 has been approved by Alexendoo

@bors
Copy link
Collaborator

bors commented May 31, 2022

⌛ Testing commit 0c6ebf1 with merge 7000e75...

@bors
Copy link
Collaborator

bors commented May 31, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Alexendoo
Pushing 7000e75 to master...

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
6 participants