From ee9188fcaa289c97872767c9c33a2e640a88ada1 Mon Sep 17 00:00:00 2001
From: Jason Newcomb <jsnewcomb@pm.me>
Date: Fri, 2 Apr 2021 19:04:45 -0400
Subject: [PATCH] `redundant_pattern_matching` fix inverted boolean when
 missing `Drop` trait

# Please enter the commit message for your changes.
 Lines starting
# with '#' will be ignored, and an empty message aborts the
 commit.
#
# Date:      Fri Apr 2 19:04:45 2021 -0400
#
# On branch
 redundant_pattern_matching
# Your branch is ahead of
 'origin/redundant_pattern_matching' by 1 commit.
#   (use "git push" to
 publish your local commits)
#
# Changes to be committed:
#	modified:  
 clippy_lints/src/matches.rs
#

# Please enter the commit message for your
 changes. Lines starting
# with '#' will be ignored, and an empty message
 aborts the commit.
#
# Date:      Fri Apr 2 19:04:45 2021 -0400
#
#
 interactive rebase in progress; onto ebc64690d
# Last commands done (6
 commands done):
#    pick 25d211ad8 Code cleanup and additional std types
 checked
#    r 0c71ce56f `redundant_pattern_matching` fix inverted boolean
 when missing `Drop` trait
# No commands remaining.
# You are currently
 editing a commit while rebasing branch 'redundant_pattern_matching' on
 'ebc64690d'.
#
# Changes to be committed:
#	modified:  
 clippy_lints/src/matches.rs
#

---
 clippy_lints/src/matches.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/clippy_lints/src/matches.rs b/clippy_lints/src/matches.rs
index 3aa70ea93302..6a71f3ffbaad 100644
--- a/clippy_lints/src/matches.rs
+++ b/clippy_lints/src/matches.rs
@@ -1737,11 +1737,11 @@ mod redundant_pattern_match {
     fn type_needs_ordered_drop(cx: &LateContext<'tcx>, ty: Ty<'tcx>) -> bool {
         if !ty.needs_drop(cx.tcx, cx.param_env) {
             false
-        } else if cx
+        } else if !cx
             .tcx
             .lang_items()
             .drop_trait()
-            .map_or(false, |id| !implements_trait(cx, ty, id, &[]))
+            .map_or(false, |id| implements_trait(cx, ty, id, &[]))
         {
             // This type doesn't implement drop, so no side effects here.
             // Check if any component type has any.