2017-02-04 22:07:54 -06:00
|
|
|
//! lint when there is an enum with no variants
|
|
|
|
|
2018-05-30 03:15:50 -05:00
|
|
|
use crate::utils::span_lint_and_then;
|
2018-12-29 09:04:45 -06:00
|
|
|
use rustc::hir::*;
|
|
|
|
use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
|
|
|
|
use rustc::{declare_tool_lint, lint_array};
|
2017-02-04 22:07:54 -06:00
|
|
|
|
|
|
|
/// **What it does:** Checks for `enum`s with no variants.
|
|
|
|
///
|
2017-08-09 02:30:56 -05:00
|
|
|
/// **Why is this bad?** Enum's with no variants should be replaced with `!`,
|
|
|
|
/// the uninhabited type,
|
2017-02-04 23:09:54 -06:00
|
|
|
/// or a wrapper around it.
|
2017-02-04 22:07:54 -06:00
|
|
|
///
|
|
|
|
/// **Known problems:** None.
|
|
|
|
///
|
|
|
|
/// **Example:**
|
|
|
|
/// ```rust
|
|
|
|
/// enum Test {}
|
|
|
|
/// ```
|
2018-03-28 08:24:26 -05:00
|
|
|
declare_clippy_lint! {
|
2017-02-04 22:07:54 -06:00
|
|
|
pub EMPTY_ENUM,
|
2018-03-28 08:24:26 -05:00
|
|
|
pedantic,
|
2017-02-04 22:07:54 -06:00
|
|
|
"enum with no variants"
|
|
|
|
}
|
|
|
|
|
2017-08-09 02:30:56 -05:00
|
|
|
#[derive(Copy, Clone)]
|
2017-02-04 22:07:54 -06:00
|
|
|
pub struct EmptyEnum;
|
|
|
|
|
|
|
|
impl LintPass for EmptyEnum {
|
|
|
|
fn get_lints(&self) -> LintArray {
|
|
|
|
lint_array!(EMPTY_ENUM)
|
|
|
|
}
|
2019-01-26 13:40:55 -06:00
|
|
|
|
|
|
|
fn name(&self) -> &'static str {
|
|
|
|
"EmptyEnum"
|
|
|
|
}
|
2017-02-04 22:07:54 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
impl<'a, 'tcx> LateLintPass<'a, 'tcx> for EmptyEnum {
|
2018-07-23 06:01:12 -05:00
|
|
|
fn check_item(&mut self, cx: &LateContext<'_, '_>, item: &Item) {
|
2018-12-07 18:56:03 -06:00
|
|
|
let did = cx.tcx.hir().local_def_id(item.id);
|
2018-07-16 08:07:39 -05:00
|
|
|
if let ItemKind::Enum(..) = item.node {
|
2017-04-27 07:00:35 -05:00
|
|
|
let ty = cx.tcx.type_of(did);
|
2018-11-27 14:14:15 -06:00
|
|
|
let adt = ty.ty_adt_def().expect("already checked whether this is an enum");
|
2017-02-04 22:52:44 -06:00
|
|
|
if adt.variants.is_empty() {
|
|
|
|
span_lint_and_then(cx, EMPTY_ENUM, item.span, "enum with no variants", |db| {
|
2018-11-27 14:14:15 -06:00
|
|
|
db.span_help(
|
|
|
|
item.span,
|
|
|
|
"consider using the uninhabited type `!` or a wrapper around it",
|
|
|
|
);
|
2017-02-04 22:07:54 -06:00
|
|
|
});
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|