avoid unnecessary nested conditionals

rust-lang · Nov 8, 2023 · 3058865 · 3058865
1 parent 03b24f2
commit 3058865
Showing 1 changed file with 9 additions and 11 deletions.
diff --git a/compiler/rustc_mir_build/src/thir/pattern/const_to_pat.rs b/compiler/rustc_mir_build/src/thir/pattern/const_to_pat.rs
@@ -228,17 +228,15 @@ impl<'tcx> ConstToPat<'tcx> {
                         }
                     }
                 }
-            } else if !self.saw_const_match_lint.get() {
-                if !have_valtree {
-                    // The only way valtree construction can fail without the structural match
-                    // checker finding a violation is if there is a pointer somewhere.
-                    self.tcx().emit_spanned_lint(
-                        lint::builtin::POINTER_STRUCTURAL_MATCH,
-                        self.id,
-                        self.span,
-                        PointerPattern,
-                    );
-                }
+            } else if !have_valtree && !self.saw_const_match_lint.get() {
+                // The only way valtree construction can fail without the structural match
+                // checker finding a violation is if there is a pointer somewhere.
+                self.tcx().emit_spanned_lint(
+                    lint::builtin::POINTER_STRUCTURAL_MATCH,
+                    self.id,
+                    self.span,
+                    PointerPattern,
+                );
             }
 
             // Always check for `PartialEq`, even if we emitted other lints. (But not if there were