Temporary change to confirm with code reviewer the next steps

Author: maxcnunes

clippy_lints/src/doc/broken_link.rs

@@ -1,6 +1,8 @@
 use clippy_utils::diagnostics::span_lint;
+use pulldown_cmark::BrokenLink as PullDownBrokenLink;
 use rustc_ast::{AttrKind, AttrStyle, Attribute};
 use rustc_lint::LateContext;
+use rustc_resolve::rustdoc::DocFragment;
 use rustc_span::{BytePos, Span};
 
 use super::DOC_BROKEN_LINK;
@@ -9,6 +11,19 @@ pub fn check(cx: &LateContext<'_>, attrs: &[Attribute]) {
     BrokenLinkReporter::warn_if_broken_links(cx, attrs);
 }
 
+// NOTE: temporary change to check if we can handle broken links from pulldown_cmark parser.
+pub fn check_v2(_cx: &LateContext<'_>, bl: &PullDownBrokenLink<'_>, doc: &String, fragments: &Vec<DocFragment>) {
+    log(format!("\n ---------------------",).as_str());
+    log(format!("\n doc={doc:#?}",).as_str());
+    log(format!("\n fragments={fragments:#?}",).as_str());
+
+    log(format!("\n bl={bl:#?}",).as_str());
+
+    let text: String = doc[bl.span.clone()].chars().collect();
+    log(format!("\n text based on 'bl.span' range={text:#?}",).as_str());
+    log(format!("\n ---------------------",).as_str());
+}
+
 /// The reason why a link is considered broken.
 // NOTE: We don't check these other cases because
 // rustdoc itself will check and warn about it:
@@ -155,3 +170,16 @@ impl BrokenLinkReporter {
         }
     }
 }
+
+// TODO: remove this helper function once all changes are good.
+fn log(text: &str) {
+    use std::fs::OpenOptions;
+    use std::io::Write;
+
+    let filename = "../rust-clippy-debug-test.txt";
+    let mut file = OpenOptions::new().write(true).append(true).open(filename).unwrap();
+
+    if let Err(e) = writeln!(file, "{text}") {
+        eprintln!("Couldn't write to file: {}", e);
+    }
+}

clippy_lints/src/doc/mod.rs

@@ -781,14 +781,6 @@ struct DocHeaders {
 /// back in the various late lint pass methods if they need the final doc headers, like "Safety" or
 /// "Panics" sections.
 fn check_attrs(cx: &LateContext<'_>, valid_idents: &FxHashSet<String>, attrs: &[Attribute]) -> Option<DocHeaders> {
-    /// We don't want the parser to choke on intra doc links. Since we don't
-    /// actually care about rendering them, just pretend that all broken links
-    /// point to a fake address.
-    #[expect(clippy::unnecessary_wraps)] // we're following a type signature
-    fn fake_broken_link_callback<'a>(_: BrokenLink<'_>) -> Option<(CowStr<'a>, CowStr<'a>)> {
-        Some(("fake".into(), "fake".into()))
-    }
-
     if suspicious_doc_comments::check(cx, attrs) || is_doc_hidden(attrs) {
         return None;
     }
@@ -823,8 +815,19 @@ fn check_attrs(cx: &LateContext<'_>, valid_idents: &FxHashSet<String>, attrs: &[
         return Some(DocHeaders::default());
     }
 
-    // Run broken link checker before parsing the document.
-    broken_link::check(cx, attrs);
+    // // Run broken link checker before parsing the document.
+    // broken_link::check(cx, attrs);
+
+    // We don't want the parser to choke on intra doc links. Since we don't
+    // actually care about rendering them, just pretend that all broken links
+    // point to a fake address.
+    #[expect(clippy::unnecessary_wraps)] // we're following a type signature
+    let fake_broken_link_callback = |bl: BrokenLink<'_>| -> Option<(CowStr<'_>, CowStr<'_>)> {
+        // NOTE: temporary change to check if we can handle broken links report from
+        // this function.
+        broken_link::check_v2(cx, &bl, &doc, &fragments);
+        Some(("fake".into(), "fake".into()))
+    };
 
     let mut cb = fake_broken_link_callback;