diff --git a/src/librustdoc/passes/collect_intra_doc_links.rs b/src/librustdoc/passes/collect_intra_doc_links.rs
index ed411bf2b8da..0d07f8f0328b 100644
--- a/src/librustdoc/passes/collect_intra_doc_links.rs
+++ b/src/librustdoc/passes/collect_intra_doc_links.rs
@@ -150,7 +150,6 @@ impl<'a, 'tcx> LinkCollector<'a, 'tcx> {
         let variant_name =
             // we're not sure this is a variant at all, so use the full string
             split.next().map(|f| Symbol::intern(f)).ok_or(ErrorKind::Resolve(ResolutionFailure::NotInScope(path_str.into())))?;
-        // TODO: this looks very wrong, why are we requiring 3 fields?
         let path = split
             .next()
             .map(|f| {
@@ -161,7 +160,6 @@ impl<'a, 'tcx> LinkCollector<'a, 'tcx> {
                 }
                 f.to_owned()
             })
-            // TODO: is this right?
             .ok_or(ErrorKind::Resolve(ResolutionFailure::NotInScope(
                 variant_name.to_string().into(),
             )))?;
diff --git a/src/test/rustdoc-ui/intra-link-errors.rs b/src/test/rustdoc-ui/intra-link-errors.rs
index 1a2f24cef242..5bfbdcc495f2 100644
--- a/src/test/rustdoc-ui/intra-link-errors.rs
+++ b/src/test/rustdoc-ui/intra-link-errors.rs
@@ -10,7 +10,6 @@
 //~| NOTE no item named `path::to` is in scope
 //~| HELP to escape
 
-// TODO: why does this say `f` and not `f::A`??
 /// [f::A]
 //~^ ERROR unresolved link
 //~| NOTE this link partially resolves