2018-11-27 14:13:57 +00:00
|
|
|
|
use super::utils::{get_arg_name, match_var, remove_blocks, snippet_with_applicability, span_lint_and_sugg};
|
2018-07-19 08:00:54 +00:00
|
|
|
|
use if_chain::if_chain;
|
2018-12-29 15:04:45 +00:00
|
|
|
|
use rustc::hir::*;
|
|
|
|
|
use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
|
|
|
|
|
use rustc::{declare_tool_lint, lint_array};
|
|
|
|
|
use rustc_errors::Applicability;
|
2018-04-19 19:34:31 +00:00
|
|
|
|
|
|
|
|
|
declare_clippy_lint! {
|
2019-03-05 16:50:33 +00:00
|
|
|
|
/// **What it does:** Checks for matches being used to destructure a single-variant enum
|
|
|
|
|
/// or tuple struct where a `let` will suffice.
|
|
|
|
|
///
|
|
|
|
|
/// **Why is this bad?** Just readability – `let` doesn't nest, whereas a `match` does.
|
|
|
|
|
///
|
|
|
|
|
/// **Known problems:** None.
|
|
|
|
|
///
|
|
|
|
|
/// **Example:**
|
|
|
|
|
/// ```rust
|
|
|
|
|
/// enum Wrapper {
|
|
|
|
|
/// Data(i32),
|
|
|
|
|
/// }
|
|
|
|
|
///
|
|
|
|
|
/// let wrapper = Wrapper::Data(42);
|
|
|
|
|
///
|
|
|
|
|
/// let data = match wrapper {
|
|
|
|
|
/// Wrapper::Data(i) => i,
|
|
|
|
|
/// };
|
|
|
|
|
/// ```
|
|
|
|
|
///
|
|
|
|
|
/// The correct use would be:
|
|
|
|
|
/// ```rust
|
|
|
|
|
/// enum Wrapper {
|
|
|
|
|
/// Data(i32),
|
|
|
|
|
/// }
|
|
|
|
|
///
|
|
|
|
|
/// let wrapper = Wrapper::Data(42);
|
|
|
|
|
/// let Wrapper::Data(data) = wrapper;
|
|
|
|
|
/// ```
|
2018-04-19 19:34:31 +00:00
|
|
|
|
pub INFALLIBLE_DESTRUCTURING_MATCH,
|
|
|
|
|
style,
|
|
|
|
|
"a match statement with a single infallible arm instead of a `let`"
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
#[derive(Copy, Clone, Default)]
|
|
|
|
|
pub struct Pass;
|
|
|
|
|
|
|
|
|
|
impl LintPass for Pass {
|
|
|
|
|
fn get_lints(&self) -> LintArray {
|
|
|
|
|
lint_array!(INFALLIBLE_DESTRUCTURING_MATCH)
|
|
|
|
|
}
|
2019-01-26 19:40:55 +00:00
|
|
|
|
|
|
|
|
|
fn name(&self) -> &'static str {
|
|
|
|
|
"InfallibleDestructingMatch"
|
|
|
|
|
}
|
2018-04-19 19:34:31 +00:00
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
impl<'a, 'tcx> LateLintPass<'a, 'tcx> for Pass {
|
|
|
|
|
fn check_local(&mut self, cx: &LateContext<'a, 'tcx>, local: &'tcx Local) {
|
|
|
|
|
if_chain! {
|
|
|
|
|
if let Some(ref expr) = local.init;
|
2018-07-12 07:30:57 +00:00
|
|
|
|
if let ExprKind::Match(ref target, ref arms, MatchSource::Normal) = expr.node;
|
2018-04-19 19:34:31 +00:00
|
|
|
|
if arms.len() == 1 && arms[0].pats.len() == 1 && arms[0].guard.is_none();
|
|
|
|
|
if let PatKind::TupleStruct(QPath::Resolved(None, ref variant_name), ref args, _) = arms[0].pats[0].node;
|
|
|
|
|
if args.len() == 1;
|
|
|
|
|
if let Some(arg) = get_arg_name(&args[0]);
|
|
|
|
|
let body = remove_blocks(&arms[0].body);
|
|
|
|
|
if match_var(body, arg);
|
|
|
|
|
|
|
|
|
|
then {
|
2018-11-27 14:13:57 +00:00
|
|
|
|
let mut applicability = Applicability::MachineApplicable;
|
2018-04-19 19:34:31 +00:00
|
|
|
|
span_lint_and_sugg(
|
|
|
|
|
cx,
|
|
|
|
|
INFALLIBLE_DESTRUCTURING_MATCH,
|
|
|
|
|
local.span,
|
|
|
|
|
"you seem to be trying to use match to destructure a single infallible pattern. \
|
|
|
|
|
Consider using `let`",
|
|
|
|
|
"try this",
|
|
|
|
|
format!(
|
|
|
|
|
"let {}({}) = {};",
|
2018-11-27 14:13:57 +00:00
|
|
|
|
snippet_with_applicability(cx, variant_name.span, "..", &mut applicability),
|
|
|
|
|
snippet_with_applicability(cx, local.pat.span, "..", &mut applicability),
|
|
|
|
|
snippet_with_applicability(cx, target.span, "..", &mut applicability),
|
2018-04-19 19:34:31 +00:00
|
|
|
|
),
|
2018-11-27 14:13:57 +00:00
|
|
|
|
applicability,
|
2018-04-19 19:34:31 +00:00
|
|
|
|
);
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|