src/tools/clippy/clippy_lints/src/methods/single_char_pattern.rs

   1 use super::utils::get_hint_if_single_char_arg;
   2 use clippy_utils::diagnostics::span_lint_and_sugg;
   3 use if_chain::if_chain;
   4 use rustc_errors::Applicability;
   5 use rustc_hir as hir;
   6 use rustc_lint::LateContext;
   7 use rustc_middle::ty;
   8 use rustc_span::symbol::Symbol;
   9
  10 use super::SINGLE_CHAR_PATTERN;
  11
  12 const PATTERN_METHODS: [(&str, usize); 24] = [
  13     ("contains", 1),
  14     ("starts_with", 1),
  15     ("ends_with", 1),
  16     ("find", 1),
  17     ("rfind", 1),
  18     ("split", 1),
  19     ("split_inclusive", 1),
  20     ("rsplit", 1),
  21     ("split_terminator", 1),
  22     ("rsplit_terminator", 1),
  23     ("splitn", 2),
  24     ("rsplitn", 2),
  25     ("split_once", 1),
  26     ("rsplit_once", 1),
  27     ("matches", 1),
  28     ("rmatches", 1),
  29     ("match_indices", 1),
  30     ("rmatch_indices", 1),
  31     ("strip_prefix", 1),
  32     ("strip_suffix", 1),
  33     ("trim_start_matches", 1),
  34     ("trim_end_matches", 1),
  35     ("replace", 1),
  36     ("replacen", 1),
  37 ];
  38
  39 /// lint for length-1 `str`s for methods in `PATTERN_METHODS`
  40 pub(super) fn check(cx: &LateContext<'_>, _expr: &hir::Expr<'_>, method_name: Symbol, args: &[hir::Expr<'_>]) {
  41     for &(method, pos) in &PATTERN_METHODS {
  42         if_chain! {
  43             if let ty::Ref(_, ty, _) = cx.typeck_results().expr_ty_adjusted(&args[0]).kind();
  44             if *ty.kind() == ty::Str;
  45             if method_name.as_str() == method && args.len() > pos;
  46             let arg = &args[pos];
  47             let mut applicability = Applicability::MachineApplicable;
  48             if let Some(hint) = get_hint_if_single_char_arg(cx, arg, &mut applicability);
  49             then {
  50                 span_lint_and_sugg(
  51                     cx,
  52                     SINGLE_CHAR_PATTERN,
  53                     arg.span,
  54                     "single-character string constant used as pattern",
  55                     "try using a `char` instead",
  56                     hint,
  57                     applicability,
  58                 );
  59             }
  60         }
  61     }
  62 }