about summary refs log tree commit diff
diff options
context:
space:
mode:
authorYoshitomo Nakanishi <yurayura.rounin.3@gmail.com>2021-03-16 10:37:05 +0900
committerYoshitomo Nakanishi <yurayura.rounin.3@gmail.com>2021-03-16 11:28:53 +0900
commit5a439f5a82815c029c8b2bfa28d9525aaac1c320 (patch)
treef68639cefa32c12240319bf50168e4f2639c28a6
parent6211b49ac1c9a013869943a003efbd5a71a73c93 (diff)
downloadrust-5a439f5a82815c029c8b2bfa28d9525aaac1c320.tar.gz
rust-5a439f5a82815c029c8b2bfa28d9525aaac1c320.zip
Remove unit_types::utils::is_unit
-rw-r--r--clippy_lints/src/unit_types/let_unit_value.rs4
-rw-r--r--clippy_lints/src/unit_types/unit_arg.rs2
-rw-r--r--clippy_lints/src/unit_types/unit_cmp.rs6
-rw-r--r--clippy_lints/src/unit_types/utils.rs5
4 files changed, 6 insertions, 11 deletions
diff --git a/clippy_lints/src/unit_types/let_unit_value.rs b/clippy_lints/src/unit_types/let_unit_value.rs
index 126f6aa746a..55715cc6bf3 100644
--- a/clippy_lints/src/unit_types/let_unit_value.rs
+++ b/clippy_lints/src/unit_types/let_unit_value.rs
@@ -7,11 +7,11 @@ use crate::utils::diagnostics::span_lint_and_then;
 use crate::utils::higher;
 use crate::utils::source::snippet_with_macro_callsite;
 
-use super::{utils, LET_UNIT_VALUE};
+use super::LET_UNIT_VALUE;
 
 pub(super) fn check(cx: &LateContext<'_>, stmt: &Stmt<'_>) {
     if let StmtKind::Local(ref local) = stmt.kind {
-        if utils::is_unit(cx.typeck_results().pat_ty(&local.pat)) {
+        if cx.typeck_results().pat_ty(&local.pat).is_unit() {
             if in_external_macro(cx.sess(), stmt.span) || local.pat.span.from_expansion() {
                 return;
             }
diff --git a/clippy_lints/src/unit_types/unit_arg.rs b/clippy_lints/src/unit_types/unit_arg.rs
index 49b5725f528..d4ef645fc8d 100644
--- a/clippy_lints/src/unit_types/unit_arg.rs
+++ b/clippy_lints/src/unit_types/unit_arg.rs
@@ -36,7 +36,7 @@ pub(super) fn check(cx: &LateContext<'_>, expr: &Expr<'_>) {
             let args_to_recover = args
                 .iter()
                 .filter(|arg| {
-                    if utils::is_unit(cx.typeck_results().expr_ty(arg)) && !utils::is_unit_literal(arg) {
+                    if cx.typeck_results().expr_ty(arg).is_unit() && !utils::is_unit_literal(arg) {
                         !matches!(
                             &arg.kind,
                             ExprKind::Match(.., MatchSource::TryDesugar) | ExprKind::Path(..)
diff --git a/clippy_lints/src/unit_types/unit_cmp.rs b/clippy_lints/src/unit_types/unit_cmp.rs
index e19fc1ec9ef..f9c0374c86d 100644
--- a/clippy_lints/src/unit_types/unit_cmp.rs
+++ b/clippy_lints/src/unit_types/unit_cmp.rs
@@ -4,7 +4,7 @@ use rustc_span::hygiene::{ExpnKind, MacroKind};
 
 use crate::utils::diagnostics::span_lint;
 
-use super::{utils, UNIT_CMP};
+use super::UNIT_CMP;
 
 pub(super) fn check(cx: &LateContext<'_>, expr: &Expr<'_>) {
     if expr.span.from_expansion() {
@@ -12,7 +12,7 @@ pub(super) fn check(cx: &LateContext<'_>, expr: &Expr<'_>) {
             if let ExpnKind::Macro(MacroKind::Bang, symbol) = callee.kind {
                 if let ExprKind::Binary(ref cmp, ref left, _) = expr.kind {
                     let op = cmp.node;
-                    if op.is_comparison() && utils::is_unit(cx.typeck_results().expr_ty(left)) {
+                    if op.is_comparison() && cx.typeck_results().expr_ty(left).is_unit() {
                         let result = match &*symbol.as_str() {
                             "assert_eq" | "debug_assert_eq" => "succeed",
                             "assert_ne" | "debug_assert_ne" => "fail",
@@ -37,7 +37,7 @@ pub(super) fn check(cx: &LateContext<'_>, expr: &Expr<'_>) {
 
     if let ExprKind::Binary(ref cmp, ref left, _) = expr.kind {
         let op = cmp.node;
-        if op.is_comparison() && utils::is_unit(cx.typeck_results().expr_ty(left)) {
+        if op.is_comparison() && cx.typeck_results().expr_ty(left).is_unit() {
             let result = match op {
                 BinOpKind::Eq | BinOpKind::Le | BinOpKind::Ge => "true",
                 _ => "false",
diff --git a/clippy_lints/src/unit_types/utils.rs b/clippy_lints/src/unit_types/utils.rs
index a15a7b812c5..4e194a05e8d 100644
--- a/clippy_lints/src/unit_types/utils.rs
+++ b/clippy_lints/src/unit_types/utils.rs
@@ -1,9 +1,4 @@
 use rustc_hir::{Expr, ExprKind};
-use rustc_middle::ty::{self, Ty};
-
-pub(super) fn is_unit(ty: Ty<'_>) -> bool {
-    matches!(ty.kind(), ty::Tuple(slice) if slice.is_empty())
-}
 
 pub(super) fn is_unit_literal(expr: &Expr<'_>) -> bool {
     matches!(expr.kind, ExprKind::Tup(ref slice) if slice.is_empty())