bjoernager/rust
bjoernager/rust
1
Fork 0
Code Activity

Update comments

Browse source 
Note that the FIXME was removed because this can't be fixed,
`register_renamed` calls LintId::of and there's no LintId for rustdoc
lints when rustc is running.
This commit is contained in:
Joshua Nelson 2021-01-18 22:24:16 -05:00
parent 0517ea7cdc
commit c7535d1361
1 changed files with 1 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

3
compiler/rustc_lint/src/lib.rs
View file

@ -325,7 +325,7 @@ fn register_builtins(store: &mut LintStore, no_interleave_lints: bool) {
// These were moved to tool lints, but rustc still sees them when compiling normally, before // These were moved to tool lints, but rustc still sees them when compiling normally, before
// tool lints are registered, so `check_tool_name_for_backwards_compat` doesn't work. Use // tool lints are registered, so `check_tool_name_for_backwards_compat` doesn't work. Use
// `register_renamed` explicitly. // `register_removed` explicitly.
const RUSTDOC_LINTS: &[&str] = &[ const RUSTDOC_LINTS: &[&str] = &[
"broken_intra_doc_links", "broken_intra_doc_links",
"private_intra_doc_links", "private_intra_doc_links",
@ -337,7 +337,6 @@ fn register_builtins(store: &mut LintStore, no_interleave_lints: bool) {
"non_autolinks", "non_autolinks",
]; ];
for rustdoc_lint in RUSTDOC_LINTS { for rustdoc_lint in RUSTDOC_LINTS {
// FIXME: maybe we could get `register_renamed` to work for tool lints?
store.register_removed(rustdoc_lint, &format!("use `rustdoc::{}` instead", rustdoc_lint)); store.register_removed(rustdoc_lint, &format!("use `rustdoc::{}` instead", rustdoc_lint));
} }
store.register_removed( store.register_removed(