From eaf851288ed31090afd924eeb22f5e0f202151d4 Mon Sep 17 00:00:00 2001
From: Camelid <camelidcamel@gmail.com>
Date: Fri, 25 Dec 2020 16:35:28 -0800
Subject: [PATCH] Simplify loop and remove old debugging code

Co-authored-by: Joshua Nelson <jyn514@gmail.com>
---
 src/librustdoc/clean/utils.rs | 19 +++++--------------
 1 file changed, 5 insertions(+), 14 deletions(-)

diff --git a/src/librustdoc/clean/utils.rs b/src/librustdoc/clean/utils.rs
index 4b9541b7e142..b49ed07f8e83 100644
--- a/src/librustdoc/clean/utils.rs
+++ b/src/librustdoc/clean/utils.rs
@@ -636,21 +636,12 @@ crate fn find_closest_parent_module(tcx: TyCtxt<'_>, def_id: DefId) -> Option<De
         let mut current = def_id;
         // The immediate parent might not always be a module.
         // Find the first parent which is.
-        loop {
-            if let Some(parent) = tcx.parent(current) {
-                if tcx.def_kind(parent) == DefKind::Mod {
-                    break Some(parent);
-                }
-                current = parent;
-            } else {
-                debug!(
-                    "{:?} has no parent (kind={:?}, original was {:?})",
-                    current,
-                    tcx.def_kind(current),
-                    def_id
-                );
-                break None;
+        while let Some(parent) = tcx.parent(current) {
+            if tcx.def_kind(parent) == DefKind::Mod {
+                return Some(parent);
             }
+            current = parent;
         }
+        None
     }
 }