diff options
| author | Nicholas Nethercote <n.nethercote@gmail.com> | 2022-10-27 14:02:18 +1100 |
|---|---|---|
| committer | Nicholas Nethercote <n.nethercote@gmail.com> | 2022-10-29 20:28:38 +1100 |
| commit | efca32e23ab1cc27cd6b1e5ec42faf848fea5fc3 (patch) | |
| tree | 803f16e6f0ef7526fec1114e4aa7349c29432a2a /clippy_lints/src/empty_enum.rs | |
| parent | 26eeeeec7610e45975745a60226dc8a980ed684e (diff) | |
| download | rust-efca32e23ab1cc27cd6b1e5ec42faf848fea5fc3.tar.gz rust-efca32e23ab1cc27cd6b1e5ec42faf848fea5fc3.zip | |
Rename some `OwnerId` fields.
spastorino noticed some silly expressions like `item_id.def_id.def_id`. This commit renames several `def_id: OwnerId` fields as `owner_id`, so those expressions become `item_id.owner_id.def_id`. `item_id.owner_id.local_def_id` would be even clearer, but the use of `def_id` for values of type `LocalDefId` is *very* widespread, so I left that alone.
Diffstat (limited to 'clippy_lints/src/empty_enum.rs')
| -rw-r--r-- | clippy_lints/src/empty_enum.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clippy_lints/src/empty_enum.rs b/clippy_lints/src/empty_enum.rs index bbebc024414..0570c2a1013 100644 --- a/clippy_lints/src/empty_enum.rs +++ b/clippy_lints/src/empty_enum.rs @@ -49,7 +49,7 @@ impl<'tcx> LateLintPass<'tcx> for EmptyEnum { } if let ItemKind::Enum(..) = item.kind { - let ty = cx.tcx.type_of(item.def_id); + let ty = cx.tcx.type_of(item.owner_id); let adt = ty.ty_adt_def().expect("already checked whether this is an enum"); if adt.variants().is_empty() { span_lint_and_help( |