Added PhantomPinned diagnostic item and prevented dead field warning on PhantomPinned#157782
Added PhantomPinned diagnostic item and prevented dead field warning on PhantomPinned#157782asder8215 wants to merge 1 commit into
PhantomPinned diagnostic item and prevented dead field warning on PhantomPinned#157782Conversation
|
cc @rust-lang/clippy |
|
r? @BoxyUwU rustbot has assigned @BoxyUwU. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
r? clippy |
|
Reminder, once the PR becomes ready for a review, use |
|
Also, in my experience, when needing to add a diagnostic item for clippy, one usually makes a PR to rustc adding the diagnostic item, waits for a sync, and then does the remaining changes as a PR to clippy. Not sure how strongly this rule is followed though – I haven't been around for that long^^ |
Yeah, I'm aware that changes to clippy is separated from PRs made to the rust repo. But I also figured that this change was small enough and quicker to do atomically here. |
68bcbdf to
808927e
Compare
808927e to
3abea2f
Compare
This PR closes #154888. See #154978 and in clippy#17056 for prior history on working on this issue.
From discussing with clippy team, we thought it'd be appropriate to use a diagnostic item attribute to mark
PhantomPinnedand prevent dead field warnings from being emitted inmissing_fields_in_debug.rsandpub_underscore_fields.rs.If you want me to do the clippy changes separately within the clippy repository, I can do that.