Skip to content

Update from_raw_with_void_ptr to support types other than Box #9700

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
Oct 25, 2022

Conversation

andreubotella
Copy link
Contributor

This PR updates the from_raw_with_void_ptr lint, which covered
Box::from_raw, to also cover the from_raw static method of the
Rc, Arc, alloc::rc::Weak and alloc::sync::Weak types.

It also improves the description and error messages of this lint.


changelog: [from_raw_with_void_ptr]: Now works with the Rc, Arc, alloc::rc::Weak and alloc::sync::Weak types.

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @xFrednet (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Oct 23, 2022
@andreubotella
Copy link
Contributor Author

This is a follow-up PR to #9690, btw.

@bors
Copy link
Contributor

bors commented Oct 23, 2022

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

@flip1995
Copy link
Member

That was quick! Thanks!

Please rebase your branch on top of the latest master, instead of producing a merge commit. Let me know, if you need help with that. After that this is good to go.

This PR updates the `from_raw_with_void_ptr` lint, which covered
`Box::from_raw`, to also cover the `from_raw` static method of the
`Rc`, `Arc`, `alloc::rc::Weak` and `alloc::sync::Weak` types.

It also improves the description and error messages of this lint.

---

changelog: [`from_raw_with_void_ptr`]: Now works with the `Rc`, `Arc`, `alloc::rc::Weak` and `alloc::sync::Weak` types.
@andreubotella andreubotella force-pushed the from-raw-with-void-non-box branch from 109c1f5 to e49cde7 Compare October 24, 2022 08:52
@andreubotella
Copy link
Contributor Author

That was quick! Thanks!

Please rebase your branch on top of the latest master, instead of producing a merge commit. Let me know, if you need help with that. After that this is good to go.

Done.

@flip1995
Copy link
Member

@bors r+

Awesome, thanks!

@bors
Copy link
Contributor

bors commented Oct 25, 2022

📌 Commit e49cde7 has been approved by flip1995

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Oct 25, 2022

⌛ Testing commit e49cde7 with merge de87e83...

@bors
Copy link
Contributor

bors commented Oct 25, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing de87e83 to master...

@bors bors merged commit de87e83 into rust-lang:master Oct 25, 2022
@andreubotella andreubotella deleted the from-raw-with-void-non-box branch October 25, 2022 07:58
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
Development

Successfully merging this pull request may close these issues.

5 participants