Don't use CURRENT_DEPTH in clean/types.rs

This part actually wasn't that hard. Ending the use of it in
`html/format.rs` is probably going to be *a lot* harder.

Author: camelid

src/librustdoc/clean/types.rs

@@ -190,8 +190,8 @@ impl Item {
         self.attrs.collapsed_doc_value()
     }
 
-    crate fn links(&self, cache: &Cache) -> Vec<RenderedLink> {
-        self.attrs.links(self.def_id.krate, cache)
+    crate fn links(&self, cache: &Cache, depth: usize) -> Vec<RenderedLink> {
+        self.attrs.links(self.def_id.krate, cache, depth)
     }
 
     crate fn is_crate(&self) -> bool {
@@ -847,9 +847,8 @@ impl Attributes {
     /// Gets links as a vector
     ///
     /// Cache must be populated before call
-    crate fn links(&self, krate: CrateNum, cache: &Cache) -> Vec<RenderedLink> {
+    crate fn links(&self, krate: CrateNum, cache: &Cache, depth: usize) -> Vec<RenderedLink> {
         use crate::html::format::href;
-        use crate::html::render::CURRENT_DEPTH;
 
         self.links
             .iter()
@@ -873,10 +872,7 @@ impl Attributes {
                     None => {
                         if let Some(ref fragment) = *fragment {
                             let url = match cache.extern_locations.get(&krate) {
-                                Some(&(_, _, ExternalLocation::Local)) => {
-                                    let depth = CURRENT_DEPTH.with(|l| l.get());
-                                    "../".repeat(depth)
-                                }
+                                Some(&(_, _, ExternalLocation::Local)) => "../".repeat(depth),
                                 Some(&(_, _, ExternalLocation::Remote(ref s))) => s.to_string(),
                                 Some(&(_, _, ExternalLocation::Unknown)) | None => String::from(
                                     // NOTE: intentionally doesn't pass crate name to avoid having

src/librustdoc/html/render/context.rs

@@ -71,6 +71,10 @@ crate struct Context<'tcx> {
 }
 
 impl<'tcx> Context<'tcx> {
+    pub(super) fn depth(&self) -> usize {
+        self.current.len()
+    }
+
     pub(super) fn path(&self, filename: &str) -> PathBuf {
         // We use splitn vs Path::extension here because we might get a filename
         // like `style.min.css` and we want to process that into

src/librustdoc/html/render/mod.rs

@@ -670,7 +670,8 @@ fn document_short(
         return;
     }
     if let Some(s) = item.doc_value() {
-        let mut summary_html = MarkdownSummaryLine(&s, &item.links(&cx.cache)).into_string();
+        let mut summary_html =
+            MarkdownSummaryLine(&s, &item.links(&cx.cache, cx.depth())).into_string();
 
         if s.contains('\n') {
             let link =
@@ -709,7 +710,7 @@ fn document_full(
 ) {
     if let Some(s) = cx.shared.maybe_collapsed_doc_value(item) {
         debug!("Doc block: =====\n{}\n=====", s);
-        render_markdown(w, cx, &*s, item.links(&cx.cache), prefix, is_hidden);
+        render_markdown(w, cx, &*s, item.links(&cx.cache, cx.depth()), prefix, is_hidden);
     } else if !prefix.is_empty() {
         if is_hidden {
             w.write_str("<div class=\"docblock hidden\">");
@@ -1487,7 +1488,7 @@ fn render_impl(
                 "<div class=\"docblock\">{}</div>",
                 Markdown(
                     &*dox,
-                    &i.impl_item.links(&cx.cache),
+                    &i.impl_item.links(&cx.cache, cx.depth()),
                     &mut ids,
                     cx.shared.codes,
                     cx.shared.edition,

src/librustdoc/html/render/print_item.rs

@@ -297,7 +297,8 @@ fn item_module(w: &mut Buffer, cx: &Context<'_>, item: &clean::Item, items: &[cl
                      </tr>",
                     name = *myitem.name.as_ref().unwrap(),
                     stab_tags = extra_info_tags(myitem, item, cx.tcx()),
-                    docs = MarkdownSummaryLine(&doc_value, &myitem.links(&cx.cache)).into_string(),
+                    docs = MarkdownSummaryLine(&doc_value, &myitem.links(&cx.cache, cx.depth()))
+                        .into_string(),
                     class = myitem.type_(),
                     add = add,
                     stab = stab.unwrap_or_else(String::new),