Simplify loop and remove old debugging code

Co-authored-by: Joshua Nelson <[email protected]>

Author: camelid

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
     }
 }