lint / ImproperCTypes: better handling of indirections, take 2 by niacdoial · Pull Request #134697 · rust-lang/rust (original) (raw)
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})
rustbot added S-waiting-on-review
Status: Awaiting review from the assignee but also interested parties.
Relevant to the compiler team, which will review and decide on the PR/issue.
labels
[commit does not pass tests]
- removed special-case logic for a few cases (including the unit type, which is now only checked for in one place)
- moved a lot of type-checking code in their dedicated visit_* methods
- reworked FfiResult type to handle multiple diagnostics per type (currently imperfect due to type caching)
- reworked the messages around CStr and CString
Comment on lines +170 to +219
Comment on lines +397 to +398
Comment on lines +872 to +873
Comment on lines +893 to +895
Comment on lines +772 to +776