src/tools/clippy/clippy_lints/src/temporary_assignment.rs

   1 use crate::utils::{is_adjusted, span_lint};
   2 use rustc_hir::def::{DefKind, Res};
   3 use rustc_hir::{Expr, ExprKind};
   4 use rustc_lint::{LateContext, LateLintPass};
   5 use rustc_session::{declare_lint_pass, declare_tool_lint};
   6
   7 declare_clippy_lint! {
   8     /// **What it does:** Checks for construction of a structure or tuple just to
   9     /// assign a value in it.
  10     ///
  11     /// **Why is this bad?** Readability. If the structure is only created to be
  12     /// updated, why not write the structure you want in the first place?
  13     ///
  14     /// **Known problems:** None.
  15     ///
  16     /// **Example:**
  17     /// ```rust
  18     /// (0, 0).0 = 1
  19     /// ```
  20     pub TEMPORARY_ASSIGNMENT,
  21     complexity,
  22     "assignments to temporaries"
  23 }
  24
  25 fn is_temporary(cx: &LateContext<'_>, expr: &Expr<'_>) -> bool {
  26     match &expr.kind {
  27         ExprKind::Struct(..) | ExprKind::Tup(..) => true,
  28         ExprKind::Path(qpath) => matches!(cx.qpath_res(qpath, expr.hir_id), Res::Def(DefKind::Const, ..)),
  29         _ => false,
  30     }
  31 }
  32
  33 declare_lint_pass!(TemporaryAssignment => [TEMPORARY_ASSIGNMENT]);
  34
  35 impl<'tcx> LateLintPass<'tcx> for TemporaryAssignment {
  36     fn check_expr(&mut self, cx: &LateContext<'tcx>, expr: &'tcx Expr<'_>) {
  37         if let ExprKind::Assign(target, ..) = &expr.kind {
  38             let mut base = target;
  39             while let ExprKind::Field(f, _) | ExprKind::Index(f, _) = &base.kind {
  40                 base = f;
  41             }
  42             if is_temporary(cx, base) && !is_adjusted(cx, base) {
  43                 span_lint(cx, TEMPORARY_ASSIGNMENT, expr.span, "assignment to temporary");
  44             }
  45         }
  46     }
  47 }