diff options
| author | bors <bors@rust-lang.org> | 2022-03-18 15:54:45 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2022-03-18 15:54:45 +0000 |
| commit | a31dcb77be146ec04a653cd1e7dee21600b3f29e (patch) | |
| tree | 6094ec5b5366e4f6fb0690380eaa6b55f3735473 /clippy_lints/src/casts/cast_enum_constructor.rs | |
| parent | 9fd3c2d7886a61a10c37f922f6e866e221565add (diff) | |
| parent | 39329d1d404d38437270acf3cb1337a231367efc (diff) | |
| download | rust-a31dcb77be146ec04a653cd1e7dee21600b3f29e.tar.gz rust-a31dcb77be146ec04a653cd1e7dee21600b3f29e.zip | |
Auto merge of #8562 - Jarcho:enum_tuple_variant_as_int, r=Manishearth
Add lint `cast_enum_constructor` fixes: #1116 changelog: Add lint `cast_enum_constructor`
Diffstat (limited to 'clippy_lints/src/casts/cast_enum_constructor.rs')
| -rw-r--r-- | clippy_lints/src/casts/cast_enum_constructor.rs | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/clippy_lints/src/casts/cast_enum_constructor.rs b/clippy_lints/src/casts/cast_enum_constructor.rs new file mode 100644 index 00000000000..1973692e105 --- /dev/null +++ b/clippy_lints/src/casts/cast_enum_constructor.rs @@ -0,0 +1,21 @@ +use clippy_utils::diagnostics::span_lint; +use rustc_hir::def::{CtorKind, CtorOf, DefKind, Res}; +use rustc_hir::{Expr, ExprKind}; +use rustc_lint::LateContext; +use rustc_middle::ty::{self, Ty}; + +use super::CAST_ENUM_CONSTRUCTOR; + +pub(super) fn check(cx: &LateContext<'_>, expr: &Expr<'_>, cast_expr: &Expr<'_>, cast_from: Ty<'_>) { + if matches!(cast_from.kind(), ty::FnDef(..)) + && let ExprKind::Path(path) = &cast_expr.kind + && let Res::Def(DefKind::Ctor(CtorOf::Variant, CtorKind::Fn), _) = cx.qpath_res(path, cast_expr.hir_id) + { + span_lint( + cx, + CAST_ENUM_CONSTRUCTOR, + expr.span, + "cast of an enum tuple constructor to an integer", + ); + } +} |
